laravel テーブルを結合させてログインしているユーザーでフォローしているユーザーのつぶやき表示

Question

前提・実現したいこと

以前も同じような内容で質問させていただきましたが3日考えても分からず、また質問させていただきます。
前提としてタイトル通りの事をやりたいのですが、ログインしているユーザーのつぶやきも表示されてしまいます。以前の質問ではテーブルを結合すればログインしているユーザー自身のつぶやきは表示されないという回答をいただきましたが、記述が分からずにいます。

以前いただいた回答：https://teratail.com/questions/298781
※解決できていないため、未解決のままにしています。

該当ソースコード

FollowsController

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\DB;
use App\User;
use App\Post;
use App\Follow;

class FollowsController extends Controller
{
　public function followList(Post $post, Follow $follow)
    {
        $user = auth()->user();
        $follow_ids = $follow->followingIds($user->id);
        $following_ids = $follow_ids->pluck('followed_id')->toArray();
        $timelines = $post->getTimelines($user->id, $following_ids);
        return view('follows.followList',['timelines' => $timelines]);
    }
}

Followモデル

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Follow extends Model
{

  protected $fillable = [
    'following_id', 'followed_id'
  ];

　public function followingIds(Int $user_id)
  {
      return $this->where('following_id', $user_id)->get();
  }

}

Postモデル

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Post extends Model
{
  protected $fillable = [
      'user_id', 'post',
  ];
　
　public function getTimeLines(Int $user_id, Array $follow_ids)
    {
        $follow_ids[] = $user_id;
        return $this->whereIn('user_id', $follow_ids)->orderBy('created_at', 'DESC')->paginate();
    }
}

ルート

Route::group(['middleware' => 'auth'], function() {
Route::get('/followList', 'FollowsController@followList');
});

followList.blade

@foreach ($timelines as $timelines)
  {{ $timelines->user_id }}
  {{ $timelines->post }}
@endforeach

followsテーブル

+--------------+------------------+------+-----+-------------------+----------------+
| Field        | Type             | Null | Key | Default           | Extra          |
+--------------+------------------+------+-----+-------------------+----------------+
| id           | int(10) unsigned | NO   | PRI | NULL              | auto_increment |
| following_id | int(10) unsigned | NO   | MUL | NULL              |                |
| followed_id  | int(10) unsigned | NO   | MUL | NULL              |                |
| created_at   | timestamp        | NO   |     | CURRENT_TIMESTAMP |                |
+--------------+------------------+------+-----+-------------------+----------------+

postsテーブル

+------------+------------------+------+-----+---------+----------------+
| Field      | Type             | Null | Key | Default | Extra          |
+------------+------------------+------+-----+---------+----------------+
| id         | int(10) unsigned | NO   | PRI | NULL    | auto_increment |
| user_id    | int(10) unsigned | NO   | MUL | NULL    |                |
| post       | varchar(500)     | NO   |     | NULL    |                |
| created_at | timestamp        | YES  |     | NULL    |                |
| updated_at | timestamp        | YES  |     | NULL    |                |
+------------+------------------+------+-----+---------+----------------+

上記の記述ですとフォローしている人のつぶやきは取得できますが、ログインしているユーザー自身のつぶやきも表示されてしまいます。

試したこと

FollowsController

public function followlist()
    {
        $user = \DB::table('follows')
        ->join('posts','posts.user_id','=','followed_id')
        ->get();
        return view('follows.followList',['user' => $user]);
    }

FollowsControllerを上記のように記述したところ、どのユーザーがログインしているに限らずすべてのつぶやきが表示されました。つまりfollowsテーブルの中身と結合しているpostsテーブルのすべての値を取得できます。

バージョン

laravel 5.5.48
php 7.4.5
homestead

ご教授の程、宜しくお願い致します。

Comments

[phper.k]

follow テーブルに following_id と followed_id が同じ値になっているものは存在しませんか？

select * from follows where following_id = followed_id

で確認してみてください。

[hina0823]

ご回答ありがとうございます。
自分自身をフォローしているか確認するという記述でお間違いないでしょうか。
実際に頂いた記述を試したところ、Empty set という結果が返ってきました。
自分自身をフォローして、頂いた記述を試すと当たり前ですが following_id と followed_id には同じ数字が入っています。
Empty set でも自分自身でフォローをしていてもbladeの表示結果は変わりませんでした。

[phper.k]

コードを試すより前に確認しておきtkったから聞きました。
コードの分量がそこそこ多いので、事実を確認しておきたかったのです。

Answer 1

Postモデル内で $follow_ids に $user_id を追加してクエリを実行しているようですが、これが原因ではないでしょうか。

　public function getTimeLines(Int $user_id, Array $follow_ids)
    {
        $follow_ids[] = $user_id;
        return $this->whereIn('user_id', $follow_ids)->orderBy('created_at', 'DESC')->paginate();
    }

Comments

[hina0823]

ご回答ありがとうございます。
Postモデルを

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

public function getTimeLines($user_id)
{
return $this
->Follows()
->where('followed_id', $user_id)
->orderBy('created_at', 'DESC')
->paginate();
}

上記のように記述したところ$timelinesの中に値が入っていない状態となりました。結合の記述が違うのは理解したのですが、具体的にどう書けばいいのかが分からずにいます。

[HRS_O]

元々の記述にあった下記では取得出来ませんでしたか？

```php
return $this->whereIn('user_id', $follow_ids)->orderBy('created_at', 'DESC')->paginate();
```

新しく提示いただいたコードだと実現したいこと異っているように見えます。

また、事前にどのようなSQLが発行されれば期待する値が取得出来るのか事前に自前でクエリを書いて、
toSqlメソッドを利用してご自身の抽出したいSQLになっているか確認したほうが良いと思います。

例えば下記のように記述することで発行されるSQLが取得出来ます。

```php
$this
->Follows()
->where('followed_id', $user_id)
->orderBy('created_at', 'DESC')
->tosql();
```

それでも結合して取得したい場合は、belongsToの挙動を確認したほうが良いと思います。
詳しくは下記の「逆の関係の定義」以降を確認してみてください。
https://readouble.com/laravel/7.x/ja/eloquent-relationships.html#one-to-many

[hina0823]

ご回答ありがとうございます。
ご指摘を頂いた通り、事実と異なったことを行っていました。申し訳ございません。
元々、POSTモデルの記述にありました $follow_ids[] = $user_id; を消して試したところうまく表示されました。またテーブルを結合させての表示も試してみたいと思います。誠にありがとうございました。