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

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

ただいまの
回答率

88.09%

Mysql、トランザクション処理が効かない理由

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,999

score 22

お世話になります。

下記の要領でトランザクション処理を行いたいのですが、
PDO::beginTransactionが効かずにどちらかが書き込みに失敗していてもどちらかの書き込みが実行されてしまいます。
(getAttributeで確認。※正直この方法できちんと確かめられているのかもわかりません。)

データベースエンジンはInnoDBになっていました。

ソースは下記のサイトを参考に作りました。
http://www.objective-php.net/mvc/model_ex
http://www.objective-php.net/mvc/data

理由のわかるかた、ご教示ください。

PHP 7.3.1
Mysql 8.0.15

// ctrl.php
  require("../〇〇/ModelBase.php");
    require("../●●/ChildrenDao.php");
    require("../〇〇/ChildrenDto.php");

    class ChildrenModel  extends ModelBase
    {
        public function addChildren($brother_data, $sister_data)
        {
            $nowattr = $this->db->getAttribute(PDO::ATTR_AUTOCOMMIT);
            echo $nowattr."<br>"; // 1

            $this->db->beginTransaction();

            $nowattr = $this->db->getAttribute(PDO::ATTR_AUTOCOMMIT);
            echo $nowattr."<br>"; // 1

            $tbl_brother = new TableBrotherDto();
            $tbl_sister = new TableSisterDto();

            $tbl_brother->SetData($brother_data);
            $tbl_sister->SetData($sister_data);

            try
            {
                $brother = new TableBrotherDao();
                $sister = new TableSisterDao();

                $users->table_insert($tbl_brother);
                $login->table_insert($tbl_sister);

                $this->db->commit();
                return true;

            } catch (Exception $e) {
                $this->db->rollBack();
                throw $e;
            }  
        }
    }
// TableChildrenDao.php
    class TableBrotherDao extends ModelBase
    {
        protected $name = 'table_brother';

    ~略~
        public function table_insert($data){
            $res = $this->insert($data);
            return $res;
        }
    }
//ModelBase.php
class ModelBase
{
    private static $connInfo;
    protected $db;
    protected $name;

 ~略~

    public function initDb()
    {
        $dsn = sprintf(
            'mysql:host=%s;dbname=%s;port=3306;',
            self::$connInfo['host'],
            self::$connInfo['dbname']
        );
        $this->db = new PDO($dsn, self::$connInfo['dbuser'], self::$connInfo['password']);
        $this->db->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);
        $this->db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
    }

  ~関数略~

    // INSERTを実行
    public function insert($data)
    {
    ~略~

       $res  = $stmt->execute();
       $return $res;   
    }

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

+2

$this->db->getAttribute(PDO::ATTR_AUTOCOMMIT)
の戻り値が1ならオートコミットしているのでrollbackしないのでは?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/02/19 12:38

    >>yamberjpさん
    この方法で解決しました!
    デフォルトでtrueの場合はPDO::beginTransactionしたところでオートコミットが無効になるわけではないのですね、勉強になりました。
    ベストアンサーにしたいのでよろしければ別記で回答を書いていただけないでしょうか。

    キャンセル

  • 2019/02/19 12:41

    すいません解決したと思ったら今度はコミットができていませんでした。
    他に問題がありそうなのでもう少し考えてみます。

    キャンセル

  • 2019/02/19 12:56

    > 今度はコミット

    もう少し簡単なモデルをつかって試されることをお勧めします
    例示されているものをみても情報が少なくてわかりませんし
    かりに全ソースを載せられてもすべてを検証することはできませんので。

    キャンセル

checkベストアンサー

+1

ChildlenModelTableBrotherDaoはそれぞれModelBaseを継承していますが、初期化を行うinitDbを見る限り、それぞれのdbプロパティに保存されるPDOオブジェクトが別々にnewされたものなのではないでしょうか。あるPDOオブジェクトがbeginTransactionしたからといって他のPDOオブジェクトには無関係です。同じPDOオブジェクトを共有するようにプログラムの構造を見直す必要があると思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/02/20 14:56

    ご回答ありがとうございました。
    たしかにこのやりかただと同一のPDOは使用されていませんね…。
    まったく気づいていませんでした。また新たに構造から考えたいと思います。
    ありがとうございました。

    キャンセル

0

トランザクション開始をtryの中で宣言しては。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/02/19 12:15

    試してみましたが結果は変わらずでした。

    キャンセル

  • 2019/02/19 12:57

    あくまで導入部分です。
    コミットもロールバックもtry-catch内でやっているのですし、宣言が外にあっては流れとしておかしいので。

    キャンセル

  • 2019/02/19 13:02

    スタートラインをコース外に設けてはいけません。

    キャンセル

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

  • ただいまの回答率 88.09%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

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