コントローラーからviewファイルに変数を渡せない（？）

Question

前提・実現したいこと

docker環境でlaravelを学習中です。

お手本のサイト
https://note.com/mukae9/n/n103587d08ef2

コントローラーに定義した変数が、viewファイルにうまく渡っていないようなエラーが出て困っています。（変数未定義らしきエラー）

発生している問題・エラーメッセージ

http://localhost/mycart
にアクセスすると
Undefined variable: message (View: /var/www/testapp/resources/views/mycart.blade.php)
とのエラーが表示されます。

「message」なんて変数はないと言われてしまっているように見えます。

該当のソースコード

↓追記

//ルーティング
//web.php

//(中略)
Route::get('/', 'ShopController@index');

Auth::routes();

Route::get('/mycart', 'ShopController@myCart')->middleware('auth');

Route::post('/mycart', 'ShopController@addMycart');

Route::get('/home', 'HomeController@index')->name('home');

//viewファイル
//mycart.blade.php
@extends('layouts.app')

@section('content')
<div class="container-fluid">
   <div class="">
       <div class="mx-auto" style="max-width:1200px">
           <h1 class="text-center font-weight-bold" style="color:#555555;  font-size:1.2em; padding:24px 0px;">
           {{ Auth::user()->name }}さんのカートの中身</h1>

           <div class="">
               <p class="text-center">{{$message}}</p><br>
               <div class="d-flex flex-row flex-wrap">

                   @foreach($my_carts as $my_cart)
                       <div class="mycart_box">
                           <p>ユーザーID：{{$my_cart->user_id}}</p>
                           <p>ストックID：{{$my_cart->stock_id}}</p>
                       </div>
                   @endforeach

               </div>

               <a href="/">商品一覧へ</a>
           </div>
       </div>
   </div>
</div>
@endsection

//コントローラー
//ShopController.php
<?php

namespace App\Http\Controllers;

use App\Models\Stock;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use App\Models\Cart;


class ShopController extends Controller
{

//中略
    public function myCart()
    {
        $my_carts = Cart::all();
        return view('mycart',compact('my_carts'));
        
    }  

    public function addMycart(Request $request)
   {
       $user_id = Auth::id(); 
       $stock_id=$request->stock_id;

       $cart_add_info=Cart::firstOrCreate(['stock_id' => $stock_id,'user_id' => $user_id]);

       if($cart_add_info->wasRecentlyCreated){
           $message = 'カートに追加しました';
       }
       else{
           $message = 'カートに登録済みです';
       }
       
       $carts = Cart::where('user_id',$user_id)->get();
		

       return view('mycart',compact('my_carts','message'));
   }	    
}

試したこと

コントローラーに当たる「ShopController.php」の末尾（下から３行目）に

return view('mycart',compact('my_carts','message'));
と記載しているので、変数messageは渡せているのでは？
と思い混乱しています。

if文の外側で変数messageを定義しても同じ結果でした。

viewファイルに当たる「mycart.blade.php」の{{$message}}を削除すると、
エラーは出ずページが表示されました。
↓

なので、変数may_cartsはviewファイルに渡せているのに、
変数messageだけ渡せていないことになると思います。

どなたか知恵を貸していただけないでしょうか。

###試したこと追記
hentaimanさんの回答を元に、関数myCartのスコープ内に$message = 'hogehoge';とreturn view('mycart',compact('my_carts','message'));を記載したところ
確かにエラーが出なくなりました。

//コントローラー
//ShopController.php
<?php

namespace App\Http\Controllers;

use App\Models\Stock;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use App\Models\Cart;


class ShopController extends Controller
{

//中略
    public function myCart()
    {
        $my_carts = Cart::all();
        $message = 'hogehoge';//追記
        return view('mycart',compact('my_carts','message'));//追記 
        
    }  

    public function addMycart(Request $request)
   {
       $user_id = Auth::id(); 
       $stock_id=$request->stock_id;

       $cart_add_info=Cart::firstOrCreate(['stock_id' => $stock_id,'user_id' => $user_id]);

       if($cart_add_info->wasRecentlyCreated){
           $message = 'カートに追加しました';
       }
       else{
           $message = 'カートに登録済みです';
       }
       
       $carts = Cart::where('user_id',$user_id)->get();
		

       return view('mycart',compact('my_carts','message'));
   }	    
}

↓実際の画面

しかし、関数addMycart内に記述しているようなif文によって変数messageの内容を変化させる方法がわかりません。

関数myCartでも

       if($cart_add_info->wasRecentlyCreated){
           $message = 'カートに追加しました';
       }
       else{
           $message = 'カートに登録済みです';
       }

のif文を使うとしたらどうすればいいかが思いつきません。

if文とその前後にコピーして関数myCart内に貼り付けたり、
変数messageをグローバル変数にしたりしましたが、
うまくいきませんでした。

###試したこと追記２
hentaimanさんのコメントを元に、「myCartでも同じようにパラメーターを受け取って$cart_add_infoを作る」を実践しました。

//コントローラー
namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use App\Models\Cart;
//中略
     //↓(Request $request)を追記
    public function myCart(Request $request)    {
        $my_carts = Cart::all();

//-----ここから追記-----
        $user_id = Auth::id();
        $stock_id=$request->stock_id;
        $cart_add_info=Cart::firstOrCreate(['stock_id' => $stock_id,'user_id' => $user_id]);

        if($cart_add_info->wasRecentlyCreated){
            $message = 'カートに追加しました';
        }
        else{
            $message = 'カートに登録済みです';
        }        
        
        return view('mycart',compact('my_carts','message'));            
    }  
