foreachの中でのINSERTについて

Question

標題の通り、inventoryというテーブルの中に数値($subtotal)を挿入したいのですが、下記のコードを実行したところ以下のエラーメッセージが出ました。
「捕捉した例外: SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens」
変数の数が異なるということでしょうか？どなたかご教授よろしくお願いいたします。

<?php
$pdo = new PDO("mysql:host=xxx; dbname=xxx; charset=utf8", "xxx", "xxx", array( PDO::ATTR_EMULATE_PREPARES => false ) );

    $statement = $pdo->prepare( "SELECT inventory.code,inventory.id ,inventory.zaikoryo , kakaku_list.kakaku , inventory.product_name ,  FROM inventory JOIN kakaku_list ON inventory.code = kakaku_list.No ;" );
    $statement->execute( );
	$results = $statement->fetchAll();

	foreach( $results as $result ){


$subtotal=$result['zaikoryo']*$result['kakaku'];
$id=$result['id'];
try
{
$dsn='mysql:dbname=xxx;host=xxx';
$usr='xxx';
$password='xxx';
$dbh=new PDO($dsn,$user,$password);
$dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$sql='UPDATE inventory SET subtotal=? WHERE id=?';
$stmt=$dbh->prepare($sql);
$data[]=$subtotal;
$data[]=$id;
$stmt->execute($data);

$dbh=null;
}
catch(Exception$e)
{
   echo '捕捉した例外: ',  $e->getMessage(), "\n";
	exit();
}
}

?>

Answer 1

SELECTで抽出したのを変数にとっておいてUPDATEしているのでしょうか？
なにか意味があるのでしょうか？

$sql='UPDATE inventory SET subtotal=zaikoryo*kakaku WHERE id=?';

ではダメなのでしょうか？

まとめ

そうじてこんな感じでしょうか？

try{
  $dsn='mysql:dbname=xxx;host=xxx';
  $usr='xxx';	
  $password='xxx';
  $pdo=new PDO($dsn,$user,$password);
  $pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
  
  $sql ="UPDATE inventory,kakaku_list ";
  $sql.="SET inventory.subtotal=inventory.zaikoryo*kakaku_list.kakaku ";
  $sql.="WHERE inventory.code = kakaku_list.No AND inventory.id=?";
  $stmt = $pdo->query($sql);
}catch(Exception$e){
  echo '捕捉した例外: ',  $e->getMessage(), "\n";
  exit();
}

Comments

[alkeran]

SQLSTATE[42S22]: Column not found: 1054 Unknown column 'kakaku' in 'field list'
というエラーが出ました。'kakaku'がテーブル'inventory'に存在しないからでしょうか？

[yambejp]

ちょっと端折りすぎました
inventoryとkakaku_listを組み合わせてupdateしてください

$sql ="UPDATE inventory,kakaku_list ";
$sql.="SET inventory.subtotal=inventory.zaikoryo*kakaku_list.kakaku ";
$sql.="WHERE inventory.code = kakaku_list.No AND inventory.id=?";
$stmt = $pdo->prepare($sql);

[alkeran]

SELECT表示はうまくできたのですが、データベースに入力されていませんでした^^;
何度も申し訳ありません。どうしてでしょうか？

[yambejp]

よくみたらidもselectで得ているのですね？
つまり全データを更新するということでしょうか？

でればwhere句の「AND inventory.id=?」はいらないので
$stmt = $pdo->query($sql);
でいいような気がします。

[alkeran]

updateされたのですが、たくさんのデータ量のためか、"Internal Server Error"が出ました。
これはどうしようもないのでしょうか？

[yambejp]

inventory.codeとkakaku_list.Noは基本的に１対１で連携していますか？
それであれば待つしかありませんね
画面上はエラーですがバックグラウンドで処理が走っているので

考え方としてはUPDATEにLIMIT 100などつけてやる方法も考えられますが
LIMIT 100,100のようなデータの途中を更新できないので
それほど有効ではないかも
どちらかとえいばWHERE code BETWEEN 1 AND 10000
などcodeの範囲を指定してやったほうが現実的かも

[alkeran]

基本的に１対１ですが、よく調べてみたら、kakaku_list.Noが空白のものが何点かありました。それが原因でしょうか？
範囲を絞って再挑戦したいと思います。

[alkeran]

教えて頂きありがとうございました。

[alkeran]

もうしわけありません追加で。
updateする必要がなければ以下のコードでよろしいでしょうか？
"SELECT SUM(inventory.zaikoryo*kakaku_list.kakaku) FROM inventory,kakaku_list"