【PHP】プリペアドステートメントを用いたUPDATE実行に失敗する

Question

解決済み

where条件の変数がint型になっていなかった為、sqlが不正な動作をした？
int型に変数をキャストしたところ、正常にUPDATEが走った

ご回答頂きありがとうございました。

経緯

PHP学習中の初心者です。
現在PHPで「指定されたID(PK)のデータを更新する」という関数を作成しようとしています。

	//--------------------------------
	// タスク情報を更新する
	//--------------------------------
	public function updateEntity($param_user_id, $param_limit_date, $param_task_type, $param_title, $param_task_status){


		try{
			// インスタンス生成（データベース接続を行うクラス）
			$pdo = new dbAccessSetup();
			$db = $pdo->beginPdoAccess();

			try{

				// 現在日時生成
				$sys_date = date('Y/m/d H:i:s');

				//
				$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

				// トランザクション開始
				$db->beginTransaction();

				// プレースホルダを用いたアップデート文生成と実行
				$wkPlaseHolderQuery = $db->prepare('update T_TASK set TASK_TYPE =:type, TITLE =:title, LIMIT_DATE =:limit_date, TASK_STATUS =:status, KOUSHIN_DT =:koushin_dt where USER_ID =:user_id;');
				$wkPlaseHolderQuery->bindParam(':type', $param_task_type, PDO::PARAM_STR, 80);
				$wkPlaseHolderQuery->bindParam(':title', $param_title, PDO::PARAM_STR, 300);
				$wkPlaseHolderQuery->bindParam(':limit_date', $param_limit_date, PDO::PARAM_STR, 10);
				$wkPlaseHolderQuery->bindParam(':status', $param_task_status, PDO::PARAM_STR, 1);
				$wkPlaseHolderQuery->bindParam(':koushin_dt', $sys_date, PDO::PARAM_STR, 19);
				$wkPlaseHolderQuery->bindParam(':user_id', $param_user_id, PDO::PARAM_INT);
                // ★A　ここまでは来る
				$wkPlaseHolderQuery->execute();
                // ★B　ここにも来る

				// コミット
				$db->commit();
                // ★C　ここにも来る

			}catch(Exception $param_rollback_ex){
				// ロールバック
				$db->rollBack();
				throw new Exception();
			}

		}catch(Exception $param_ex){
			print '【システムエラー】';
		}

	}

(1)引数に値が入ってきて、
(2)UPDATE文を作成し
(3)実行し、コミットする

という流れの関数で、★ABCのいずれにも来ている事は確認できているのですが、
実データがUPDATEされていません。

何が悪さをしているのか、コードを書き換えて試してもみましたが、
解決出来ず困っています。
何が原因か心当たりのある方がいれば、ご教示頂ければ幸いでございます。

【前提条件】

• テーブル名、およびカラム名は間違えていません。(名前は仮のものです)
• dbAccessSetupクラスでは正常にDB接続が行われています。
• 引数には全て、必ず値が入ります。
• PHP5.3を使用。
• MySQLを使用。

実現したいこと

UPDATEを実データに反映させたい。

以上、何卒宜しくお願いいたします。

修正依頼に対する追記

• PHPバージョン 3.5→ 5.3の誤りでした。大変失礼致しました。
• コードを修正いたしました。Exceptionにて、

		}catch(Exception $param_ex){
			print '【システムエラー】' . $param_ex->getMessage();
			exit;
		}

と改めましたが、例外処理ではキャッチできていないようです。

Comments

[退会済みユーザー]

PHP 3.5なんてつかってるからじゃね

[m.ts10806]

発生している現象・エラーが出ていればエラーメッセージを追記してください。https://teratail.com/help/question-tips#questionTips3-4 Exceptionにて ->getMessage() にてエラーメッセージが取得できるはずです。http://php.net/manual/ja/class.pdoexception.php

[m.ts10806]

また、PHP3.5って今動くのでしょうか。PDOは5.1以上からなのですが（5.3だとしてもそんな古いバージョンを今使うのはいかがなものかと・・）

Answer 1

学習目的なのであれば、MySQL 側にクエリーログを取るように仕込むと、原因が確認しやすいです。

↓らへんが怪しい気がする。
$wkPlaseHolderQuery->bindParam(':user_id', $param_user_id, PDO::PARAM_INT);
キャストが必要とかそんなやつじゃないかなぁ。。。

参考：PHPでデータベースに接続するときのまとめ

Comments

[shakevo]

回答頂きありがとうございます。
すぐに検証してみたいと思います。

[shakevo]

キャストを行ったところ、更新することができる様になりました…。
int型として認識されていなかったということでしょうか？
where条件が原因で間違い無さそうです。ありがとうございました。ベストアンサーとさせて頂きます。

[退会済みユーザー]

PDO::PARAM_INT が罠ですｗ
http://blog.a-way-out.net/blog/2013/12/18/pdo-prepare-statement-numeric-literal-part2/
意図した動作をしないので＾＾；

[shakevo]

勉強になりました…！
ありがとうございます。

Answer 2

単純にwhere USER_ID =:user_idの条件が誤っていいてUpdateが0件だった、という可能性は無いですか？
$param_user_idを確認してみる、もしくはwhere句を外したら全てのレコードが更新されるかを確認してみては
如何でしょうか？

Comments

[shakevo]

where文をはずしたところ、全件更新が走りました。
where条件の変数がint型ではなかったことが原因のようでした。
ご回答頂きありがとうございました。

Answer 3

あまり関係ないかもしれませんが

$db->prepare('update ・・・・ USER_ID =:user_id;');

SQL文のケツのセミコロンをとってみてください