Laravel 変数の受け渡しに関して

Question

LaravelでControllerからViewに変数を受け渡す際に疑問点がありましたのでご質問させていただきます。
一度今回の質問点を簡単なプログラムにて再現するために作成した以下のコードを見ていただきたいのですが、もしuserのidが2出会った際はメッセージ用の変数$msgに代入される様になっています。この様にした際、userのidが2以外の数であった際、変数$msgは代入されず未定義エラーとなってしまいます。そこで条件分岐の前で初期化して宣言するなどの対策(*1)があるかと思われます。しかし、条件分岐などで使われる変数が多かった際などに関しては一度初期化するとコードが必然的に肥満化してしまうと思うのですが、どの様な方法が一般的に使われるのでしょうか？

自分で考えた解決方法としては以下が挙げられます。

• 配列で管理する

配列で複数の変数を管理することで全てを宣言する必要がない。

• 条件分岐先でreturnし、変数を渡してあげるなど

以上のことなどが挙げられました。是非皆さんはどの様に行なっているのか。そして何か推奨されている方法などがあればご教示していただきたいと思います！

public function index()
{
    // $msg = ''; (*1)
    if(Auth::id === 2) {
        $msg = "あなたのIDは2です";
    } 
    return view('index')->with(['msg' => $msg]);
}

Comments

[m.ts10806]

このコードだと2以外だとabortでエラーハンドリングされて404.bladeにいくのでは？

[tu789]

ご指摘ありがとうございます。別のページだとまた話が変わってきてしまいますので修正しました。

[yuki84web]

条件分岐後に何らかの$msgを渡すことが前提となっているのに、偽となる条件時に$msgに何も代入しないというのが矛盾しています。初期化だけして使われなかった場合が肥大化になるというのなら、必ず何か代入するような初期化する必要のない処理にすれば良いと思います。

[hentaiman]

既に回答にあるテンプレートのif分岐が一般的だと思うし自分もそうしますが、その方法では解決しなかったんでしょうか？

Answer 1

状況による以外の答えはない。

その例ならコントローラーではなくviewでやることだし。

@if(Auth::id() === 2)
    {{ 'あなたのIDは2です' }}
@endif

少なくともコントローラーで大量の変数使ってたら根本的に間違ってると分かる。

LaravelならViewComposerで「コントローラの外」に出せる。