質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

データベース

データベースとは、データの集合体を指します。また、そのデータの集合体の共用を可能にするシステムの意味を含めます

Q&A

解決済

1回答

3442閲覧

laravel,isFollowingがうまく動かない

tenlife

総合スコア70

Laravel

LaravelとはTaylor Otwellによって開発された、オープンソースなPHPフレームワークです。Laravelはシンプルで表現的なシンタックスを持ち合わせており、ウェブアプリケーション開発の手助けをしてくれます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

データベース

データベースとは、データの集合体を指します。また、そのデータの集合体の共用を可能にするシステムの意味を含めます

0グッド

0クリップ

投稿2020/04/12 09:28

編集2020/04/12 21:00

初心者です。

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

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

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

controller

1 public function follow(User $user) 2 { 3 $follower = Auth::user(); 4 $is_following = $follower->isFollowing($user->id); 5 if(!$is_following){ 6 $follower->follow($user->id); 7 } 8     return back(); 9} 10public function unfollow(User $user) 11 { 12 13 $follower = Auth::user(); 14 $is_following = $follower->isFollowing($user->id); 15 if(!$is_following){ 16 $follower->unfollow($user->id); 17 } 18 19 return back(); 20 }

User

1 public function followers() 2 { 3 return $this->belongsToMany(self::class, 'follows', 'followed_id', 'following_id'); 4 } 5 6 public function follows() 7 { 8 return $this->belongsToMany(self::class, 'follows', 'following_id', 'followed_id'); 9 } 10// フォローする処理 11 public function follow(Int $user_id) 12 { 13 return $this->follows()->attach($user_id); 14 } 15 16 public function unfollow(Int $user_id) 17 { 18 return $this->follows()->detach($user_id); 19 } 20 21 // フォローしているか 22 public function isFollowing(Int $user_id) 23 { 24 return $this->follows()->where('followed_id', $user_id); 25 } 26 27 // フォローされているか 28 public function isFollowed(Int $user_id) 29 { 30 return $this->followers()->where('following_id', $user_id); 31 }

Follows

1<?php 2 3namespace App; 4 5use Illuminate\Database\Eloquent\Model; 6 7class Follows extends Model 8{ 9 protected $fillable = ['following_id', 'followed_id']; 10}

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

view

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

追記

web

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

migration

1public function up() 2 { 3 Schema::create('follows', function (Blueprint $table) { 4 $table->bigIncrements('id'); 5 $table->bigInteger('following_id')->unsigned(); 6 $table->bigInteger('followed_id')->unsigned(); 7 8 $table->foreign('following_id')->references('id')->on('users')->onDelete('cascade'); 9 $table->foreign('followed_id')->references('id')->on('users')->onDelete('cascade'); 10 $table->timestamps(); 11 }); 12 }

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

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

php7.4.2
laravel6.18

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Lulucom

2020/04/12 12:41 編集

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

2020/04/12 21:01

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

2020/04/12 21:46

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

回答1

0

ベストアンサー

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

php

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

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

Userモデル

php

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

投稿2020/04/13 00:25

編集2020/04/13 01:01
Lulucom

総合スコア1899

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tenlife

2020/04/13 00:45

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

2020/04/13 00:52 編集

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

2020/04/13 01:02

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

2020/04/13 04:44

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

2020/04/13 06:02 編集

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問