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

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

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

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

Q&A

解決済

1回答

3347閲覧

N+1問題を解決したい

satail

総合スコア31

Ruby on Rails

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

0グッド

0クリップ

投稿2017/06/01 13:35

###前提・実現したいこと
bulletを使ってN+1問題を直しています。
直していく中で、ActivityページのN+1問題でどのように直したらいいかわからなくって困っています。どのようにしたらいいでしょうか

###発生している問題

GET /activities USE eager loading detected Comment => [:message] Add to your finder: :includes => [:message]

###該当のソースコード

activities_controller

def index @activities = PublicActivity::Activity.includes(:owner, :trackable).order("created_at DESC").where(owner_id: current_user.following_ids, owner_type: "User").page(params[:page]).per(30) end

_create.html(comment)

added comment <% if activity.trackable %> to <%= link_to activity.trackable.message.title, activity.trackable.message %> <% else %> which has since been removed <% end %>

activities/index.html

<% @activities.each do |activity| %> <div class="activity"> <%= activity.created_at %> <%= link_to activity.owner.username, activity.owner if activity.owner %> <%= render_activity activity %> </div> <% end %>

comment.rb

class Comment < ApplicationRecord include PublicActivity::Common belongs_to :message belongs_to :user end

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

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

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

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

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

guest

回答1

0

ベストアンサー

ruby

1def index 2 @activities = PublicActivity::Activity.includes(:owner, {:trackable=>:message}).references(:owner, {:trackable=>:message}).order("created_at DESC").where(owner_id: current_user.following_ids, owner_type: "User").page(params[:page]).per(30) 3 end

でいかがでしょう
ポイントは、incloudesのネストと
referencesです。

投稿2017/06/02 03:08

moke

総合スコア2241

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

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

satail

2017/06/02 08:46

ActiveRecord::EagerLoadPolymorphicError Cannot eagerly load the polymorphic association :owner <% @activities.each do |activity| %> とエラーが出てしまいます。
moke

2017/06/02 10:08

polymorphicを使っているんですね。素晴らしい。 でも、そういう時は、PublicActivity::Activity modelとOwner modelを晒していただかないと とりあえず想像ですがPublicActivity::Activity のhas_one :owner〜を下記のように書き換えて見てください。 has_one :self_ref, :class_name => self, :foreign_key => :id has_one :owner ,:through => :self_ref, :source => :relation, source_type: 'Owner'
satail

2017/06/03 18:39

丁寧にありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問