laravel,isFollowingがうまく動かない

Question

初心者です。

laravelでフォロー処理を行なっています。

フォロー、解除する際にフォローしているユーザーなのか、していないユーザーなのかを判断したいです。

またフォローされているかも同じくやりたい次第です。

 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();
    }

   public function followers()
    {
        return $this->belongsToMany(self::class, 'follows', 'followed_id', 'following_id');
    }

    public function follows()
    {
        return $this->belongsToMany(self::class, 'follows', 'following_id', 'followed_id');
    }
// フォローする処理
    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 $this->follows()->where('followed_id', $user_id);
    }

    // フォローされているか
    public function isFollowed(Int $user_id)
    {
        return $this->followers()->where('following_id', $user_id);
    }

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Follows extends Model
{
    protected $fillable = ['following_id', 'followed_id'];
}

ビューでの条件分岐でもうまく判定できていません。一度フォローを外すと常にフォロー解除のボタンしか出てきません。
データベースは削除されています。

@if(Auth::user()->isFollowing($user->id))
                        <form method="POST" action="{{ route('unfollow', ['user' => $user->id]) }}">
                            @csrf
                            <button type="submit" class="btn btn-outline-info btn-sm" style="width: 100%;">フォロー解除</button>
                        </form>
                    @else
                        <form method="POST" action="{{ route('follow', ['user' => $user->id]) }}">
                            @csrf
                            <button type="submit" class="btn btn-outline-info btn-sm" style="width: 100%;">フォローする</button>
                        </form>
                    @endif

追記

Route::post('/mypage/{user}/follows', 'UserController@follow')->name('follow');
Route::post('/mypage/{user}/unfollows', 'UserController@unfollow')->name('unfollow');

public function up()
    {
        Schema::create('follows', function (Blueprint $table) {
            $table->bigIncrements('id');
            $table->bigInteger('following_id')->unsigned();
            $table->bigInteger('followed_id')->unsigned();

            $table->foreign('following_id')->references('id')->on('users')->onDelete('cascade');
            $table->foreign('followed_id')->references('id')->on('users')->onDelete('cascade');
            $table->timestamps();
        });
    }

少し質問が雑になってしまい申し訳ありません。

どなたか知恵をお借りしたいです。
よろしくお願いします。

php7.4.2
laravel6.18

Comments

[Lulucom]

コントローラのunfollowアクション、モデルのリレーションメソッド、中間テーブルの構造、ルーティングも掲載された方が良いかと思います。また、コメントアウトされている部分は（紛らわしいので）掲載しない方が良いかなと思います。
あとfollowアクションは最後にどこかにリダイレクトさせないのでしょうか？

[tenlife]

コメントありがとうございます。
修正しましたが大丈夫でしょうか？
処理後はback()を使用しています。

[tenlife]

userモデルのisFollowingでdd($user_id)をすると値は取れるので、その後の処理に問題があるのでしょうか？

Answer 1

unfollowアクションのif文の条件式は以下のようにすべきではないでしょうか。

    if ($is_following) { // フォロー中の場合

（追記）existsの呼び出し結果を返すべきではないでしょうか。isFollowedメソッドも同様です。

Userモデル

    public function isFollowing($user_id)
    {
        return $this->follows()->where('followed_id', $user_id)->exists();
    }

Comments

[tenlife]

修正しましたが、同じ動作でした。isFollowingを入れるとフォローも解除も動かなくなります。
ただ、userモデルでdd($user_id);を実行すると値は取れていました。
ということはその後のコードに問題があるのでしょうか？

[Lulucom]

ざっと見たところでは問題無い気がしてます。
フォロー（やアンフォロー）したとき、中間テーブルには、想定通りに該当レコードが作成（や削除）されてますか？

[Lulucom]

見落としがありました。existsについて追記しました。

[Lulucom]

isFollowingの戻り値をddなどで確認すると気付けたかもしれないですね。

[tenlife]

いつも丁寧に本当にありがとうございます。とても勉強になっています。
existsは初めて知りました。
レコードがあるかないかを判断してくれるのですね。レコードがあるかないかで処理を分ける今回のような機会に良いですね。
勉強します。
ありがとうございました！