RailsでN+1問題を解決するためincludesしたところエラーが出てしまう

Question

■モデルに関して

class Post < ActiveRecord::Base
  has_one :video
  has_one :image
end

class Video < ActiveRecord::Base
  belongs_to :post
end

class Image < ActiveRecord::Base
　belongs_to :post
end

N+1問題を解決するため、@postsの取得にincludesを使用して下記のように記述しました。

■コントローラでの取得

@posts = Post.includes([:video, :image]).limit(5)

ただ、@postsは必ずしもVideoやImageを持つ訳ではなく、取得した@postsがVideoのみの場合にBulletから下記のような警告が出ます。

Remove from your finder: :include => [:Image]

反対に、@postsがImageのみの場合には下記のような警告が出ます。

Remove from your finder: :include => [:Video]

このような場合、@postsの取得はコントローラーでどのように記述するのでしょうか？

Answer 1

はっきり言って、ある程度「eager loadが余計です」のメッセージが出るのは仕方ないです（どちらかと言えば、「全く不要なeager load」への注意喚起のためのものだと思います）。

データを取ってみなければ、videoやimageの存在がわからないので、追加しておいて問題ないとは思います。自分が作っているものでも、状況によってはビューでキャッシュしていて、全部キャッシュにヒットすればeager loadは読まない、ということもありますが、正直その判定を付けるに値するメリットがあるわけでもないでしょう。

Comments

[merryken]

お返事が遅くなり申し訳ございません。

>データを取ってみなければ、videoやimageの存在がわからないので
仰るとおりで、自分の中のモヤモヤも解決しました！