//-----ここまで追記-----

    public function addMycart(Request $request)
   {
       $user_id = Auth::id(); 
       $stock_id=$request->stock_id;

       $cart_add_info=Cart::firstOrCreate(['stock_id' => $stock_id,'user_id' => $user_id]);

       if($cart_add_info->wasRecentlyCreated){
           $message = 'カートに追加しました';
       }
       else{
           $message = 'カートに登録済みです';
       }
       
       $carts = Cart::where('user_id',$user_id)->get();
		

       return view('mycart',compact('my_carts' , 'message'));

   }

しかし、ページにアクセスすると下記のようなエラーが表示されます。

SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'stock_id' cannot be null (SQL: insert into `carts` (`stock_id`, `user_id`, `updated_at`, `created_at`) values (?, 1, 2020-12-27 13:43:31, 2020-12-27 13:43:31))

根本的にやり方を間違えているのでしょうか。

###試したこと追記3
hentaimanさんのコメントに基づき、関数myCartを書き換えました。

//ShopContoroller

//中略

    public function addMycart(Request $request)
   {
       $user_id = Auth::id(); 
       $stock_id=$request->stock_id;

       $cart_add_info=Cart::firstOrCreate(['stock_id' => $stock_id,'user_id' => $user_id]);

       if($cart_add_info->wasRecentlyCreated){
           $message = 'カートに追加しました';
       }
       else{
           $message = 'カートに登録済みです';
       }

       $carts = Cart::where('user_id',$user_id)->get();
		

       return view('mycart',compact('carts' , 'message'));

   }
 //ここから   
  public function myCart(Request $request){
    $user_id = Auth::id(); 
    $carts = Cart::where('user_id',$user_id)->get();
    //または↓
    //$carts = Cart::all();

    $stock_id=$request->input('stock_id', null)
    $message = '';
    if($stock_id === null){
        $message = 'なにもしません';
    }else{
        $cart_add_info=Cart::firstOrCreate(['stock_id' => $stock_id,'user_id' => $user_id]);
        if($cart_add_info->wasRecentlyCreated){
            $message = 'カートに追加しました';
        }else{
            $message = 'カートに登録済みです';
        }
    }
    return view('mycart',compact('carts' , 'message'));
}  
//ここまで

エディター上で「$message = '';」の部分にエラーがあると表示され、ページにアクセスすると
「syntax error, unexpected '$message' (T_VARIABLE)」のエラーが表示されてしまいました。

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

laravel6.0
php7.3.24
docker20.10.0
mac 10.15.7

Answer 1

function addMycartにはmessageあるけどfunction myCartにはmessageないからmyCartのルートに行ってるだけでしょう　多分ね

追記

どういう意図の設計なのかいまいち分かりませんが、以下のような事をやりたい？

public function myCart(Request $request){
	$user_id = Auth::id(); 
	$carts = Cart::where('user_id',$user_id)->get();
	//または↓
	//$carts = Cart::all();

	$stock_id=$request->input('stock_id', null)
	$message = '';
	if($stock_id === null){
		$message = 'なにもしません';
	}else{
		$cart_add_info=Cart::firstOrCreate(['stock_id' => $stock_id,'user_id' => $user_id]);
		if($cart_add_info->wasRecentlyCreated){
			$message = 'カートに追加しました';
		}else{
			$message = 'カートに登録済みです';
		}
	}
	return view('mycart',compact('carts' , 'message'));
}

Comments

[bokupiroki]

ありがとうございます。
確かにそうかもしれません。

後ほど確認して結果を報告します。

[bokupiroki]

おっしゃる通り、試しに$messageを関数myCart内で定義すればエラーが出なくなりました。
しかし、もともとやりたかったif文を使って表示されるメッセージを変化させる方法がわからずにいます。

試したことを追記したので、質問欄をご確認いただけると幸いです。

[hentaiman]

質問に記載されている範囲の事しか分かりませんが、少なくとも
> $request->stock_id;
を元に
> $cart_add_info=Cart::firstOrCreate(['stock_id' => $stock_id,'user_id' => $user_id]);
で取得したデータを元にifで判断した訳ですから、myCartでも同じようにパラメーターを受け取って$cart_add_infoを作る必要があります

ただ恐らくmycartは現状を表示するだけのアクションだと思うので$message自体が不要で、むしろ何かしら追加する時のアクション先（formタグのactionの指定）をaddMycartのルートにする必要があるでしょう

以上が質問内容から推測できる、やりたいであろう事を実現するための回答です。
そうではなく全く別の条件を元にしたifを使ってmessageを切り替えたいのならその情報も質問に書いてないと分かりません（書いてあっても回答出来るかは分かりませんが）

[bokupiroki]

ありがとうございます。
やりたいことは、推測していただいたことの通りであっています。

「試したこと２」を質問欄に追記しました。

もしかしたら何かもっと最初の段階で根本的なミスをしているのかも知れません。
ほかの部分も含め、引き続き見直しをします。

[hentaiman]

どういうルーティングで各アクションがどういう意図で分けられているのか分かりませんが、てきとーにコードを書き替えてみました。（動作未検証）

[hentaiman]

ちなみに試した事２でエラーが出ているのは、firstOrCreateがstock_idを取れる前提の処理だからなので、stock_idを取れるか取れないかで事前の切り分けが必要です。（それが追記内容）

[bokupiroki]

遅くなり申し訳ありません。
いただいたコードを試した結果を質問欄に記載しました。
引き続き調査いたします。

[hentaiman]

エラーの上の行でセミコロン無いね
こんな事やりたいのかな？って言うのをサラッとコードに落としただけで動作未確認なので、シンタックスエラーなどはテキトー直して使ってください