PDOのUpdate（ where句）でデータが流し込めないです

Question

お世話になりあmす。

下部の$comb配列があり$comb[$ii][0]の数値がRDSのテーブルsecidに存在し、これをkeyにして、comb[$ii][1]及びcomb[$ii][２]をfor文で順番に格納していきたいのですが、うまくいきません。エラー結果は下部の通りでした。

問題はWHERE secid = $comb[$ii][0]かと思うのですが
このような条件Where句はPDOで許容されないのでしょうか？

実際にはこの条件とプレースホルダに数値及び変数を入れると正しくmysqlにデータがupdateされます。
WHERE secid =8951;
$a=6;
$b=12;

$stmt->bindParam(':first', $a, PDO::PARAM_INT);
$stmt->bindParam(':second', $b, PDO::PARAM_INT);

どなたか、この原因と解決法をご教示願えませんでしょうか。
よろしくお願い申し上げます。

配列$comb
array(57) {
  [0]=>
  array(3) {
    [0]=>
    string(4) "8951"
    [1]=>
    string(2) "6 "
    [2]=>
    string(3) " 12"
  }
  [1]=>
  array(3) {
    [0]=>
    string(4) "8952"
    [1]=>
    string(2) "3 "
    [2]=>
    string(2) " 9"
  }
  [2]=>
  array(3) {
    [0]=>
    string(4) "8953"
    [1]=>
    string(2) "2 "
    [2]=>
    string(2) " 8"

以下,省略

$dsn = 'XXX';
$user = 'XXX';
$password = 'XXX';

try {
  $pdo = new PDO($dsn, $user, $password);
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

  for ($ii=0; $ii <=57; $ii++) {
  $stmt = $pdo->prepare("UPDATE wp5_entitymaster  SET first=:first, second=:second WHERE secid = $comb[$ii][0]");
  $stmt->bindParam(':first', $comb[$ii][1], PDO::PARAM_INT);
  $stmt->bindParam(':second', $comb[$ii][2], PDO::PARAM_INT);
  $stmt->execute();


}

}catch (PDOException $e){
var_dump($e->getMessage());

string(207) "SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '['0']' at line 1"

Answer 1

UPDATEをバルクで処理する

$data=[];
$max=57;
$sql="INSERT INTO wp5_entitymaster(secid,first,secound) VALUES";
for($i=0;$i<$max;$i++){
  $sql.=($i==0?"":",")."(?,@a:=?,@b:=?)";
  $data=array_merge($data,[$comb[$i][0],$comb[$i][1],$comb[$i][2]]);
}
$sql.=" ON DUPLICATE KEY UPDATE first=@a,secound=@b";
//test
print $sql;
print_r($data);
$stmt = $pdo->prepare($sql);
$stmt->execute($data);

Comments

[SugiuraY]

ご回答いただき、ありがとうございます。
参考にさせていただきます。よろしくお願い申し上げます。

Answer 2

  for ($ii=0; $ii <=57; $ii++) {
  $stmt = $pdo->prepare("UPDATE wp5_entitymaster  SET first=:first, second=:second WHERE secid = :secid");
  $stmt->bindParam(':first', $comb[$ii][1], PDO::PARAM_INT);
  $stmt->bindParam(':second', $comb[$ii][2], PDO::PARAM_INT);
  $stmt->bindParam(':secid', $comb[$ii][0], PDO::PARAM_INT);
  $stmt->execute();

より、こっちのほうがいいですね。

  $stmt = $pdo->prepare("UPDATE wp5_entitymaster SET first=:first, second=:second WHERE secid = :secid");
for ($ii=0; $ii <=57; $ii++) {
      $stmt->bindParam(':first', $comb[$ii][1], PDO::PARAM_INT);
      $stmt->bindParam(':second', $comb[$ii][2], PDO::PARAM_INT);
      $stmt->bindParam(':secid', $comb[$ii][0], PDO::PARAM_INT);
      $stmt->execute();
}

Comments

[SugiuraY]

ご回答いただき、ありがとうございます。
すみません、先日このような変数もプレースホルだにいれるべきとご教示いただいたばかりでした。大変失礼いたしました。
ちなみにですが、直接変数をwhere句に使用する場合には、下記のtakasima20様のおっしゃる. $comb[$ii][0]の結合演算子や、{ }でくくる等を使用しなければならないのはSQLのお作法（SQL文内の変数はすべて、この様に記載しなければならない？）という理解でよろしかったでしょうか？

ご教示いただいたのに重ねてで申し訳御座いませんが
よろしくお願い申し上げます。

[退会済みユーザー]

SQLに $（ドルマーク）がある時点で、有無を言わさずアウトです。ドットでつなごうとなんだろうとNG。

[SugiuraY]

早速のご回答ありがとうございます。
承知をいたします。素直に頭に叩き込みます（猛省です）。
よろしくおねが申し上げます。

[SugiuraY]

forの位置も了解いたしました。
ご鞭撻、深謝です。

[退会済みユーザー]

個人情報流出なんて、ほとんどがSQLインジェクションが原因ですからね…。
$sql = "..............."; // ←この行に $ があったらNG　

Answer 3

こんな感じでどうスかね。

$stmt = $pdo->prepare("UPDATE wp5_entitymaster  SET first=:first, second=:second WHERE secid = " . $comb[$ii][0]);

Comments

[SugiuraY]

ご教示いただいた方法で、解決することができました、また{}でくくる方法も許容される様です。よろしくお願い申し上げます。