laravelでの画面遷移について

Question

プログラミング独学三か月、Laravel初心者です。
突如プロジェクトに参加し、Laravelを正式に使用しなければいけなくなりました。
勉強と解読同時に進めているのですがどうしても先に進めなくなってしまったので、お力添えいただけますと幸いです。

■実現したいこと
画面の遷移をしたい
（sectionController.phpのfunctionは既に記載済みだったのでview.bladeに正しいリンクがはれれば実現可能だと思われる）

■試したコード
当方記載コード（ミスはおそらくここ）

<a href="{{ route('admin.sections.create') }}"
   class="btn btn-primary">セクション</a>

前任者の書いたコード

 /*
 * @param  \Illuminate\Http\Request  $request
 * @return \Illuminate\Http\Response
 */
    public function create(Request $request)
{
    $id = $request->query('id');
    $date = $request->query('date');
    //パラメータ
    if(empty($id) || empty($date)) {
        return redirect()->route('admin.home');
    }  
    return view('section.create')->with($_param + compact('section'));
}

■結果
'admin.home'に遷移してしまった（return view('section.create')にしたい）

■原因予測
return redirect()->route('admin.home');とあるのでパラメータの$idと$dateの情報が送れていない

■質問
どのように記載すれば$idと$dateの情報が送れるのか

■参考
HTTPリクエスト
5.5でひとつ古いバージョンですが、見やすかったのでこちら参照しました。
functionのかきかたはありましたが、view側の記載がなく、実装できませんでした。

■環境
Laravel5.6 windows10

■質問追記
そもそもpublic function create(Request $request)の引数として$idと$dateを渡すということが見当違いなのでしょうか？

Answer 1

$request->query

これはクエリ文字列からデータを取得するコードです。クエリ文字列は何なのかというと、

https://teratail.com?id=1

こういうurlの末尾のパラメーターの事ですね。
つまり前任者のプログラムとしては、idとdataパラメーターを渡して欲しいわけです。

Undefined variable: id

このエラーはidパラメーターを渡していないから表示されているわけですね。

名前付きルートでの値の渡し方は、

{{ route('admin.sections.create',['id' => 1, 'data'=>'text']) }}

こんな感じです。

追記

前任者が書いたプログラムにをみてください。

$id = $request->query('id');
$date = $request->query('date');

これは何をしているのかというと、「idというクエリ文字列」と「dateというクエリ文字列」を取得しようとしています。もし取得しようとしているクエリ文字列がなかったら、当然エラーが出ますよね。ないものは取得しようがありません。なのでidとdateが必要というわけです。

そして

（request $request）に対してパラメーターを渡す。

という認識は少し違います。これは「Illuminate\Http\Request」クラスを依存注入しているのです。

$request->query

というのは、簡単にいえばRequestクラスのqueryメソッドを使っているということになります。queryメソッドは送られてきたクエリ文字列を取得する処理が書かれているというわけです。

依存注入についてはこちらに詳しく書かれていますが、もしかしたらphpに置けるオブジェクト指向をもう少し学ばれた方が良いかもしれません。

Comments

[akoro]

'id' => 1の1の部分は卒度変わるので$id設定したのですが、なぜそれでもundefinedなのでしょうか？

[退会済みユーザー]

view.bladeにはそもそも$idは渡していますか？

$id = 1;
return view('view')->with(compact('id'));

例えばこんな感じで書かないと、view側で$idは使えません。

[akoro]

受け渡しされていなかったです。
ちなみに同じくもうひとつのの書いていた第いただいたものだとdata、当方だとdateに当たる変数の$dateもないのですが、なぜ$idだけエラーがでているのですが。。。

[退会済みユーザー]

画面が推移した先の最初の処理、

$id = $request->query('id');

で引っかかっているからですね。ここでidを取得しようとして、idパラメーターがないとエラーになるわけです。試しに$idだけ渡して$dateを抜いてみてください。今度は$dateでエラーが出るはずです。

[akoro]

たしかし$dateでえらーでました。
そもそも引数として（request $request）に対してどのパラメーターを受け渡すのか、の判断ができないでうすが、idとdateを渡すという認識自体は合っているのでしょうか？

[akoro]

難しくて現状解読は困難ですが情報ありがとうございます。

Answer 2

見やすかったのでこちら参照しました。

マイナーバージョンでも1つ違うと適用できなくなるので
「みやすいから」と自己都合で勝手に別のバージョンの情報を参照するのは事故の元です。
きちんと自身が利用しているバージョンのドキュメントを確認してください。

Comments

[akoro]

いろいろ検索書けましたらバージョンが変わる＝機能追加ではないのですね。
完全に対応しない仕様もあるということを初めて知りました。
ご指摘ありがとうございます。

[m.ts10806]

互換性が100%保証されるものではないですし、
バージョンが違う資料を見ていたらそれは何か問題かあっても文句は言えないです。
ドキュメントがバージョンで別にできるということはそれなりの理由があります。

Answer 3

{{ route('admin.section.create', ['id' => $project->getId(), 'date' => $date->date->format('Ymd')]) }

回答でも頂いていたようにURLの設定をみて、パラメータを判断。
コントローラやヴューコンポーザなどに散りばめられていた情報を拾って上記の式になった。

サービスコンテナ公式ドキュメント参照
依存注入が正直まだ理解しきれていないところがあるため、現状これ以上詳しく加筆できません。
随時更新します。

Comments

[m.ts10806]

閉じずにきちんと整理してから投稿すべきかと。誰も急いでませんよ

Answer 4

route('admin.sections.create', [
  'id' => $id,
  'date' => $date,
])

見るならこっち
https://readouble.com/laravel/5.5/ja/urls.html

Comments

[akoro]

参考ページありがとうございます。
Undefined variable: idのエラーが発生してしまったのですが、何が原因でしょうか？