今回はincludesとかを使ってN+1問題を解決していきたいです。
さっさと解決しようかと思ったのですが、一筋縄じゃいかなかったので、今から頑張って解決しようと思います！

またこの方の記事を全面的に参考にして(パクって)記事を書いていこうと思います。

あとタイトルがクソすぎるので、いいタイトルあったら編集リクエスト待ってます。

今回のモデルのアソシエーション

drinkがいわゆる投稿です。

userはいろんな投稿をするので、

userとdrinkで１対多の関係

これだけだったら単純なのですが、

それらに加えて、imageがあります。

投稿画像とユーザー画像がそれぞれある感じ。

bulletが検知したN+1

user: root USE eager loading detected Drink => [:image_attachment] Add to your query: .includes([:image_attachment])

DrinkモデルがN+1問題を引き起こしてるっぽい。

解決していこう

drinks_controller


    @pagy,@drinks = pagy(Drink.includes(image_attachment: :blob)
                    .where.not(user_id: 6)
                    .where("user_id IN (#{following_ids}) OR user_id = :user_id", user_id: @user.id)
                    .order('drinks.created_at DESC'))

んーややこしい。。。。

色々調べてみるとこーゆー感じで書き直したらいいんじゃないかと思った。

drinks_controller


    @pagy,@drinks = pagy(Drink.includes(user: {image_attachment: :blob})
                    .where.not(user_id: 6)
                    .where("user_id IN (#{following_ids}) OR user_id = :user_id", user_id: @user.id)
                    .order('drinks.created_at DESC'))

したら今度は、

user: root USE eager loading detected Drink => [:image_attachment] Add to your query: .includes([:image_attachment])
とbulletに検知された。

userの方は解決できたっぽいけど、、、。
今度はimageの方でN+1が発生してる。。。。

この書き方でどうだ！！！(無策)


    @pagy,@drinks = pagy(Drink.includes(:user ,{image_attachment: :blob})
                    .where.not(user_id: 6)
                    .where("user_id IN (#{following_ids}) OR user_id = :user_id", user_id: @user.id)
                    .order('drinks.created_at DESC'))

解決できた。

ログを実際に見てみる


Processing by DrinksController#index as HTML
web_1              |   User Load (0.6ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 7 LIMIT 1
web_1              |   ↳ app/helpers/sessions_helper.rb:52:in `current_user'
web_1              |    (0.8ms)  SELECT COUNT(*) FROM `drinks` WHERE `drinks`.`user_id` != 6 AND (user_id IN (SELECT followed_id FROM relationships WHERE follower_id = 7) OR user_id = 7)
web_1              |   ↳ app/controllers/drinks_controller.rb:14:in `index'
web_1              |   Rendering layout layouts/application.html.erb
web_1              |   Rendering drinks/index.html.erb within layouts/application
web_1              |   Drink Load (1.1ms)  SELECT `drinks`.* FROM `drinks` WHERE `drinks`.`user_id` != 6 AND (user_id IN (SELECT followed_id FROM relationships WHERE follower_id = 7) OR user_id = 7) ORDER BY drinks.created_at DESC LIMIT 2 OFFSET 0
web_1              |   ↳ app/views/drinks/_drinks_index.html.erb:12
web_1              |   User Load (0.6ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 7
web_1              |   ↳ app/views/drinks/_drinks_index.html.erb:12
web_1              |   ActiveStorage::Attachment Load (0.7ms)  SELECT `active_storage_attachments`.* FROM `active_storage_attachments` WHERE `active_storage_attachments`.`record_type` = 'Drink' AND `active_storage_attachments`.`name` = 'image' AND `active_storage_attachments`.`record_id` IN (8, 7)
web_1              |   ↳ app/views/drinks/_drinks_index.html.erb:12
web_1              |   ActiveStorage::Blob Load (0.8ms)  SELECT `active_storage_blobs`.* FROM `active_storage_blobs` WHERE `active_storage_blobs`.`id` IN (12, 13)
web_1              |   ↳ app/views/drinks/_drinks_index.html.erb:12
web_1              |   ActiveStorage::Attachment Load (0.6ms)  SELECT `active_storage_attachments`.* FROM `active_storage_attachments` WHERE `active_storage_attachments`.`record_id` = 7 AND `active_storage_attachments`.`record_type` = 'User' AND `active_storage_attachments`.`name` = 'image' LIMIT 1
web_1              |   ↳ app/views/drinks/_drinks_index.html.erb:18
web_1              |   Like Load (0.7ms)  SELECT `likes`.* FROM `likes` WHERE `likes`.`user_id` = 7 AND `likes`.`drink_id` = 8 LIMIT 1
web_1              |   ↳ app/views/likes/_like.html.erb:2
web_1              |    (0.6ms)  SELECT COUNT(*) FROM `likes` WHERE `likes`.`drink_id` = 8
web_1              |   ↳ app/views/likes/_like.html.erb:6
web_1              |   Rendered likes/_like.html.erb (Duration: 9.1ms | Allocations: 1639)
web_1              |   Like Load (0.7ms)  SELECT `likes`.* FROM `likes` WHERE `likes`.`user_id` = 7 AND `likes`.`drink_id` = 7 LIMIT 1
web_1              |   ↳ app/views/likes/_like.html.erb:2
web_1              |    (0.7ms)  SELECT COUNT(*) FROM `likes` WHERE `likes`.`drink_id` = 7
web_1              |   ↳ app/views/likes/_like.html.erb:6
web_1              |   Rendered likes/_like.html.erb (Duration: 10.2ms | Allocations: 1558)
web_1              |   Rendered drinks/_drinks_index.html.erb (Duration: 56.6ms | Allocations: 12908)
web_1              |   Rendered drinks/index.html.erb within layouts/application (Duration: 57.9ms | Allocations: 12979)
web_1              | [Webpacker] Everything's up-to-date. Nothing to do

#　なぜこの書き方で上手くいったのかの僕の予想

@drinks = pagy(Drink.includes(:user ,{image_attachment: :blob})

って書き方は、
drinkとアソシエーション組んでるuserとimageをどっちもincludesメソッドの引数に指定したから上手くいったのでしょう。

一方失敗した下のような書き方は

@drinks = pagy(Drink.includes(user: {image_attachment: :blob})

userモデルとアソシエーションを組んでるimegeをN+1問題を起こさないようにしてーー。

って意味のコードになってしまったのでしょう。。。。きっと。。。