【Laravel】Form::modelを使ったアクションで、引数に値を入れているつもりなのに渡せていないとエラーが出る

Question

前提・実現したいこと

LaravelとMySQLを使用して本のタイトル一覧を表示するプログラムを作成し、ログイン機能をつけ、現在はお気に入り機能をつけているところです。

お気に入り追加やお気に入り一覧表示の機能はなんとかつけられたのですが、お気に入り解除の機能でどうしてもエラーが出てしまいます。

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

attach()やdetach()を使用してお気に入りの追加と解除の機能をつけようとしていたのですが、解除の方だけどうしても上手くできずエラーが出てしまいます。
引数がないというエラーだと思うのですが、なぜ渡せていないのかがわかりません。
indexFavorites.blade.php内のForm::model内の記述で$book->idを渡せていると思っていたのですが認識違いでしょうか？

また解除用のフォームのactionが指定したURLと異なっているのですが関係はありますでしょうか？
action="http://〜〜8000/favorites?1"などになっていて、
最後に?1となっている数字の部分が$book->idと同じ値になっているのですが、コントローラでは
return redirect('/favorites');
で元のページに戻るようにしています。
加えて、このリダイレクトの中を('/')や('/books')に変更しても反映されず/favorites?1のままでした。

質問文がわかりにくくすみません、お答えいただけると幸いです。
よろしくお願いいたします。

// エラーメッセージ
Type error: Too few arguments to function App\Http\Controllers\FavoritesController::unfavorite(), 0 passed and exactly 1 expected

該当のソースコード

//web.php
Route::get('/', function (){ //トップページ表示
    return view('welcome');
});

Route::get('login', 'Auth\LoginController@showLoginForm')->name('login');
// login.blade.phpを返すというファンクションが既に設定されている
Route::post('login', 'Auth\LoginController@login')->name('login.post');
Route::get('logout', 'Auth\LoginController@logout')->name('logout.get');

Route::get('signup', 'Auth\RegisterController@showRegistrationForm')->name('signup.get'); //新規ユーザー登録ページ表示
Route::post('/', 'Auth\RegisterController@register')->name('signup.post'); //新規ユーザー登録アクション

/** ログインしないと入れない処理 */
Route::group(['middleware' => ['auth']], function(){
    Route::get('books/create', 'BooksController@create')->name('books.create'); //新規作成ページ表示
    Route::get('books/{id}', 'BooksController@show')->name('books.show'); //詳細ページ表示
    Route::post('books', 'BooksController@store')->name('books.store'); //登録アクション →登録後トップページに飛ばす
    Route::put('books/{id}', 'BooksController@update')->name('books.update'); //更新アクション →更新後詳細ページに飛ばす
    Route::delete('books/{id}', 'BooksController@destroy')->name('books.destroy'); //削除アクション →削除後詳細ページに飛ばす
    
    Route::get('books', 'BooksController@index')->name('books.index'); //一覧ページ表示
    Route::get('books/{id}/edit', 'BooksController@edit')->name('books.edit'); //更新ページ表示
    
    Route::get('favorites', 'FavoritesController@indexFavorites')->name('books.indexFavorites');
    Route::patch('books/{id}', 'FavoritesController@favorite')->name('books.favorite');
    Route::patch('favorites', 'FavoritesController@unfavorite')->name('books.unfavorite');
});

//FavoritesController.php

<?php
namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;

class FavoritesController extends Controller
{
    //
    public function indexFavorites() {
        $user = Auth::user();
        $books = $user->favorites()->paginate(5);
        return view('books.indexFavorites', ['user' => $user, 'books' => $books,]);
    }

    public function is_favorite($book_id) { //お気に入りの中に入っているかのチェック
        $user = Auth::user();
        return $user->favorites()->where('book_id', $book_id)->exists(); 
    }

    public function favorite($book_id) { //単数形
        $user = Auth::user();
        $is_favorite = $this->is_favorite($book_id);
        $exist = $user->$is_favorite; //引数に入っている本がお気に入りに入っているかどうか
        if ($exist){
            return false; //すでにお気に入りに入ってたらfalseを返す
        } else {
            $user->favorites()->attach($book_id);  //お気に入りに入ってなかったらお気に入りに入れて、trueを返す
            return redirect('/favorites');
        }
    }

    public function unfavorite($book_id) { //単数形
        $user = Auth::user();
        $is_favorite = $this->is_favorite($book_id);
        $exist = $user->$is_favorite; //引数に入っている本がお気に入りに入っているかどうか
        if($exist){
            $user->favorites()->detach($book_id);  //お気に入りに入ってたらお気に入りから消して、trueを返す
            return redirect('/favorites');
        } else {
            return false; //お気に入りに入ってなかったらfalseを返す
        }
    }
}

//indexFavorites.blade.php
@extends('layouts.app')
@section('content')

<h1>{{ $user->name }}さんのお気に入り一覧</h1>
@if(count($books) > 0)
  <ul class="book-list">
    @foreach ($books as $book)
      <li>
        {!! link_to_route('books.show', $book->id, ['id' => $book->id]) !!}
        <p>{{ $book->book_title }}</p>
        {!! Form::model($book, ['route' => ['books.unfavorite', $book->id], 'method' => 'patch']) !!}
          {!! Form::submit('お気に入り解除') !!}
        {!! Form::close() !!}
      </li>
    @endforeach
  </ul>
  {!! $books->render() !!}
