Railsで無駄なクエリの発行を抑制したい

Question

Ruby on Railsにて、各モデルで次のような関係をもっているとします。

# User: ユーザ
has_many :posts
has_many :reactions

# Post: 記事
belongs_to :user
has_many :reactions

# Reaction: リアクション（記事に対してユーザがつける絵文字）
belongs_to :user
belongs_to :post

このとき、次のように「ユーザがリアクションをつけたかどうか」を取得すると、記事の数だけSELECT文が発行されてしまいます。

# posts#index
@posts = Post.all

# posts/index.json.jbuilder
json.partial! 'post', collection: @posts, as: :post

# posts/_post.json.jbuilder
json.is_reacted post.reactions.exists?(user: current_user)

これを joins などで1つのクエリにまとめることはできますか？
根本的な考え方など指摘がありましたら回答いただければと思います。

---

追記：以下がイメージとして近いのですが、これをSQLで実現できれば考え方をご教示いただきたいです。
Avoid N+1?

Answer 1

n+1問題というやつですね
eager_loadしてあげましょう

Post.eager_load(:reactions).all

Comments

[DaijiroYamada]

ご回答ありがとうございます。ただこの方法だとPostに紐づくReactionの数だけ `SELECT xxx FROM reactions` が発行されるようです。

[YukiYonekura]

すいません、よく読んでいませんでした。
#exists?でクエリが発行されているのですね。

Post.allで全県取得してから絞り込むという処理となっていますが、
以下のようユーザのリアクションしたpostだけを取得するというようにするとよいのではないかと思います。

``` Ruby
Post.joins(:reactions).merge(Reaction.where(user: current_user))
```

[DaijiroYamada]

ありがとうございます。リアクションしたものだけでよければご回答のものでOKなのですが、ユーザがリアクションしたいもの以外にも取得しつつ、リアクションしたものについてtrue/falseを判断できる必要があります。

[chelsy7110]

Post.includes(:reactions)ならeager_loadされますし、post.reactions.present?でリアクションがあるかないかも判断できると思います。