Int型で取りたいがnullになってしまう

Question

前提・実現したいこと

laravelでtwitter風のサイトを作っています。
フォローする、フォロー解除の機能を実装中に以下ようなエラーが発生しました。

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

Argument 1 passed to App\Models\User::isFollowing() must be of the type int, null given, called in /home/vagrant/code/Laravel/app/Http/Controllers/UserController.php on line 108

該当のソースコード

\Controller\UserController

 //フォロー

    public function follow(User $user)

    {

        $follower = auth()->user();

        //フォローしているか

        $is_following = $follower->isFollowing($user->id);

        if(!$is_following) {

            //フォローしていなければする

            $follower->follow($user->id);

            return back();

        }

    }

    //フォロー解除

    public function unfollow(User $user)

    {

        $follower = auth()->user();

        //フォローしているか

        $is_following = $follower->isFollowing($user->id);

        if($is_following) {

            //フォローしていれば解除する

            $follower->unfollow($user->id);

            return back();

        }

    }

User.php

//フォローする

    public function follow(Int $user_id)

    {

        return $this->follows()->attach($user_id);

    }

    //フォロー解除

    public function unfollow(Int $user_id)

    {

        return $this->follows()->detach($user_id);

    }

    //フォローしているか

    public function isFollowing(Int $user_id)

    {

        return (boolean) $this->follows()->where('followed_id', $user_id)->first(['id']);

    }

    //フォローされているか

    public function isFollowed(Int $user_id)

    {

        return (boolean) $this->followers()->where('following_id', $user_id)->first(['id']);

    }

view\users\index.blade.php

          @if(auth()->user()->isFollowed($user->id))
          <div class="px-2">
            <span class="px-1 bg-secondary text-light">フォローされています</span>
          </div>
          @endif
          <div class="d-flex justify-content-end flex-grow-1">
            @if(auth()->user()->isFollowing($user->id))
            <form action="{{ route('unfollow', ['id' => $user->id]) }}" method="POST">
              {{ csrf_field() }}
              {{ method_field('DELETE') }}
              <button type="submit" class="btn btn-danger">フォロー解除</button>
            </form>
            @else
            <form action="{{ route('follow', ['id' =>
              $user->id]) }}" method="POST">
              {{ csrf_field() }}
              <button type="submit" class="btn btn-primary">フォローする</button>
            </form>
            @endif

    Route::post('users/{id}/follow', 'UserController@follow')->name('follow');
    Route::delete('users/{id}/unfollow', 'UserController@unfollow')->name('unfollow');

$userをvar_dump

object(App\Models\User)[302]
  protected 'fillable' => 
    array (size=6)
      0 => string 'name' (length=4)
      1 => string 'email' (length=5)
      2 => string 'password' (length=8)
      3 => string 'screen_name' (length=11)
      4 => string 'team' (length=4)
      5 => string 'profile_image' (length=13)
  protected 'hidden' => 
    array (size=2)
      0 => string 'password' (length=8)
      1 => string 'remember_token' (length=14)
  protected 'casts' => 
    array (size=1)
      'email_verified_at' => string 'datetime' (length=8)
  protected 'connection' => null
  protected 'table' => null
  protected 'primaryKey' => string 'id' (length=2)
  protected 'keyType' => string 'int' (length=3)
  public 'incrementing' => boolean true

試したこと

User.phpのpublic function isFollowing(Int $user_id)のIntを?Intにしてみましたが、フォローやフォロー解除ができませんでした。
なぜnullになってしまうのか全くわかりませんでした。user_idが取れていまいのでしょうか？

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

laravel 7.3.0
php 7.4.4
Homestead

Answer 1

?をつけたところでnullが渡ればnullです。

<?php
function test(?int $id){
    var_dump($id);
}

test(null);

まず、dump()なりdd()なり入れて想定通りの情報が渡ってきているか確認してください。

    public function follow(User $user)

    {
        dd($user);

        $follower = auth()->user();

        //フォローしているか
        $is_following = $follower->isFollowing($user->id);

user_idが取れていまいのでしょうか？

現状からは「エラーの通り」としか言いようがなかったりします。
$userがどこからどのように取得されて渡されるのか全体の流れが分かるコード、データをご提示ください。

Comments

[naoya1212]

返信ありがとうございます。
$userはログイン中のユーザーデータからとっています。
dd($user)で確認したところ、ログインしているユーザーの情報は取れていることが確認できたので、idは取れていました。User.phpの//フォローしているか//の処理が間違っているのでしょうか？

[m.ts10806]

>ログインしているユーザーの情報は取れていることが確認できた
結果をご提示ください。
データ取れているならモデルの問題ではないはずです。
というか、URLでフォローする・解除するユーザのIDをパラメータとして渡してるならそこからとらないと「自分自身をフォローしてるか」になりませんか？

[mepon]

var_dumpの結果を見た感じだと取得できているように見えないのですが取れてるんですね。
laravelのroute model bindingではrouteのパラメータとcontrollerの引数を一致させる必要があったと認識してますが。
なので$userで受け取りたい場合正しくは
Route::post('users/{user}/follow', 'UserController@follow')->name('follow');
では？

[naoya1212]

失礼いたしました。勘違いで、$userにはデータが入っていませんでした。
なので、フォローする、解除するユーザーのI Dを$userに渡したところ解決いたしました。
変数の情報がわからない時は、var_dumpやddをすることで解決に繋がることを実感いたしました。
ありがとうございました。

[naoya1212]

Route::post('users/{user}/follow', 'UserController@follow')->name('follow');
最初はこのように行ったのですが、
Missing required parameters for [Route: unfollow] [URI: users/{user}/unfollow].
となってしまいましたのでidに変えて行っていました。

[naoya1212]

Route::post('users/{user}/follow', 'UserController@follow')->name('follow');
で、初めは行ったのですが、
Missing required parameters for [Route: unfollow] [URI: users/{user}/unfollow].
となってしまったので{user}をidに変えて行っていました。

[m.ts10806]

おそらく基本的なところかと。
あくまでURIで渡せるのは文字列ですしね。
そこにUserモデルをあてはめようとしてるので不整合が起きてNULLになったということだと思います。（どこにも情報を注入できるタイミングがない）
デバッグは必須です。
特にフレームワークとなると様々な情報が開発者が意識しないところで飛び交うので、確実におさえておく必要があります。

[naoya1212]

ご丁寧にありがとうございます。
URIの部分、もっと勉強します。
とても勉強になりました。ありがとうございました。

Answer 2

$userにデータが渡っていなかった。

//フォロー
    public function follow(User $user,$id)
    {
        $user = User::find($id); 
        //var_dump($user);
        $follower = auth()->user();
        //フォローしているか
        $is_following = $follower->isFollowing($user->id);
         if(!$is_following) {
            //フォローしていなければする
             $follower->follow($user->id);
             return back();
         }
        //return view('sample');
    }

としたところうまくいきました。