配列で受け取った値がUPDATEできない

質問

お世話になります。質問させていただきます。

複数の一行テキストに入力された値を配列にして受け取り、
まとめてデータベースへUPDATEさせようと勉強しているのですが、
最終の値が全てのカラムにUPDATEされてしまいます。

--------------------------------------------------

if( isset( $_POST['comment'] )) {
foreach ($_POST['comment'] as $value) {

echo $value."<br>";

$sql = "UPDATE テーブル名 SET comment = ?";
$stmt = $conn->prepare($sql);
$stmt->execute(array($value));

}
}

--------------------------------------------------
echo $valueと吐き出すと、正常に値が反映されているのですが、
UPDATE文の「 comment 」カラムには最終の値が全ての
「 comment 」カラムに入ってしまいます。

例えば、４つの一行テキストがあって、

【 id1→「 a 」 】、【 id2→「 b 」 】、【 id3→「 c 」 】、【 id4→「 d 」 】と

したいのですが、

【 id1→「 d 」 】、【 id2→「 d 」 】、【 id3→「 d 」 】、【 id4→「 d 」 】と

最終の「 d 」が全てのカラムに入ってしまいます。

UPDATEさせる前にどの様な処理を施せば良いのでしょうか？
ご存知の方がいらっしゃいましたらアドバイスをお願いいたします。

各バージョンは、
PHP5.3.3、MySQL5.0.95
を使用しています。

よろしくお願いいたします。

回答No.4

noname#244856 · ベストアンサー

>> 依然最終の値のみがUPDATEされる

つまりそれはidを固定値にしているからではないでしょうか？

value | id
--------
　a 　|　1
　b 　|　2
　c 　|　3
　d 　|　4

こうする必要があるのは自明ですよね。

$stmt = $conn->prepare('UPDATE テーブル名 SET comment = ? WHERE id = ?');
$comment = array('a', 'b', 'c', 'd');
foreach ($comment as $i => $value) {
$stmt->execute(array($value, ++$i));
}

お礼

すみません。やっと意味が分かりました。

++$iとして、id「 1,2,3,4,・・・n 」と$valueが入るカラムを
揃えるという感じなのですね。

本当にありがとうございました。
ようやく思い通りに実装できました。

回答No.5

Bnbnbnta101

sqlのupdate文の条件指定しなければダメでしょう。

お礼

ご回答ありがとうございます。

問題解決しました。

参考になるご回答をありがとうございました。

回答No.3

noname#244856

訂正

× $sql = "UPDATE テーブル名 SET comment = ? AND WHERE id = ?";
○ $sql = "UPDATE テーブル名 SET comment = ? WHERE id = ?";

回答No.2

noname#244856

IDの指定が無いのが原因です。しかし…

>> $sql = "UPDATE テーブル名 SET comment = ".$value." where id=id".$i;

他者の回答を否定するのは筋ではないですが、ちょっとこれは…セキュリティ的に問題ありでしょう。普通にプレースホルダを使って

$sql = "UPDATE テーブル名 SET comment = ? AND WHERE id = ?";

これでどうぞ。また、プリペアドステートメントの生成はforeachループ外で行い、foreachループ内で行うのは値をバインドを伴うexecuteのみにしたほうが実行速度は向上します。毎回同じプリペアドステートメントを生成する必要は無いので使いまわしましょう。

PHPでデータベースに接続するときのまとめ
http://qiita.com/mpyw/items/b00b72c5c95aac573b71

お礼

ご回答ありがとうございます。

実は質問をする前に、ご指摘をいただいた

$sql = "UPDATE テーブル名 SET comment = ? WHERE id = ?";

という様にプレースホルダを使用して実装していましたが、依然最終の値のみが
UPDATEされるという結果になったので、WHERE句とidの箇所を削除して質問させていただきました。

もう一度、ご指摘いただいた形式で書き直しましたが、やはり結果は同じです。

$iというのは何か関係があるのでしょうか？

他に思い当たる節がございましたらご回答いただけると幸いです。

回答No.1

mjtanaka

idの指定がないからではないでしょうか。

$i = 1;
if( isset( $_POST['comment'] )) {
foreach ($_POST['comment'] as $value) {

echo $value."<br>";

$sql = "UPDATE テーブル名 SET comment = ".$value." where id=id".$i;
$stmt = $conn->prepare($sql);
$stmt->execute(array($value));

$i++;
}
}

このような感じでどうでしょうか。

お礼

ご回答ありがとうございます。

ご指摘の通り実装すると、「 Call to a member function execute() on a non-object 」という
PDOStatementエラーが返りました。

$stmt->executeの箇所で失敗しているようです。

参考になるご回答をありがとうございました。