laravel,取得したidを使ってテーブルからの取得がうまくいかない

Question

初心者です。

ユーザーがいいねした投稿を全て取得する、ということをやりたいのですがうまくいきません。

$myfavs = Skiday::where('id', $fav_ski->skiday_id)->get();
いいねした投稿が複数あっても１つしか取得してこない

$myfavs = Skiday::find($fav_ski->skiday_id)->get();
いいねした投稿が１つでも複数取得してしまう

public function user_favs($user)
    {
        // ユーザー取得
        $user = User::findOrFail($user);
　　　　　// 取得したユーザーのイイねレコード全部取得
        $likes_ski = $user->likes()->get();
　　　　　// 取得したレコードの数だけforeachで回す
            foreach($likes_ski as $fav_ski){
　　　　　　　　　 // イイねレコードにあるskiday_idを使ってイイねしたskidayを取得
                $myfavs = Skiday::where('id', $fav_ski->skiday_id)->get();
                // $myfavs = Skiday::find($fav_ski->skiday_id)->get();
            }
        if(isset($myfavs)){
            return view('users.favs')->with('myfavs', $myfavs);
        }else{
            return view('users.favs');
        }
    }

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

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

public function skidays()
    {
        return $this->hasMany('App\Skiday');
    }

    public function likes()
    {
        return $this->hasMany('App\Like');
    }

public function likes()
    {
        return $this->hasMany('App\Like');
    }

リレーション関係をうまく使えば他の書き方もあるのかなと思っています。

自分では良いコードが見つからず、困っています。

どなたか知恵を貸していただきたいです。

よろしくお願いします。

php7.4.2
laravel6.18

Comments

[m.ts10806]

テーブル定義をご提示ください

Answer 1

やり方は色々あると思うのですが、例えば多対多のリレーションを使うのはどうでしょうか。構造が大きく変わってしまうかもしれませんが。

例

• users, skidays, likesの3テーブルを使う。

users
    id

skidays
    id

likes
    user_id
    skiday_id

• UserモデルにbelongsToManyメソッドを定義する。

    public function favs()
    {
        return $this->belongsToMany('App\Skiday', 'likes');
    }

• favs()メソッド（またはfavs動的プロパティ）を使いユーザーの「いいね」を取得する。

　　 // ユーザーが「いいね」したSkidayを取得
    $skidays = $user->favs()->get(); // メソッドを使う場合
    $skidays = $user->favs; // 動的プロパティを使う場合

• 「いいね」（likesテーブルのレコード）はattach/detachなどで操作する（他の方法もある）。

    // $user は User のインスタンス
    // $skidayId は Skiday の id

    $user->favs()->attach($skidayId); // いいね

    $user->favs()->detach($skidayId); // いいねの解除

Comments

[tenlife]

丁寧に説明ありがとうございます！
このようにリレーションで書く方が見やすく、とても参考になります。
ありがとうございました。