質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.34%

  • PHP

    21273questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • MySQL

    6146questions

    MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

  • PDO

    342questions

    PDO(PHP Data Objects)はPHPのデータベース抽象化レイヤーです。

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

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 185

HAC

score 289

 解決済み

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;
        }


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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • asahina1979

    2018/06/08 11:30

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

    キャンセル

  • mts10806

    2018/06/08 11:31

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

    キャンセル

  • mts10806

    2018/06/08 11:33

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

    キャンセル

回答 3

checkベストアンサー

+2

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/08 11:46

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

    キャンセル

  • 2018/06/08 11:51

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

    キャンセル

  • 2018/06/08 12:16

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

    キャンセル

  • 2018/06/08 16:28

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

    キャンセル

+1

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/08 11:53

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

    キャンセル

0

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

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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • PHP

    21273questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • MySQL

    6146questions

    MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

  • PDO

    342questions

    PDO(PHP Data Objects)はPHPのデータベース抽象化レイヤーです。