bulletでN+1問題が解決されない箇所がある。

Question

前提・実現したいこと

N+1問題で、解決される箇所とされない箇所があります。

bulletを用いてN+1問題に該当しているページがないか調べていたところ、rooms#index（他ユーザーからのメッセージ一覧ページ）において下記の警告が出ました。

user: hoge
GET /rooms/index
USE eager loading detected
  Entry => [:room]
  Add to your query: .includes([:room])
Call stack
  C:/hoge/app/controllers/rooms_controller.rb:26:in `block in index'
  C:/hoge/app/controllers/rooms_controller.rb:25:in `index'


user: hoge
GET /rooms/index
USE eager loading detected
  Entry => [:user]
  Add to your query: .includes([:user])
Call stack
  C:/hoge/app/lang_land/app/views/rooms/index.html.erb:7:in `block (2 levels) in _app_views_rooms_index_html_erb__1042494587_155611800'
  C:/hoge/app/lang_land/app/views/rooms/index.html.erb:6:in `block in _app_views_rooms_index_html_erb__1042494587_155611800'
  C:/hoge/app/lang_land/app/views/rooms/index.html.erb:2:in `_app_views_rooms_index_html_erb__1042494587_155611800'

こちらが元々のrooms_controller.rbです。

  def index
    @rooms = Room.all
    @user = @current_user
    @currentEntries = @current_user.entries
    myRoomIds = []
    @currentEntries.each do | entry |
      myRoomIds << entry.room.id
    end
    @anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?', @user.id)
    @currentRooms = @current_user.rooms
  end
end

警告は、
Entry => [:user]
Entry => [:room]
この2点に関しての様なので、警告の内容に従って以下の様にコントローラを編集しました。

【変更前】
@anotherEntries = Entry.where(room_id: myRoomIds).where('user_id != ?', @user.id)

【変更後】
@anotherEntries = Entry.includes(:user, :room).where(room_id: myRoomIds).where('user_id != ?', @user.id)

しかし、Entry => [:user]に関する警告は表示されなくなったものの、以下のEntry => [:room]に関する警告は依然として出続けます。
userとroomの順番を変えるなどしても結果は同じでした。どこに問題があるのかが分かりません。

user: hoge
GET /rooms/index
USE eager loading detected
  Entry => [:room]
  Add to your query: .includes([:room])
Call stack
  C:/hoge/app/controllers/rooms_controller.rb:26:in `block in index'
  C:/hoge/app/controllers/rooms_controller.rb:25:in `index'

試したこと

同じような問題が生じている方もいらっしゃる様ですが、具体的な解決方法は見付きれませんでした。

補足情報（FW/ツールのバージョンなど）

ruby 2.6.4p104
RubyGems 3.0.3
Rails 5.2.3

Answer 1

どこに問題があるのかが分かりません。

メッセージにblock in indexとあるように、問題のある箇所はこちらです。

    @currentEntries.each do | entry |
      myRoomIds << entry.room.id
    end

通常のリレーションであれば、Entryにroom_idがあるかと思いますので、entry.room.idではなくentry.room_idを見るのが適切かと思います。

Comments

[punchan36]

有難うございます！
ご指摘のありました箇所を「myRoomIds << entry.room_id」とする事で、警告のメッセージは表示されなくなりました。上手くいったようです。