サイトに乗っている通りにやってもバインドしてからのインサート実行がうまく行きません。

Question

<?php

// 変数の初期化 & 日時の取得
date_default_timezone_set('Asia/Tokyo');
$sql = null;
$res = null;
$dbh = null;
$date = date('Y-m-d H:i:s');

try {
	// DBへ接続
  $dbh = new PDO('mysql:host=localhost; dbname=sample; charset=utf8', 'root', 'root');
  // var_dump($dbh);
  if(!$dbh) {
    echo "接続出来てない";
  }
	// // SQL作成
	// $sql = "INSERT INTO sample_app (
	// 	user_name, email, password, role, created, updated
	// ) VALUES (
	// 	':user_name', ':email', ':password', ':role', ':created', ':updated'
  // )";
  // $stmt = $dbh->prepare($sql);
  // $params = array(
  //   ':user_name' => 'ユーザーネーム',
  //   ':email' => 'メール',
  //   ':password' => 'パス',
  //   ':role' => 0,
  //   ':created' => $date,
  //   ':updated' => '0000-00-00 00:00:00'
  // );
  // $stmt->execute($params);

	// // SQL実行
  // $res = $dbh->query($sql);
  // return $res;
  $user_name = "name";
  $email = "email";
  $password = "pass";
  $role = 0;
  $updated = '0000-00-00 00:00:00';

  // SQL作成
	$stmt = $dbh->prepare("INSERT INTO sample_app (
		user_name, email, password, role, created, updated
	) VALUES (
		':user_name', ':email', ':password', ':role', :created', ':updated'
  )");
  $stmt->bindParam(':user_name', $user_name, PDO::PARAM_STR);
  $stmt->bindParam(':email', $email, PDO::PARAM_STR);
  $stmt->bindParam(':password', $pass, PDO::PARAM_STR);
  $stmt->bindValue(':role', $role, PDO::PARAM_INT);
  $stmt->bindParam(':created', $date, PDO::PARAM_STR);
  $stmt->bindParam(':updated', $updated, PDO::PARAM_STR);
  $stmt->execute();
} catch(PDOException $e) {
	echo $e->getMessage();
	die();
}

// 接続を閉じる
$dbh = null;




?>

エラー文
Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in /Applications/MAMP/htdocs/test.php on line 55
何度やってもどこが間違っているのかよくわかりません。
どなたかご教授ください

Comments

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

>サイトに乗っている通りにやっても
「参考した先が絶対に間違ってない」という根拠がないので、
その参考にしたサイトURLを提示してください。間違っていれば筆者に指摘する必要があります。
あと環境情報も。
全て同じに整えたのなら、コードは書いたとおりに動くので「動かない」という現象は起きません。
大抵は「通りにできていない」からできてません。コードは書いたとおりに動いています。

Answer 1

$stmt = $dbh->prepare("INSERT INTO sample_app (
user_name, email, password, role, created, updated
) VALUES (
':user_name', ':email', ':password', ':role', :created', ':updated'
)");

↑ :created シングルクォートの「閉じ」しかないように見えます。
あと、:role はPARAM_INTでバインドしてるので数値だと思いますが、シングルクォートでくくられてるので、実行時に文字列扱いになってると思います。(エラーメッセージが言ってるのはこっちぽいですね)

(訂正)
プリペアードステートメントのプレースホルダには、全てクォートは不要、が正解でした。
(m.ts10806さん、指摘ありがとうございます)

$stmt = $dbh->prepare("INSERT INTO sample_app (
user_name, email, password, role, created, updated
) VALUES (
 :user_name, :email, :password, :role, :created, :updated
)");

Comments

[退会済みユーザー]

ご返答ありがとうございます。シングルクオーテーションの部分の修正をしましたが、インサート結果が帰ってきませんでした。
シングルクォーテーションでくくられているから文字列扱いとはどのことを仰っていますでしょうか。
他のやり方などでも同様プリペアドステートメントでの結果が帰ってこなく解決方法が見つかりません。

[umau]

':role' ←こうかくと、 '0' こうなっちゃうので、ここはくくらずに :role だけにしないと、Insert文に戻した時に文字扱いになってしまう、という意味です。

[umau]

もう少し丁寧に書くと、プリペアードステートメント使わず、普通にinsert文を書くとしたら、こんな感じでかきますよね。

insert into hoge_tbl ( str1, num2 ) values ( '123', 123 )

クォートでくくると文字として扱われてしまうので、数値項目はそのまま書きます。

プリペアードステートメント使う場合も同じです。

"insert into hoge_tbl ( str1, num2 ) values ( ':str1', :num2 )"

[m.ts10806]

>"insert into hoge_tbl ( str1, num2 ) values ( ':str1', :num2 )"
いや、これだとSQLエラー出るのでは。

[m.ts10806]

https://www.php.net/manual/ja/pdostatement.bindvalue.php
PHPマニュアルでも文字列だろうと何だろうとバインドパラメータ側にクォートはついていません。
bindValueの第2引数で型を指定することで内部的にきちんと（エスケープされたうえで）対応されていると思って良いです。クォートが入ってしまうことでSQLインジェクションしやすくなりますし。。

[umau]

＞m.ts.10806さん
指摘ありがとうございます！実際に動かして試しましたが、エラーにはならないですがWarningが出てInsertできませんでした。曖昧な記憶で言っちゃダメですね、、気をつけます。

[m.ts10806]

はい。記憶は曖昧なものですし誰でもそうなので、PHPマニュアルなどで確実に、
可能ならミニマムコード組んでみるくらいはしてもいいかなと。