【Rails5】リレーション先を含めたor検索について

Question

やりたいこと

中間テーブルを含めた検索をかけたいです。
なんとか実装までは漕ぎ着けたんですが、他に良い方法（または間違っているなど）など、ご指摘いただると幸いです。

試していること

def index
# events_controller.rb
  scope = Event.includes(:users, :creator, :updater)
  scope = scope.select { |e| (e.publish? or e.joins?(current_user)) }
  scope = Event.revive_active_record(scope)

  # ここから検索条件
  @q = scope.ransack(params[:q])
  ・
  ・
  ・
end

# event.rb
has_many :attendee_events, dependent: :destroy
has_many :users, through: :attendee_events

def joins?(user)
  users.exists?(user.id)
end

def self.revive_active_record(arr)
  arr.first.class.where(id: arr.map(&:id))
end

Comments

[gentaro]

間違っているのかどうかは自分で動かせば（テストすれば）わかるはずです。
他に方法があるのか、という漠然とした質問は基本的にナシだと思います。（あなたが何を持って「良い」とするのか、誰もわからない）
とりあえず↓読んで質問のやり方を変えましょう。
https://teratail.com/help/question-tips
https://teratail.com/help/avoid-asking

Answer 1

100件あると e.joins?(current_user) が 100回動いて100回SQLが発行されませんか？
N+1と呼ばれる問題で、データが増えるたびにどんどん遅くなるので危険です
またページングまで考慮すると最初の20件だけ取る場合でも全件チェックしているのも効率が悪いです

# 逆向きにUserから参照できるようにしておきます
class User
  has_many :attendee_events
  has_many :events, through: :attendee_events
end

def index
  # 同じモデルのリレーションなら where.or が使えます
  scope = Event.publish.where.or(current_user.events)

  # or の部分には includes はいらないのでこちらにつければ十分だと思われます
  scope = Event.includes(:users, :creator, :updater)
               .revive_active_record(scope)
end

# 1. 条件を追加する場合はクラス関数ではなく scope をオススメ、 whereで繋がります
# 2. 引数を配列ではなくRelationにできたので select(:id) オススメ、全件抽出せずサブクエリで指定できます
# 3. ここまで短くなればそもそも分離しなくて良いかもですね
scope :revive_active_record, ->(relation) do
  where(id: relation.select(:id))
end