PHP・MySQLでトランザクション中にupdateしたレコードを再度、updateで未反映

結論（2018/04/24）

FuelPHPでしか確認していないので分からないが・・・トランザクション開始したらCommitするまで同一レコードの修正は・・・可能

経緯

▼下記のようなコトをしていた。
Ａ１．レコード追加（id=連番、filePath=''、他のカラムたち）
Ａ２．「１」の「lastInsertId()」でファイル名や保存PATHを決めたい
Ａ３．「２」でファイル名や保存PATHを「１」の「カラム：filePath」に保存

「Ａ１」はinsert or updateのどちらかが実行される
「Ａ２」はファイルのアップロード
「Ａ３」は「Ａ１」のレコードをupdateする。
※「Ａ１」の前でトランザクション開始、「Ａ３」のあとにCommit。

▼問題の内容と経緯
Ｂ１．FuelPHPで「updateしたときの戻り値」は「更新したレコード数」なのに・・・「count()」していた。
Ｂ２．FuelPHPのバージョンアップに合わせて、PHPもバージョンアップ（PHP7.2にした）
Ｂ３．PHP7.2にバージョンアップしたら「配列じゃない値はcount()できない」ってトコでつまづいた。
Ｂ４．そのため「Ｂ１」の「updateしたときの戻り値」を「count()」しないように修正
Ｂ５．その結果「Ａ３」の「updateしたときの戻り値」が「0（false）」になった
Ｂ６．「Ｂ５」の結果により例外処理の分岐に入って・・・ロールバックされた

原因は、「Ａ３」で変更内容が無いのにupdateしたこと。
FuelPHPで「updateしたときの戻り値」は「更新したレコード数」ってのは「変更したレコード数」になるらしく「変更していないレコード数」は含まれない。

そのため「Ａ３」で変更内容が無いときの「updateしたときの戻り値」は「0（false）」となる。

更新日時の指定の問題

問題をややこしくしていたのは更新日時の指定。

「Ａ１」と「Ａ３」のそれぞれで更新日時をいれていて、その指定が原因にたどり着く邪魔をしていた。
そういう指定をした私がダメなんだけどね・・・。

Ｃ１.「$now=time()」で現在の日時を取得
Ｃ２．「Ａ１」で更新日を「Ｃ１」の「$now」を設定
Ｃ３．「Ａ３」も更新日を「Ｃ１」の「$now」を設定
※２回updateを実行しようが、「（１回の処理だから更新日は）一緒でいいや」って感じで、このようにしていた。

「Ａ１」で「update」が実行されたときは、更新日時だけは確実に変わるから「更新したレコード数」がある。
「Ａ３」で「update」が実行されると更新日時も使い回しているから「更新したレコード数」が「0」のときがある。
※仮に都度「time()」を指定しても、ミリ秒じゃないと同一日時になってしまうことがある。

更新箇所を確実に作るためにもレコードのバージョンを設定して更新時に「+1」しておけばいいんだろうね。
※参照：MySQLの「+1」した値をupdateしたい

結論（2018/04/23）

以下は、以前に勘違いしていた内容
※FuelPHPが絡まないと勘違いじゃないかもしれない・・・。とりあえずメモとして残しておく。

トランザクション開始したらCommitするまで同一レコードの修正は・・・ムリ

InsertやUpdateされるとCommitされるまでロックがかかるっぽい。

正確には出来るみたいだけど・・・出来てたんだけど出来ないことが分かった。

経緯

１．レコードの追加（or 修正）したい
２．「１」で保存した値を元にファイル保存したい
３．「２」で保存したファイルPATHを「１」のレコードで修正したい

ってなコトをしたかった。具体的にいうと・・・
１．レコード追加（id=連番、filePath=''、他のカラムたち）
２．「１」の「lastInsertId()」でファイル名や保存PATHを決めたい
３．「２」でファイル名や保存PATHを「１」の「カラム：filePath」に保存
・・・こんな感じ。

流れ的に
１．ファイルをup
２．レコードの追加（or 修正）
が望ましい。

でも、「２．レコードの追加（or 修正）」で失敗したときに「１．ファイルをup」のファイルを削除しなくちゃいけない。
DBだったらトランザクションで戻ってくれるし・・・ってことで下記のような処理にしていた。

１．レコードの追加（or 修正）したい
２．ファイルをupしたい
３．upしたPATHを追加（or 修正）したレコードに対して更新したい
※「１」が成功して「３」が「失敗」ってのはレアケースだから、都度対応にするって感じ。

今まで「１」「３」の処理は下記のようにしていた

$result = DBに保存（※FuelPHPが用意してくれている関数を実行）;
if ( !count($result) ) {
  ～ エラー処理 ～
}

※かなり昔に書いた内容を使い回していたので・・・なぜ「count()」にしたのかは記憶にない。
※さらにFuelPHPが用意してくれていたupdate用の関数は成功したレコード数を返してくれる。

このソースで今までエラー（警告）が出なかったけど・・・PHPのバージョンを7.2にしたら警告が出てきた。
→「$result」が配列じゃないから「count()」は使えないっていう内容。

ということで「１」「３」を下記のように修正

$result = DBに保存（※FuelPHPが用意してくれている関数を実行）;
if ( !$result ) {
  ～ エラー処理 ～
}

するとエラーに・・・。調べると「１：$result=1」「３：$result=0」となっている。
で、やっと「トランザクション中に同一レコードの修正って出来ない感じ？」と思い、調べてみて気付いた。

でも・・・ロックかかっているなら「count($result)」のときにcommitで更新されているのは何故なんだろうと思ったら「count(0)」は「1」になるっぽい。

対応案

下記の流れで「１」or「２」で一回コミットすればいいんだけど・・・気持ち悪い。

１．レコードの追加（or 修正）
２．「１」で保存した値を元にファイル保存
３．「２」で保存したファイルPATHを「１」のレコードに追加