laravelでupdateでエラーが出てしまう

Question

初心者です。

updateで編集機能を実装しています。

送信ボタンを押すと下記のエラーが表示されます

Argument 2 passed to App\Http\Controllers\ReviewController::update() must be an instance of App\Review, string given

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Http\Requests\ReviewRequest;
use App\Skiday;
use App\Review;
use Illuminate\Support\Facades\Auth;

public function edit(Skiday $skiday, Review $review)
    {
        return view('review.edit', ['skiday' => $skiday, 'review' => $review]);
    }

    public function update(ReviewRequest $request, Review $review)
    {
        $review = Review::find($review);
        // dd($request);
        $review->satisfaction = $request->satisfaction;
        $review->review = $request->review;
        $review->save();
        return view('review.show')->with('review', $review);
    }

class Review extends Model
{
    protected $fillable = ['satisfaction', 'review'];

    public function skiday()
    {
        return $this->belongsTo('App\Skiday');
    }
}

class Skiday extends Model
{
    public function review()
    {
        return $this->hasOne('App\Review');
    }
}

@extends('layouts.app')

@section('content')
<div class="card">
  <h5 class="card-header">レビュー編集</h5>
  <div class="card-body">
  <form method="POST" action="/skidays/{{$review->skiday->id}}/review/{{$review->id}}/edit">
    @csrf
    @if ($errors->any())
    <div class="alert alert-danger">
        <ul>
        @foreach ($errors->all() as $error)
            <li>{{ $error }}</li>
        @endforeach
        </ul>
    </div>
    @endif
    <div class="form-row">
      <div class="form-group col-md-6">
        <label for="exampleFormControlSelect1"><i class="fas fa-skiing show-awesome"></i>満足度</label>
        <select class="form-control" name="satisfaction">
          <option>-</option>
          <option>最高</option>
          <option>良い</option>
          <option>普通</option>
          <option>悪い</option>
        </select>
      </div>
    </div>
    <div class="form-group">
      <label for="exampleFormControlTextarea1"><i class="far fa-hand-point-up show-awesome"></i>レビュー</label>
      <textarea class="form-control" name="review" rows="3" placeholder="雪質や混み具合など他に行く人たちに伝えたい事など">{{$review->review}}</textarea>
    </div>
    <button type="submit" class="btn btn-primary">編集する</button>
    <a href="/calendar/{{$skiday->id}}" class="btn btn-secondary">戻る</a>
  </form>
  </div>
</div>
@endsection

Route::get('/skidays/{skiday}/review/{review}/edit', 'ReviewController@edit')->name('review.edit');
Route::post('/skidays/{skiday}/review/{review}/edit', 'ReviewController@update')->name('review.update');

似ているようなエラー
https://stackoverflow.com/questions/30018862/argument-2-passed-to-app-http-controllers-profilecontrollerstore-must-be-an

原因がどこにあるのかわからず困っています。

どなたか知恵を貸していただきたいです。

よろしくお願いします。

php7.4.3
laravel6.18

Answer 1

ルートパラメータが {skiday} と {review} の2つなので、アクションの引数もそれに合わせて以下のようにする必要があるのではないでしょうか。

    public function update(ReviewRequest $request, Skiday $skiday, Review $review)

あるいは、アクションの引数を現状のままとして、ルートパラメータ {skiday} の方を削除すれば良いのかと思います。アクション内で $skiday が不要ならこちらの方が良いかもしれません。

Route::post('/review/{review}/edit', 'ReviewController@update')->name('review.update');

Comments

[tenlife]

丁寧に説明ありがとうございます。とてもわかりやすく感謝しています。
後者の案を使わせていただきました。ありがとうございました。