phpからMySQLへの書き込みでの問題

Question

phpからMySQLへの書き込みを行いたい

何故かNo0のデータにすべて書き込もうとしてしまうのが現状です。
入力データはNo81スタートで0は入っていないことをコード中のvar_dump($tmp)で確認しました。
ON DUPLICATE KEY UPDATEをつけたら全て0に書かれてしまいデータのどれかが間違っているというわけではないようです。
SQL文の間違いではないかと思うのですが、自分で検索した限り間違いはないように思うのです。
SQLの主キー、ユニークはNoになっています。

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

SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '0' for key 'PRIMARY'

該当のソースコード

foreach($dataArr as $tmp){
    //var_dump($tmp);
    dbset($tmp);
}
var_dump("ok222");
function dbset($set){
    try{
        $sql_str="INSERT INTO `image_data`(`No`, `title`, `img`, `img_name`, `time`, `comment`, `Year`, `Seasons`, `Class`, `link_name`,`link_url`) 
                                    VALUES (:no, :title, :img, :name,     :date,   :comment, :year,   :season,   :class, :link_name, :link_url)";
        //文字エンコーディング
        $dbh = new PDO(SQLSETTING, USERNAME, PASSWORD,
            array(
                PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
                PDO::MYSQL_ATTR_MULTI_STATEMENTS => false,
                PDO::ATTR_EMULATE_PREPARES => false
            )
        );
        $stmt = $dbh->prepare($sql_str);
        $stmt->setFetchMode(PDO::FETCH_ASSOC);
        $a = $set["no"];
        $stmt->bindParam(':no', $a, PDO::PARAM_STR);
        $b = $set["title"];
        $stmt->bindParam(':title', $b, PDO::PARAM_STR);
        $c = $set["img"];
        $stmt->bindParam(':img', $c, PDO::PARAM_STR);
        $a = $set["name"];
        $stmt->bindParam(':name', $a, PDO::PARAM_STR);
        $b = $set["date"];
        $stmt->bindParam(':date', $b, PDO::PARAM_STR);
        $c = $set["comment"];
        $stmt->bindParam(':comment', $c, PDO::PARAM_STR);
        $a = $set["year"];
        $stmt->bindParam(':year', $a, PDO::PARAM_STR);
        $b = $set["season"];
        $stmt->bindParam(':season', $b, PDO::PARAM_STR);
        $c = $set["class"];
        $stmt->bindParam(':class', $c, PDO::PARAM_STR);
        $a = $set["link_name"];
        $stmt->bindParam(':link_name', $a, PDO::PARAM_STR);
        $b = $set["link_url"];
        $stmt->bindParam(':link_url', $b, PDO::PARAM_STR);
        //SQL実行
        $stmt->execute();
    }catch(PDOException $e){
        echo $e->getMessage();
    }finally{
        echo "func_end";
    }
}

array(12) {
  ["no"]=>
  int(84)
  ["title"]=>
  string(27) "ツマグロヒョウモン"
  ["img"]=>
  string(20357) "????JFIF


``??>CREATOR: gd-jpeg v1.0 (using IJG JPEG v90), default quality
バイナリデータは省略します。"
  ["name"]=>
  string(8) "04_09_26"
  ["date"]=>
  string(8) "04_09_26"
  ["comment"]=>
  string(6) "交尾"
  ["year"]=>
  string(2) "04"
  ["season"]=>
  string(3) "aut"
  ["class"]=>
  string(4) "tyou"
  ["link_name"]=>
  string(64) "「リンク名」"
  ["link_url"]=>
  string(48) "http:/hogehoge.co.jp/hoge.shtml"

Comments

[m.ts10806]

bindParamには副作用があるので可能な限りbindValueを採用したほうが良いです。
https://qiita.com/_dozen_/items/f6239cf1bdab6f8b0026

[can1980]

驚きました。
まさしく同じ現象でした。
文字型であった事によるエラーであったようです。
bindParam→bindValueで、何の問題もなくSQLにデータ挿入できました。
bindParamは使わない方がいいみたいですね。

Answer 1

＞m.ts10806
＞
＞2022/01/05 05:20
＞
＞bindParamには副作用があるので可能な限りbindValueを採用したほうが良いです。
＞https://qiita.com/_dozen_/items/f6239cf1bdab6f8b0026

ご指摘御いただいた個所を修正したところ、意図したとおり動作しました。
bindParamの副作用だったようです。

Comments

[退会済みユーザー]

そのqiitaの記事を読んだあとなら理解できると思いますが、
$a, $b, $cを繰り返し使いまわししていて、
bindValue()なら代入時点で評価されるものが、
bindParam()だとクエリ実行時点で変数が評価され、
$a = $set["link_name"];
が
:no
:name
:year
:link_name
に適用されようとしたものと思われます。
直接 $set["～"] を充てたらまた違ったことになったかもしれませんが、
INSERT INTO文の実行くらいならおとなしくbindValue()を使うべきです。

Answer 2

$stmt->bindParam(':no', $a, PDO::PARAM_INT);

では？　ただ、コードが断片的で前後のコードもなく、また、データベースの定義などの情報もなく判断材料に乏しいため、正直これでなおるかどうかは分かりません。

Comments

[can1980]

PARAM_INTの指摘ありがとうございます。見落としてました。