別のcolumnに同じvalueが入力されてしまいます

Question

    //$dataは、keyがcolumn名、valueが値

        $pdo = db_connect();
        try{
            $pdo->beginTransaction();
            $sql = "UPDATE `testdb` SET ";
            foreach ( array_keys($data) as $key ) {
                $sql .= " `{$key}` = :{$key} ";
                $sql .= ",";
            }
            $sql = substr($sql, 0, -1);//最後のコンマを取り除く
            $sql .= " WHERE `id` = :id ";
            $stmh = $pdo ->prepare($sql);
            foreach ($data as $key1 => $value) {
                $stmh ->bindParam(":{$key1}", $value, PDO::PARAM_STR);
            }
            $stmh ->bindParam(':id', $id, PDO::PARAM_STR);
            $stmh ->execute();
            $pdo ->commit();    //データ確定
            $count = $stmh ->rowCount();
        } catch (Exception $ex) {
             $pdo ->rollBack();
             die('update_member_database:'.$ex ->getMessage());
        }
        $pdo = NULL;
    if($count == 1){
        return TRUE;
    }  else {
        return FALSE;
    }
}

以上のようなPHP文を書きました。$dataにKey=>valueを複数入れると一度にupdateできるようにしました。

$data['dataid'] = $query['dataid'];
$data['nameid'] = $query['nameid'];

上記の$dataで実行してみた所,dataid,nameidともにdataidと同じ値になってしまいました。配列に入れる順番を反対にした所、今度はともにnameidの値になってしまいました。
なにかコードの書き方として間違っている部分がありますでしょうか？
bindParamらへんがアヤシイと感じているのですが原因がわからず困っています。入力される$dataはエスケープ処理が必要な為bindParamを"example = 'abcde'"のようなことはできません。

Answer 1

$stmh ->bindParam(...)を$stmh->bindValue(...)に変更してみてください。

bindParam() の第2引数は、いわゆるリファレンス渡しになります。
http://php.net/manual/ja/pdostatement.bindparam.php

指定された変数名にパラメータをバインドする

そのため、foreach 文の中で同じ変数（$value）を何度も "bindParam()" した場合、
その変数に最後に代入された値が、全てのプレースホルダに渡される結果となります。

一方、bindValue() の方はいわゆる値渡しであるため、そのようなことにはなりません。
http://php.net/manual/ja/pdostatement.bindvalue.php

値をパラメータにバインドする

Comments

[namuyan]

参照値だから同じ値が入っていたのですね、納得しました。
無事に解決できました！ありがとうございます！

Answer 2

            foreach ($data as $key1 => $value) {
                $stmh ->bindParam(":{$key1}", $value, PDO::PARAM_STR);

コードは問題なさそうでした。
この２つのコードの間にログを仕込んで$valueの値が入ってるか確認されてはいかがでしょうか？

Comments

[namuyan]

解答有難うございます。
試しましたが$key1も$valueも正しく入っているようです。

後だしの情報になってしまいますがdataidはBIGINT UNSIGNED,nameidはVARCHR(10)と型が違いますが影響しますでしょうか？INTとVARCHRをいままで混ぜて使っていましたが問題は無かったので大丈夫だと考えていましたが

[kaji]

dataidの方はPDO::PARAM_INTを指定すべきかもしれませんが今回は多分関係ないです。
以下を
foreach ($data as $key1 => $value) {
$stmh ->bindParam(":{$key1}", $value, PDO::PARAM_STR);
}
を一旦展開していれてみてはいかがでしょうか？
$stmh ->bindParam(":dataid", $data['dataid'], PDO::PARAM_INT);
$stmh ->bindParam(":nameid", $data['nameid'], PDO::PARAM_STR);

あとは権限があれば
/etc/my.cnfをいじって
general_log = 1
general_log_file = /var/log/mysql/mysql.log
vi /var/log/mysql/mysql.logでupdate生ログを確認してみてはどうでしょうか？

[namuyan]

foreachを展開して上記のようにやってみたら問題なくupdateされました,INTでもSTRでも問題ありません
/etc/mysql/my.cnfのコメントアウトを除いて再起動してみましたがlogは何も作られませんでした

Answer 3

端折って動かしてみましたが、クエリの作りも、bindParamに渡している値も問題なさそうでした。
ご自分の環境で、作成された$sqlや、bindParamに渡している値が想定通りか確認してみてください。

Answer 4

$stmh->bindParam(':id',$id,PDO::PARAM_STR);
を、foreach内に同じように回したらいいんじゃないですか？

すみません、ただ、ぱっと見ただけで思ったので実際にはやってないし、違うかもしれません、、、、