配列の値を条件に一致するレコードをまとめて削除したい

Question

前提・実現したいこと

配列に入っている値（$request['deleteItemsIds']）と以下のテーブルのitem_idが一致した場合に、テーブルからレコードを削除したいです。
例えば、$request['deleteItemsIds']に[1,2]となっていた場合、idが1〜6のレコードを削除したいです。

該当のソースコード

whereInを使って配列と一致する条件を指定し、delete()を試みたのですが、テーブルが更新されません。もし、記載方法に誤りがある様でしたら、ご指摘いただきたいです。

なお、foreachで配列から１つずつ値を取り出して、delete()するとレコードが削除され、期待通りの結果が得られています。

    public function delete(Request $request)
    {
 　　　//こちらはテーブルが更新されない
      ShopCategory::whereIn('item_id', $request['deleteItemsIds'])->delete();

      //こちらはテーブルから条件に一致したレコードが削除される
      foreach($request['deleteItemIds'] as $deleteItemId){
        ShopCategory::where('item_id', $deleteItemId)->delete();
      }
    }

アドバイスよろしくお願いします。

補足情報（FW/ツールのバージョンなど）

Laravel Framework 7.14.1
PHP 7.4.6
mysql Ver 8.0.20

Answer 1

//こちらはテーブルが更新されない
ShopCategory::whereIn('item_id', $request['deleteItemsIds'])->delete();

deleteItemsIds

//こちらはテーブルから条件に一致したレコードが削除される
foreach($request['deleteItemIds'] as $deleteItemId){

deleteItemIds

これが原因だとしたらですが、まずは使用しているパラメータの確認をする事です。
動作がおかしい時に最初にやる事は自分のコードの些細な個所を疑う事じゃないかなと未だに思いますよ。自分も似たような凡ミスやるし。

Comments

[T.Takeda]

ご回答ありがとうございます。
ご指摘の通り余計なsが原因でした。凡ミスで申し訳ございません。

Answer 2

まず、どんなSQLが実行されているのか確認しましょう。

\DB::enableQueryLog();
ShopCategory::whereIn('item_id', $request['deleteItemsIds'])->delete();
dd(\DB::getQueryLog());

パッと見た感じだと問題なく動作しそうな気がしますけど...????

• 参考: Laravel SQLの実行クエリログを出力する

Comments

[T.Takeda]

ご回答ありがとうございます。
'deleteItemsIds'の記載ミスが原因だった様です。
ログ出力の方法について教えていただきありがとうございました。

Answer 3

<?PHP
$del=[];
if((isset($request['deleteItemsIds']) and is_array($request['deleteItemsIds']))){
  $del=$request['deleteItemsIds'];
}
$data=[];
$sql="delete from shop_categories where 0 ";
if(count($del)>0){
  $sql.=" id in(".implode(",",array_fill(0,count($del),"?")).") ";
  $data=array_merge($data,$del);
}
print $sql.";<br>";
print_r($data);

Comments

[T.Takeda]

ご回答ありがとうございます。

以下の箇所では何が行われているのか教えていただけませんでしょうか。
$sql.=" id in(".implode(",",array_fill(0,count($del),"?")).") ";

また、よろしければ質問に記載させていただいてるwhereInでは削除できないのかも教えていただけると幸いです。

よろしくお願いします。

[yambejp]

＞ $sql.=" id in(".implode(",",array_fill(0,count($del),"?")).") ";

ごめんなさい「or」が抜けてました
$sql.=" or id in(".implode(",",array_fill(0,count($del),"?")).") ";

やっていることは$delの数だけ"?"だけで埋めた配列を作り、","で連結しています
つまり$delが3要素あれば?,?,?、4要素あれば?,?,?,?という文字列が作られます

[T.Takeda]

ご回答ありがとうございます。
教えていただいたコードでも試してみたいと思います！