複数行のupdateが機能しない

Question

前提・実現したいこと

updateを複数行に対して行うphpプログラムを製作しています。
しかし下記コードではどうしても動きません。
基本的な質問で申し訳ないのですがご回答よろしくお願い致します

発生している問題・エラーメッセージ

エラーは出ておりません。

該当のソースコード

session_start();
$pdo=new PDO('mysql:host=localhost;dbname=xxx;charset=utf8', 'root', '');

$example_id=$_REQUEST['example_id'];
$example_id=$_REQUEST['aaa'];
$example_id=$_REQUEST['bbb'];
$example_id=$_REQUEST['ccc'];
$example_id=$_REQUEST['ddd'];


$sql=$pdo->prepare('update example set aaa=?,bbb=?,ccc=?,ddd=? where example={$example_id}');

$sql->execute($aaa, $bbb, $ccc, $ddd]);

試したこと

updateを複数行で行うソースの検索など

補足情報（FW/ツールのバージョンなど）

PHP のバージョン: 7.1.13

Answer 1

where条件の　$example_id　を指定しているカラム名、合っていますか？？

example_id　とかではないですか？

（途中でうっかり投稿しました・・追記します）

$example_id=$_REQUEST['example_id'];
$example_id=$_REQUEST['aaa'];
$example_id=$_REQUEST['bbb'];
$example_id=$_REQUEST['ccc'];
$example_id=$_REQUEST['ddd'];

沢山指定している風ですが、
$example_id　には、最後の$_REQUEST['ddd'];　しか入っていませんよ。

where条件も、複数指定にするのであれば、複数条件を指定できるものでなければいけません。
mysqlの条件の作り方についてもう一度確認してください。

ちなみに、

example　というカラムがあるとして、

update example set aaa=?,bbb=?,ccc=?,ddd=? where example=?

で実行した場合、

exampleが一致するデータは全て、aaa,bbb,ccc,dddが同じ値になりますが、

そうしたい　ってことで合ってます？？

やりたいことをもう一度整理すべきのように見えました。

Comments

[Dongrill]

ご回答ありがとうございます。
シングルクゥオーテーションの中では変数が使えないことを忘れておりました。
単純ミスでお騒がせしてしまい申し訳ありませんでした。

Answer 2

$sql=$pdo->prepare('update example set aaa=?,bbb=?,ccc=?,ddd=? where example={$example_id}');
これ、「example={$example_id}」が変数展開されないまま実行されてますよね。

いっそ、
$sql=$pdo->prepare('update example set aaa=?,bbb=?,ccc=?,ddd=? where example_id=?');
にしちゃって、execute()するときに与えてみてはいかがでしょうか。
$sql->execute($aaa, $bbb, $ccc, $ddd, $example_id);

それと

$example_id=$_REQUEST['example_id'];
$example_id=$_REQUEST['aaa'];
$example_id=$_REQUEST['bbb'];
$example_id=$_REQUEST['ccc'];
$example_id=$_REQUEST['ddd'];

変数名、直そうね。

Comments

[Dongrill]

ご回答ありがとうございます。
ご指摘の通り変数展開ができてませんでした。
$sql=$pdo->prepare("update example set aaa=?,bbb=?,ccc=?,ddd=? where example={$example_id}");
とすることで無事アップデートできました。
本当にありがとうございました。

[yambejp]

いや、「example={$example_id}」はNGですから・・・

[Dongrill]

申し訳ありません。
example_id={$example_id}
のミスでした。
ご指摘ありがとうございます。

[退会済みユーザー]

変数を文字列としてそのまま展開してしまうとSQLインジェクションの恐れがあり、避けるべきです。
execute()で与える方法に変えましょう。

[Dongrill]

確かにexample_id={$example_id}も含めてexecuteから与えるべきでした。
盲点になっておりました。
本当にありがとうございました。

Answer 3

プレースホルダのコーディングに不備がある指摘は、他の方と同様ですが、

updateを複数行で行う

という部分が明確でない気がします。

sample

id	category	value1	value2
1	x	a	b
2	x	c	d
3	y	e	f
4	y	g	h

上記のようなテーブルがあったとして「複数行に対して行う」というのが、
1.複数のidに対して
2.categoryを指定して結果複数行
何れかによって発行するSQLは異なります。

1の場合

update sample set value1='hoge1', value2='hoge2' where id in (1,2)

2の場合

update sample set value1='hoge1', value2='hoge2' where category='x'

Answer 4

コードのミスは他の人が指摘していますが、複数行のUPDATEができないってのはWHEREにPRIMARY KEY を指定している、ってオチかも？
CREATE TABLE, CREATE INDEX 程度は提示した方が良いです。

Answer 5

テーブル構成がよくわからないですが例示の際は
テーブル名とカラム名を同じものにしないほうが良いと思います

$sql=$pdo->prepare('update example set aaa=?,bbb=?,ccc=?,ddd=? where example={$example_id}');

なぜexampleカラムにだけ変数をダイレクトに流し込んでいるのでしょうか？

$sql->execute($aaa, $bbb, $ccc, $ddd]);

typoがありますが、ほんとにエラーが出ていないのでしょうか？

$stmt=$pdo->prepare('update example set aaa=?,bbb=?,ccc=?,ddd=? where example=?');
$stmt->execute([$aaa,$bbb,$ccc,$ddd,$example_id);

Comments

[Dongrill]

ご回答ありがとうございます。
シングルクゥオーテーションの中では変数が使えないことを忘れておりました。
単純ミスでお騒がせしてしまい申し訳ありませんでした。

テーブル名の件ですがアドバイスありがとうございます。以後参考にさせて頂きます。