Laravelでリレーション関係が深いデータを取得するコードのアドバイス（コードレビュー）をして頂きたいです。

解決したいこと（というよりアドバイス頂きたいこと）

現在、Laravel6で小説の投稿サイトを作成しています。

そこで、リレーション関係の深いデータを取得する場面に出会しました。

一応やりたいことはできたのですが、何だかもっと良いコードの書き方があるような気がします。

そのため、自分のコードが綺麗かどうなのか。もっと他によいコードの書き方があるのか。あるいは中間テーブルを作った方が良いのか、などをアドバイスして頂きたく投稿させていただきました。

##やりたいこと
やりたいことはとてもシンプルでして、『あるユーザーがコメントされたレコードを全て取得したい』です。
（あるユーザーがコメントしたレコードではありません。）

##【追記】
やりたいことが説明不足だったため追記します。
下記画像のように、ユーザー（作家）のプロフィールからコメントされた一覧を表示しようと考えています。

テーブル設計

テーブル設計は以下の通りです。

【補足】
・users テーブル・・・ユーザー情報の管理
・novelsテーブル・・・小説情報の管理
・episodesテーブル・・・エピソード情報の管理（小説の中にエピソードが沢山ある）
・commentsテーブル・・・コメント情報の管理（エピソードの中にコメントが沢山ある）

commentsテーブルのuser_idにはコメントをしたユーザーのIDを保存しています。

自分で書いたコード

テーブル設計が上記の通りなので、コメントされたデータを取得しようとすると、usersテーブル→novelsテーブル→episodesテーブル→commentsテーブルというように、関係が深くなってしまいます。

下記のコードでひとまず、自分が取得したいデータは全て取得することができました。

php
1$novels_id  = User::find($user_id)->novels()->pluck('id');
2$episodes_id = Episode::whereIn('novel_id', $novels_id)->pluck('id');
3$comments = Comment::with('episode.novel', 'user')->whereIn('episode_id', $episodes_id)->get();

しかし、あまり綺麗なコードでもないと思うので、もっと他に良い書き方などがありましたら、アドバイス頂きたいです。

##他に考えた方法
他に考えた方法としては

・Commentsテーブルにnovel_idカラム(FK)やwriter_idカラム(FK)などを追加する
・中間テーブルを作る。

などは考えたのですが、それなら上記のコードのままでも良いかなぁ、、、と思いました。

お手数をおかけしますが、アドバイスなど頂けましたら大変助かります。

何卒よろしくお願い致しますm(_ _)m

行動規範の内容に同意します

回答3件

DBの取得に関するロジックが肥大化してしまう要因としては，

保存形式が悪いつまりDB構造が原因の場合
取得のクエリに無駄がある

などが挙げられると思います。もっとシンプルなコードにできるか考える際は1,2の順番で考えていくといいでしょう。

まず前者の視点(1)では，既存の回答でもあるように今のDB構造で問題ないと思います。

では，後者の(2)を見てみるとどうやら無駄な記載がある様にも思えますし，Laravelのリレーションに関する便利な機能(Eloquent ORM)を活用できていない様にも見えます。

やりたいことはとてもシンプルでして、『あるユーザーがコメントされたレコードを全て取得したい』です。

（あるユーザーがコメントしたレコードではありません。）

自分の投稿した小説のエピソードにコメントがあった際はそれを取得するという認識でよろしかったでしょうか？その場合は以下で取得できます。

PHP
1
2// App\Models\User.php
3
4    public function hasComments()
5    {
6        return $this->hasMany(Novel::class)
7            ->has('episodes.comments')
8            ->with(['episodes.comments', 'user'])
9            ->get();
10    }

PHP
1// Controller
2
3    public function __invoke()
4    {
5        $user_id = 1; // 検証用
6        $hoge = User::find($user_id)->hasComments();
7        dd($hoge->toArray()); // 検証用
8    }
9

投稿2021/02/02 18:50

kai0310

総合スコア2070

howaito

2021/02/03 01:46

kaiさん、ご回答ありがとうございますm(_ _)m やりたいことが説明不足だったため、勘違いさせてしまい申し訳ございません。今回やりたかったことは、既に追記した通り、コメントされた一覧を表示する。というものでした（汗）ただ、回答して頂いたようにモデル側でwithメソッドを使う方法はあまり利用していなかったので、この機会に今後は活用させていただこうと思います！ありがとうございますm(_ _)m

kai0310

2021/02/03 06:50

やりたいことについての追記を確認しましたが、まだ何をしたいかが分からないという状況です。ユーザー（作家）のプロフィールからコメントされた一覧とありますがあるユーザとどういう関係があるのですか？ - あるユーザがコメントしたものなのか - あるユーザが投稿したエピソードについたコメントなのかなどもっと具体的に教えてください。第三者に自分の考えていることを伝えるということは大切なことです。

howaito

2021/02/03 09:19 編集

度々申し訳ありません。コメントしていただいた後者（あるユーザが投稿したエピソードについたコメント）が今回実現したいことです！そのため、回答して頂いた方法ですと、下記のよう（正確にはオブジェクトの有無で分岐も必要?）になると思いました。 ``` foreach ($novels as $novel) { foreach ($novel->episodes as $episode) { foreach ($episode->comments as $comment) { dd($comment); } } } ``` 個人的にforeachを二重にするとわかりにくいかな？と考えています。そのため、自分でもforeachを二重にする方法での実装方法は考えついたのですが、現在は質問時に投稿したコードを利用しています。 >第三者に自分の考えていることを伝えるということは大切なことです。おっしゃる通りだと思います。ご指摘ありがとうございますm(_ _)m

行動規範の内容に同意します

リレーションは複雑なのでドキュメントをよく読まないと難しいけど。
Has Many Throughでuserから直接episodeを取得できる。
https://readouble.com/laravel/6.x/ja/eloquent-relationships.html#has-many-through

User

    public function episodes()
    {
        return $this->hasManyThrough(Episode::class, Novel::class);
    }

$user->episodes();

$user->episodes()->comments;

投稿2021/02/03 02:33

kawax

総合スコア10377

howaito

2021/02/03 09:21

この度は回答頂きありがとうございますm(_ _)m アドバイス頂いた方法ですと、「$user->episodes()->comments;」こちらはエラーが発生しました。下記のようにするとcommentを取得することができると思うのですが、間違い無いでしょうか？ ``` foreach ($user->episodes(); as $episode) { foreach ($episode->comments as $comment) { dd($comment); } } ``` ただ、個人的にforeachを二重にするとわかりにくいかな？と考えています。そのため、自分でもforeachを二重にする方法での実装方法は考えついたのですが、現在は質問時に投稿したコードを利用しています。

行動規範の内容に同意します