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

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

ただいまの
回答率

89.20%

N+1の問題に対するincludeやjoinで、リレーションモデルのその先のリレーションでN+1を解消するにはどうすればいいでしょうか?

解決済

回答 2

投稿

  • 評価
  • クリップ 2
  • VIEW 5,011

cDm_48

score 172

次のような例の場合で、postに対するcategoryがN+1の問題が生じてしまいます。
こういうケースにおいて、joinなりincludeなどで解消することはできないのでしょうか?

Userに対してはpostのincludeはできますが、categoryに対してはできませんでした。

設計から見直す必要があるのでしょうか?

よろしくお願いします。

 controller

class UserController < ApplicationController
  @users = User.all.include(:posts)
end

 view

- @users.each do |user|
  - user.posts.each do |post|
    = post.name
    = post.category.name <==こういうの
    = post.like_count

 modelのリレーション

User.has_many :posts

Post.has_one :category
Post.belongs_to :user

Category.belongs_to :post
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

@user = User.includes(posts: :category).all


で行けませんか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/05/29 20:47

    ご回答ありがとうございます。
    こういう記法でできるんですね。

    なかなか参照できる記事が見つからなかったので、大変勉強になりました。


    今回の用件上、さらにネストしたリレーションがありまして、
    次のようなケースです。

    Category.belongs_to :book
    Booka.has_one :category


    - @users.each do |user|
    - user.posts.each do |post|
    post.category.book <==こういうケース


    この場合もN+1が発生してしまうため何とかしたいです。

    よろしくお願いします。

    キャンセル

  • 2016/05/29 21:08

    ↑のようなケース色々と試していると
    .includes(posts: { category: :book } )
    でできました。


    しかし、こういうやり方をcontrollerに書くのがセオリーなんでしょうか・・・。

    また、こういうのをもし調べようと思った際に、どういうキーワードで検索したらでますでしょうか?(Railasではこいうのを何と表現しているのかを知りたいです。)

    キャンセル

  • 2016/05/29 21:56

    返答が遅れてすみません。
    ちなみに、
    user has_many :posts
    user has_many :favorites
    post has_one :category
    favorite has_many :comments
    みたいなモデルがあったとしたら、
    User.includes([posts: :category, favorites: :comments]).all
    みたいな書き方も出来ます。

    コントローラーに書くか、モデルに書くかですが、ここはケースバイケースでしょう。
    常にn+1問題が発生するような検索をする場合は、
    modelのhas_many等に:includesオプションをつけてやれば、常にeager_loadingしてくれます。
    この辺は、Railsガイド(http://railsguides.jp/association_basics.html)のアソシエーションの項目に出てきます。

    検索するなら、eager_loading, includes, joinsあたりがキーワードになるでしょうか。

    キャンセル

  • 2016/05/30 12:59

    コントローラーに書くか、モデルに書くかですが、ここはケースバイケースなんですね。
    まだ、私は経験不足なので、ひとまず、Controller毎に書こうとおもいます。

    includesの記法も、まだまだ、勉強不足だったので、大変参考になりました!

    ありがとうございます!

    キャンセル

0

コントローラのアクション内で

@users = User.all
@posts = @users.posts.includes(:category)

でうまくいきませんか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/05/29 16:08

    ありがとうございます。
    しかしそれをすると今度はUserがN+1を起こすと思います。


    仮に
    @users = User.include(:posts)

    としても、

    view側では次のようなループを回したいため、今回のようなやり方では実現できないのかなと思いました。私自身も勉強不足ですので、思っているのと違ってましたらすみません。

    - @users.each do |user|
    - user.posts.each do |post|
    = post.name
    = post.category.name <==こういうの
    = post.like_count


    キャンセル

  • 2016/05/29 16:10

    また、今回はリレーションを必要なものだけ取り出しましたが、
    実際にはUserには他のリレーションもincludeしていたりしています。

    キャンセル

  • 2016/05/29 16:20 編集

    あ、なんか勘違いしていました。すみません。

    修正。

    @users = User.all
    @posts = Post.all.includes(:category)

    でいけませんか?

    追伸)

    確認は`bullet`のgemがおすすめです

    キャンセル

  • 2016/05/29 17:01 編集

    `bullet`のgemはいれてます!

    @posts = Post.all.includes(:category)

    ですと、view側ではどういうしょりになるのでしょうか?


    こういうのはできないのではないかと思います。

    - @users.each do |user|
    - user.posts.each do |post|

    @usersでループを回しのuser毎のpostsを取得したいのですが。

    キャンセル

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

  • ただいまの回答率 89.20%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる
  • トップ
  • Ruby on Railsに関する質問
  • N+1の問題に対するincludeやjoinで、リレーションモデルのその先のリレーションでN+1を解消するにはどうすればいいでしょうか?