@else
  <p>お気に入りはまだありません</p>
@endif

<a href="{{ url('/') }}">戻る</a>

@endsection

//show.blade.php
@extends('layouts/app')
@section('content')

  <h1>id: {{ $book->id }} の詳細ページ</h1>
  title: {{ $book->book_title }}<br>
  {!! link_to_route('books.edit', '編集', ['id' => $book->id,]) !!}
  {!! Form::model($book, ['route' => ['books.favorite', $book->id], 'method' => 'patch']) !!}
    {!! Form::submit('お気に入り登録') !!}
  {!! Form::close() !!}
  {!! Form::model($book, ['route' => ['books.destroy', $book->id], 'method' => 'delete']) !!}
    {!! Form::submit('削除') !!}
  {!! Form::close() !!}

  {!! link_to_route('books.index', 'トップに戻る') !!}

@endsection

//BooksController.php
<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Book;

class BooksController extends Controller
{
    // トップページ （一覧ページ）
    public function index() {
        $books = Book::paginate(5);
        return view('books.index', ['books' => $books,]);
    }

    // 新規登録ページ表示
    public function create() {
        $book = new Book;
        return view('books.create', ['book' => $book,]);
    }
    
    // 新規登録アクション
    public function store(Request $request) {
        $this->validate($request, ['book_title' => 'required|max:191',]);
        
        $book = new Book;
        $book->book_title = $request->book_title;
        $book->save();  
        return redirect('/books');
    }
    
    // 詳細ページ
    public function show($id) {
        $book = Book::find($id);
        return view('books.show', ['book' => $book,]);
    }

    // 編集ページ表示
    public function edit($id) {
        $book = Book::find($id);
        return view('books.edit', ['book' => $book,]);
    }
    
    // データ更新アクション
    public function update(Request $request, $id) {
        $book = Book::find($id);
        $book->book_title = $request->book_title;
        $book->save();
        return redirect('/books');
    }

    // データ削除アクション
    public function destroy($id) {
        $book = Book::find($id);
        $book->delete();

        return redirect('/books');
    }
}

//Book.php
namespace App;

use Illuminate\Database\Eloquent\Model;

class Book extends Model
{
    //
    protected $fillable = ['book_title', 'user_id'];

    public function user() {
        return $this->belongsTo(User::class);
    }
}

// User.php
namespace App;

use Illuminate\Notifications\Notifiable;
use Illuminate\Foundation\Auth\User as Authenticatable;

class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];

    public function books() {
         return $this->hasMany(Book::class);
    }

    public function favorites() { //複数形
        return $this->belongsToMany(Book::class);
    }

試したこと

・お気に入り解除用のフォームをループから出す（show.blade.phpで記述してみる）
⇒変化なし
・unfavorite()のメソッド名をremoveFavorite()に変えてみる
⇒変化なし
・お気に入り解除フォームの['route' => ['books.unfavorite', $book->id]の箇所を
actionを使った書き方に変えてみた（実際にどう記述してみたか忘れてしまいました…）
⇒変化なし
・お気に入り解除時のメソッドをpatchからpostにしてみた
⇒変化なし
・Form部分の記述が間違っていないか確認
⇒間違ってはいない気はしますが結果エラーなので自信はありません

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

Mac OS
VScode
Laravel Framework 5.5.50
MySQL 5.7.32
Laravel Collective 5.4.0

Answer 1

Route::patch('favorites', 'FavoritesController@unfavorite')->name('books.unfavorite');

ここでパラメータ定義していないからでは？

Comments

[hazuki_]

ご回答ありがとうございます。
すみません、もう少し詳しく教えていただいても良いでしょうか？
パラメータ定義とは
https://readouble.com/laravel/5.7/ja/routing.html
のルートパラメーターのことでしょうか？

[phper.k]

unfavorite($book_id) と定義してんなら、

Route::patch('favorites/{book_id}', 'FavoritesController@unfavorite')->name('books.unfavorite');

こうしなきゃいかんでしょ

[hazuki_]

お返事が遅くなりすみません！ありがとうございます、試してみます！

[hazuki_]

あとすみません、
>>unfavorite($book_id) と定義してんなら、

Route::patch('favorites/{book_id}', 'FavoritesController@unfavorite')->name('books.unfavorite');

こうしなきゃいかんでしょ
⇒の理由が少しわかっていないです、すみません。
引数の部分をルーティングの際にパスに使わないといけない、という感じでしょうか？

[phper.k]

理由も何も、そういうルールで書くのがLaravel ですから。
ネットの情報つぎはぎでやるんじゃなくて、入門書でも読んで体型的に覚えたほうがいいんじゃないですか？

[hazuki_]

どういうルールなのかがわからないので探してみます！
指摘通りやってみたらエラー内容が変わったので他の修正します。ありがとうございました！

[hazuki_]

※メモです
ルーティングの中の{ }の意味をちゃんとわかっていませんでしたが
https://readouble.com/laravel/5.5/ja/routing.html
のルートパラメーターの箇所を見てわかりました。

unfavorite($book_id)の引数部分は
Route::patch('favorites/{book_id}', 'FavoritesController@unfavorite')->name('books.unfavorite');
のようにしてルート（生成されたURL？）から取り出せるようにしないといけなかったみたいです。
book_idの箇所は勝手に値が入って、URL生成に使われるだけという勘違いをしていました。