🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

Q&A

解決済

2回答

1005閲覧

laravel 一つのレコードだけ削除したい

higa_yomityu

総合スコア41

Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

0グッド

0クリップ

投稿2021/03/17 03:30

##現状

カート内から商品IDを削除する機能は作ることができたのですが、IDが同一のものは全て削除されてしまうという機能になってしまっています。

削除する際に一つのレコードだけを削除したいです。

##コード

DELETE

1public function deleteCart(Request $request){ 2 $user_id=Auth::id();//ログイン者のID 3 $delete=$request->delete;//削除したい商品のID 4 5 $message="一つの商品がカートから削除されました"; 6 7 DB::table('carts')->where('product_id','=',$delete)->where('user_id','=',$user_id)->delete(); 8 9 return view('/deletecart',compact('message')); 10 }

誰がどの商品をカートに入れたかがわかるテーブルです。
このテーブルでは同じ商品を何度もいれることができる構造になっています。

##試したこと
->first()や->limit(1)などは試したのですが、想定の処理を実行できませんでした。
##質問
①一つのレコードだけを削除する方法はありますか?
②そもそもwhere句の条件式が間違っているのでしょうか?

よろしくおねがいします。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2021/03/17 04:06 編集

カート内の商品一覧でユニークidを使いましょう、としか言えない。
guest

回答2

0

ベストアンサー

現実での動作を考えて見ては?
今のプログラムは
0. たけのこの里をカゴにひとつ追加する
0. たけのこの里をカゴにふたつ追加する
0. たけのこの里を戻す

です。これだとカゴの中のどのたけのこの里かは気にしておらずたけのこの里という商品を全て戻すという事になります。

実際に必要な動作は

  1. たけのこの里をカゴにひとつ追加する
  2. たけのこの里をカゴにふたつ追加する
  3. カゴから取り出したたけのこの里を戻す

です。という事は

$delete=$request->delete;//削除したい商品のID


DB::table('carts')->where('product_id','=',$delete)->where('user_id','=',$user_id)->delete();

の部分がそれぞれ

$delete=$request->delete;//手に取った商品のカート内で割り振られたID DB::table('carts')->where('id','=',$delete)->where('user_id','=',$user_id)->delete();

でないといけません

質問者の処理が使えるパターン

  1. たけのこの里をカゴにひとつ追加する
  2. たけのこの里をカゴにふたつ追加する
  3. きのこの山をカゴにふたつ追加する
  4. きのこの山を戻す

カゴの中がたけのこの里だけとなりスッキリします
これはこれで必要な動作です

投稿2021/03/17 04:52

編集2021/03/17 04:53
hentaiman

総合スコア6426

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

higa_yomityu

2021/03/17 06:12 編集

現実世界に置き換えると自分が行っていた処理を映像で見る事ができたので凄く考えやすかったです。 自分が行っていた処理はカート内にあるたけのこの里を全部削除していたのですね。 どのたけのこの里かを判別するためにAutoIncrementを使用することで想定の処理をすることができました! 丁寧な説明ありがとうございます!!!
guest

0

このテーブルでは同じ商品を何度もいれることができる構造になっています

ということなら、カート内の商品一覧画面でどの行なのか判別できるユニークidを使わないと
狙ったような機能にならないですよね。
(AUTO INCREMENT値とか使ってレコードを識別しているんじゃないかと察しますが。)

投稿2021/03/17 04:06

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

higa_yomityu

2021/03/17 06:13

AutoIncrementを使用して想定の処理ができました!ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問