Undefined variable: order

事象：
ECサイトの機能として、ユーザからの注文が確定した際にcartsテーブルのorder_countカラム(ユーザがカートに入れた商品数)から値を取得し、商品の情報を管理しているproductsテーブル内のstockカラム(商品ごとの在庫データ)の値をアップデートする処理を行うため、以下のコードを書いたのですが、タイトルに記載のエラーが表示されてしまいます。

該当のソースコード

controller.phpに用意したチェックアウト処理

PHP
1    public function checkout(Cart $cart, Product $product)
2    {
3        $user = Auth::user();
4        $user_id = Auth::id();
5        $mail_data['user'] = $user->name;
6        $items = $cart->getData($user_id);
7        $result = $product->updateStock($items);
8        $mail_data['checkout_items'] = $cart->checkoutCart();
9        Mail::to($user->email)->send(new Thanks($mail_data));
10        return view('checkout');
11    }

Cartモデルの関数。ユーザがカートに入れた商品のidと注文数を取得。

PHP
1    public function getData($user_id)
2    {
3        $items = $this->where('user_id', $user_id)->get('stock_id', 'order_count');
4        return $items;
5    }

　　Productモデルの関数。上記のgetDataで取得したstock_idとorder_countを使って商品在庫を更新。

PHP
1    public function updateStock($items)
2    {
3        foreach ($items as $item) {
4            $stock_id = $item->stock_id; ←データの取得はできている
5
6            $order_count = $item->order_count; ←ここでエラーが発生している様子
7
8            $stock = $this->where('id', $stock_id)->get('stock');
9            $stock->each(function ($old_stock) {
10                $new_stock = $old_stock - $order_count;
11            });
12            $this->where('id', $stock_id)->update(['stock' => $new_stock]);
13        }
14
15        return 0;　←終了の仕方が変かと思われますが、気にしないでいただきたいです。
16    }

試したこと

var_dumpで$order_countの中身を確認しましたが、値は入っているように見えます。
order_countと同じタイミングで取得しようとしたstock_idもデータの取得ができていました。
getData()を内のget()の引数をなしで実行してみたところ、$order_countの値は取得できているように見えましたが、タイトルと同様のエラーが吐かれました。

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

環境：
XAMPP
Apache 2.4.46
PHP 7.4.19
MySQL(MariaDB) 14.14

cartsテーブル内容

productsテーブル内容

updateStock()のforeach内で$itemをvar_dumpした結果

kai0310

2021/06/23 13:52

コードが見づらいので、Markdown記法を利用してください。

退会済みユーザー

2021/06/24 00:07

Cartモデルに対応したテーブルにデータが有ることはわかりましたが、実際Cartモデルを定義しているコードを見ていないので、メンバーにorder_countがあるのかどうかまでは。

G8tekeeper

2021/06/24 18:55

Cartモデルを定義しているコードというのは、cartsテーブルのmigrationファイルのことですか、それとも「class Cart extends Model」から始まるファイルのことを示していますか。

行動規範の内容に同意します

回答1件

ベストアンサー

エラーが発生しているのは

$new_stock = $old_stock - $order_count;

ここのオーダーカウントかと思います。
ここは無名関数のスコープに入っているので外側の変数は使えません。
使うならuseで明示的に引っ張ってくる必要があります

投稿2021/07/13 15:37

mikkame

総合スコア5036

G8tekeeper

2021/07/14 12:27

回答いただきありがとうございます。当該質問に関するエラーは解消されました。質問ついでで恐縮ですが、下記エラーが新たに表示されました。 Object of class App\Models\Product could not be converted to number これは「$old_stock」がオブジェクト型であるため、「order_count」との引き算ができないことから発生しているエラーだと思われます。型キャストやintval()などを試みましたが、変換できないようです。他にオブジェクト型→数値に変換する方法をご存じでしたら、ご教授いただきたいです。