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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

Active Record

Active Recordは、一つのオブジェクトに対しドメインのロジックとストレージの抽象性を結合するデザインパターンです。

Q&A

解決済

1回答

3522閲覧

複数テーブルのincludesとreferencesを設定しN+1問題を解決したい

Cziffra

総合スコア10

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Ruby on Rails 4

Ruby on Rails4はRubyによって書かれたオープンソースのウェブフレームワークです。 Ruby on Railsは「設定より規約」の原則に従っており、効率的に作業を行うために再開発を行う必要をなくしてくれます。

Active Record

Active Recordは、一つのオブジェクトに対しドメインのロジックとストレージの抽象性を結合するデザインパターンです。

0グッド

0クリップ

投稿2016/05/27 11:40

編集2016/05/27 12:50

###前提・実現したいこと
Rails 4.2.5です。
Viewを表示する際にN+1問題が発生しているので原因と対策を教えていただきたく思います。

親イベントと子イベントをjoinした一覧の表示を表示したいのですが、設定は下記の通りとなっております。

Model関連

関連テーブルは下記の三つ。

  • Event(親イベント)

親イベント情報が格納されており複数の子イベントと複数のカテゴリを持っている。

  • ChildEvent(子イベント)

親イベントの日程が複数ある場合や日に応じて会場が変わるなどする場合、日程情報や会場情報は子イベントに格納され開催回数分作成される。
親イベント1に対し1レコードは子イベントが作成される。

  • Category(カテゴリ)

親イベントに所持され、親イベントは複数のカテゴリを持てる。
子イベントは親が所持するカテゴリを参照する。

それぞれは
Event has_many ChildEvent
Event has_many Category
ChildEvent has_many Event
Category has_many Event
で設定しておりそれぞれ中間テーブルが存在している。

※またChildEventはEvent経由でCategoryを持っているので
ChildEvent has_many Category through: :events
も設定している。

N+1が起こらないように子イベントの一覧を表示するChildEventのscopeとして下記の設定をしている。

ruby

1class ChildEvent < ActiveRecord::Base 2 def self.with_parent_events 3 ChildEvent.includes(:events).references(:events).order(child_event.start_datetime) 4 end 5 def self.with_categories 6 ChildEvent.includes(:categories).references(:categories) 7 end 8end

Controller関連

上記の状態でControllerは下記のようにDBから取得している。

ruby

1@categories = Category.all 2@events = EventChild.with_parent_events.with_categories

View関連

Slimまずはすべてのイベントを表示し、つぎはカテゴリ毎にdivで囲んで出力するよう下記のようにしている。

slim

1# すべてのイベントを表示 2div 3 - @events.each do |event| 4 | #{event.title} 5# イベント毎に表示 6- @categories.each |category| 7 div 8 - @events.each do |event| 9 - if event.categories.ids.include?(category.id) 10 | #{event.title}

###結果
上記のif文でN+1が発生しているようでしたが、Bulletでも検出できず悩んでおりました。
また

ruby

1ChildEvent.includes(events: :categories).references(:events,:categories)

などのscopeでも試しましたが同じ結果でした。
なにかアドバイスをいただければ幸いです。

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

自己解決

根本的な解決ではないのですが、viewを変更することでいけました。

ruby

1# すべてのイベントを表示 2div 3 - @events.each do |event| 4 | #{event.title} 5# イベント毎に表示 6- @categories.each |category| 7 div 8 - @events.each do |event| 9 - event.categories.each do |c| 10 - if c.id == category.id 11 | #{event.title}

解決済みとさせていただきますが、より良い書き方がございましたら、ぜひコメントを残してくださると嬉しいです。

投稿2016/05/27 14:30

編集2016/05/27 14:31
Cziffra

総合スコア10

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問