在PHP中将多行（单个查询）插入MySQL：Prepare-Execute vs. Prepare-Bind-Execute

Question

我正在编写一些PHP，以在POST中接受数字和名称的数组，并将其插入到MySQL表（名为Contacts_table ）中，这是可以正常工作且没有任何错误的版本：

<?php

  // Includes
  require_once 'Admin/Connector.php';

  // Test if payload exists
  if($_POST){

  // Read payload into arrays
      $ar = 0;
      foreach($_POST as $entry){
          $namenum = explode(',', $entry);
          $names[$ar] = $namenum[1];
          $numbers[$ar] = $namenum[0];
          $ar += 1;
      }
      $namenum = NULL;

  // Build SQL query
      $sql = 'INSERT INTO Contact_table (NAME, PHONE) VALUES ';
      $insertQuery = array();
      $insertData = array();
      $n = 0;
      foreach ($numbers as $num) {
        $insertQuery[] = '(?, ?)';
        $insertData[] = $names[$n];
        $insertData[] = $num;
        $n++;
      }
      $sql .= implode(', ', $insertQuery);
      $sql .= ' ON DUPLICATE KEY UPDATE name = COALESCE(VALUES(name), name);';
      $n = NULL;
      $num = NULL;

  // Connect to MySQL database
      $connect = dbconn(PROJHOST,PROJDB,PROJDBUSER,PROJDBPWD);

  // Execute SQL query
      $query = $connect->prepare($sql);
      $query->execute($insertData);
      $insertQuery = NULL;
      $insertData = NULL;
      $sql = NULL;
      $query = NULL;

  // Close connection to MySQL database
      $connect = NULL;

  }

?>

但是，如您所见，我没有在这里使用bindParam（）函数，而是直接将值提供给execute（）函数。 许多人建议我改用bindParam（）来提高服务器性能。 是正确的还是我对这个程序的现状更好？ 我确实尝试使用bindParam编写和运行上述代码的版本：

<?php

  // Includes
  require_once 'Admin/Connector.php';

  // Test if payload exists
  if($_POST){

  // Read payload into arrays
      $ar = 0;
      foreach($_POST as $entry){
          $namenum = explode(',', $entry);
          $names[$ar] = $namenum[1];
          $numbers[$ar] = $namenum[0];
          $ar += 1;
      }
      $namenum = NULL;

  // Build SQL query
      $sql = 'INSERT INTO Contact_table (NAME, PHONE) VALUES ';
      $insertQuery = array();
      $insertData = array();
      $n = 0;
      foreach ($numbers as $num) {
        $insertQuery[] = '(?, ?)';
        $insertData[] = $names[$n];
        $insertData[] = $num;
        $n++;
      }
      $sql .= implode(', ', $insertQuery);
      $sql .= ' ON DUPLICATE KEY UPDATE name = COALESCE(VALUES(name), name);';
      $n = NULL;
      $num = NULL;

  // Connect to MySQL database
      $connect = dbconn(PROJHOST,PROJDB,PROJDBUSER,PROJDBPWD);

   // Prepare SQL query 
      $query = $connect->prepare($sql);

  // Bind variables
      foreach($insertData as $key => &$ins) {
          $connect->bindParam($key+1,$ins);
      }

  // Execute SQL query
      $query->execute();
      $insertQuery = NULL;
      $insertData = NULL;
      $sql = NULL;
      $query = NULL;
      $key = NULL;
      $ins = NULL;

  // Close connection to MySQL database
      $connect = NULL;

  }

?>

但是此代码拒绝运行并返回致命错误– 调用未定义的方法PDO :: bindParam（） 。 我在这里做错了什么？ 我知道，如果在循环中包含execute（） ，则可以编写简单得多的代码，但这会产生多个查询，而我不惜一切代价避免这样做。 我的目标是无论如何都可以进行单个查询。

Answer 1

你不能$connect->bindParam($key+1,$ins); 。 因为PDO对象没有这种方法。 只有PDOStatement具有。 这就是为什么您收到错误消息的原因。

你应该 ：

 $query->bindValue($key+1,$ins);

您应该使用bindValue因为如果没有，则所有插入的值都将获得相同的值（调用execute之前$ ins中的最后一个）。