Q&A
「INSERTするつもりで既存のレコードを上書きしてしまう事故」は、どのような場面で起きるものを想定していますでしょうか。
CakePHP3のsave()は、主キーが一致する既存のレコードがあるか調べて
あればUPDETE、なければINSERTという動きになりますが、
INSERTするつもりで既存のレコードを上書きしてしまう事故を防ぐために
save()をINSERTに固定したいです。
システムとしてはデータの更新も必要ですのでUPDATEを禁止するわけではなく、
データを挿入する場面では「明示的にINSERTを指定したい」という感じです。
「アップデート処理時のバリデートを常にfalseを返すようにしてみる」
https://qiita.com/ikenji/items/63404023740aa1183306
検索して上記のページを見つけましたが、常にUPDATE禁止になってしまいますし
バリデートで実現というのもしっくりきません。
何か良い方法があったら教えてください。
よろしくお願いします。
追記です(11/14)
- レアケースなのでなるべくわざわざロックはしたくないです。
(主キーかぶりのときはエラー画面になれば十分。データの破壊だけは避けたい。)
- システムとしてはデータの更新も必要ですので対象のTableクラスでUPDATEは禁止したくないです。
(「UPDATE禁止のTableクラス」「UPDATE禁止じゃないTableクラス」の2つ作るってのも
微妙ですよねぇ…)
- 「INSERTするつもりで既存のレコードを上書きしてしまう事故」は
次の2パターンを想定しています。
- 新しいレコードの主キーを発行する仕組みにバグがあり
重複した主キーが発行されてしまった場合、既存データが破壊されてしまう。
0. 新しいレコードの主キーを「既存の最大ID＋1」で発行する場合において、
アクセスが集中し2つのリクエストで同一主キーが発行されると
先にINSERTされたレコードが上書きされてしまう。
回答5件
0
「INSERTするつもりで既存のレコードを上書きしてしまう事故」は次の2パターンを想定しています。
どちらも主キーをデータベースサーバ側で自動生成にしてしまえば発生しないかと思います。
投稿2019/11/14 00:12
総合スコア145183
ご回答ありがとうございます。
おっしゃる通りなのですが、お客様事情でオートナンバーが使えない場合や、
複雑なルールで主キーを発行しなければならない場合があります。
0
背景がわかりませんが、通常はEntityを適切に利用していればそういったことを気にしなくても良いように思えます。
cakephpの実装としておすすめできない方法をお伝えしますが、一般的でないということをご認識ください
saveメソッドに入ってくるEntityがINSERTになるかUPDATEになるかはisNewメソッドで判定可能です。
php
1$entity->isNew(); // 新規登録の場合はtrue
ご質問の要件を満たすならばbeforeSaveでisNew判定を行い、isNewがfalseの場合はreturn falseとすればsaveを中断できます。
明示的にINSERT/UPDATEを切り分けるならば、クエリビルダを使用する方法があります。
適切なバリデーションやイベントが実行されないので、こちらもあまりおすすめはできません。
https://book.cakephp.org/3/ja/orm/query-builder.html#insert
通常のsaveを潰さないので後者の実装が良いとは思います。
投稿2019/11/10 22:57
編集2019/11/10 22:58
総合スコア897
ご回答ありがとうございます。
beforeSaveでisNew判定を行いtrueの場合、save()が必ずINSERTになることは保証されるんでしょうか？
(isNew判定の直後に別リクエストにより同一主キーのレコードがINSERTされる可能性はないでしょうか？)
背景としては、何らかの事情でオートナンバーの主キーが使えない場合において
次の2パターンを気にしています。
(1)新しいレコードの主キーを発行する仕組みにバグがあり
重複した主キーが発行されてしまった場合、既存データが破壊されてしまう。
(2)新しいレコードの主キーを「既存の最大ID＋1」で発行する場合において、
アクセスが集中し2つのリクエストで同一主キーが発行されると
先にINSERTされたレコードが上書きされてしまう。
レアケースですので、わざわざロックするのは大げさですし、
エラー画面になるのも問題なしですが、データが破壊されることだけは避けたいです。
0
nojimage様とのコメントのやりとりの中で解決しました。(最初の回答に記載いただいた方法ではありません)
INSERTに固定するには
（isNewがtrueである状態において）
saveのcheckExistingオプションをfalseにするだけで実現できました。
save($entity, ['checkExisting' => false])
APIリファレンスに下記の記載があり、
•checkExisting: Whether or not to check if the entity already exists, assuming that the entity is marked as not new, and the primary key has been set.
「新規でない(isNew=false)かつ主キーが設定されいる前提で、entityの存在チェックをするかどうか。」のオプション
（つまり、isNew=falseのときしか関係ないオプション）
と思っていましたが
「entityの存在チェックをするかどうか。entityが存在する場合は、新規でない(isNew=false)かつ主キーが設定されいるとみなして処理をする。」
（つまり、entityが存在する場合はUPDATEする）
という解釈が正解でしょうか。（英語に詳しい方、解説お願いします…）
投稿2019/11/16 02:09
総合スコア508
0
ベストアンサー
Cake\ORM\Table クラスにおいて、insert / update を分岐しているのは以下ですので読みましょう。
CakePHP 3.8.6 src/ORM/Table.php#L1999-L2023
ShoheiTai さんの回答にあるように、Model.beforeSaveのイベントで isNew を判定して処理を止めるのも一つの方法ですし、判定ではなく isNew(true) として必ずINSERT処理に行くように変更するのも手です。
さらに心配であれば、対象のテーブルクラスで、 Table::_update() をオーバーライドして、呼び出し時に例外を発生させるようにすれば、そのテーブルクラスのsave系メソッドを利用する限りはupdateはできません。
php
1// SomeTable.php 2public function _update($entity, $data) 3{ 4 throw new Exception('UPDATE禁止'); 5}
投稿2019/11/13 02:29
編集2019/11/13 02:30
総合スコア957
ご回答ありがとうございます。
これは対象のTableクラスをUPDATE禁止にしてしまうということでしょうか？
UPDATEは更新時に必要ですので禁止にはしたくないです。
明示的にINSERTを指定したいだけなのですが、簡単なようで意外に難しいのですね…
saveの代わりになる、insert() メソッドを作ってその中でエンティティに isNew(true) をセットしてからsaveを呼び出す方向はどうでしょうか
DBにid=123のレコードが存在する状態で
Tableクラスに
public function insert($entity) {
$entity->isNew(true);
return $this->save($entity);
}
を作成して、
Controllerで
$hoge['id']=123;
$this->Hoge->insert($hoge);
を実行したところ、id=123のレコードが上書きされてしまいました。
SQL Logを見ると、isNew(true) しているにもかかわらず
レコード存在確認のSQL（SELECT 1 AS "existing" ～）が実行され、UPDATEが実行されていました。
saveのオプションに checkExisting というのがあるので、それを false に指定しましょう。
ありがとうございます。うまくいきました！
save($entity, ['checkExisting' => false]) としたところ
期待通りの挙動になりました。
こんな単純なことだったのですね…
0
直前にSELECTかけて結果０件なら にすれば良いと思います。
投稿2019/11/10 04:20
総合スコア80850
あなたの回答
tips
プレビュー
