テーブル内に該当するレコードが存在するかどうかを確認したい

Question

中間テーブルを介したhas_manyな関係にあるテーブルに、該当するレコードが存在するかを確認したい

作ったコードがスマートではないため、もっと良いコードがあれば教えてください。

前提条件

class Account < ApplicationRecord
  has_many :group_accounts
end

class GroupAccount < ApplicationRecord
  belongs_to :group
  belongs_to :account
end

class Group < ApplicationRecord
has_many :group_accounts
end

class CreateGroups < ActiveRecord::Migration[5.1]
  def change
    create_table :groups do |t|
      t.string :team_name
      t.timestamps
    end
  end
end

class CreateGroupAccounts < ActiveRecord::Migration[5.1]
  def change
    create_table :group_accounts do |t|
      t.integer :group_id
      t.integer :account_id
      t.string :join_switch
      t.timestamps
    end
  end
end

class CreateAccounts < ActiveRecord::Migration[5.1]
  def change
    create_table :users do |t|
      t.string :name
      t.timestamps
    end
  end
end

作ったコード

<% @switch_true=[] %>

<% Groups.each do |group| %>
　<% @pickup_group_accounts = group.group_accounts.where('account_id = ?', current_account.id).where('join_switch = ?', "true") %>
　<% if @pickup_group_accounts.present? %>
　　<% @switch_true.push @pickup_group_accounts %>  
　<% end %>
<% end %>

<% if @switch_true.present? %>
　参加者あり
<% else %>
　参加者なし
<% end %>

ご説明と解決方法の検討（自己検討）

Groupテーブルからレコードを順番に取り出し、Groupに紐づいた中間テーブル（group_accounts）で検索条件を指定し、さらにAccountsテーブルで検索条件を指定した上で、該当レコードを抽出しています。

テーブル同士がhas_manyな関係にあるため、eachメソッドのループ内で

<% Groups.each do |group| %>
　<% @pickup_group_accounts = group.group_accounts.where('account_id = ?', current_account.id).where('join_switch = ?', "true") %>
　<% if @pickup_group_accounts.present? %>
　　参加者あり
　<% else %>
　　参加者なし
　<% end %>
<% end %>

というやり方をすると、重複した結果（参加者あり）が複数取り出されてしまいましたので、上記の方法で判別していますが、力技っぽくていまいちな印象を持っています。

なんとなくjoinメソッドを使えば上手く取り出せそうな感じがしましたが、どうすれば良いか思いつかず、詰まってしまいました。
どなたかご教授いただけませんでしょうか？

Answer 1

書き方もそうですがSQL発行回数も意識すると良いでしょう
グループごとにチェックを行うと Group.each の1回と、グループ数のN回のSQLが発行されます
N+1と言われる問題で、グループが増えたびに遅くなる問題を抱えています

■条件付きの has_many を定義がお勧め

表示時に条件を書かずに比較的スッキリするしSQL発行回数もpreloadを使えるためGroupが増えても2回です

class Group < ApplicationRecord
  has_many :group_accounts
  has_many :joined_group_accounts, -> { where(join_switch: "true") }, class_name: 'GroupAccount'
end

<% Group.preload(:joined_group_accounts).each do |group| %>
　<% if group.joined_group_accounts.map(&:account_id).include?(current_account.id) %>
　　参加者あり
　<% else %>
　　参加者なし
　<% end %>
<% end %>

■ has_many through もお勧め

中間テーブルを意識しなくて良くなります

class Group < ApplicationRecord
  has_many :group_accounts
  has_many :joined_group_accounts, -> { where(join_switch: "true") }, class_name: 'GroupAccount'
  has_many :joined_accounts, through: :joined_group_accounts, source: :account
end

<% Group.preload(:joined_accounts).each do |group| %>
　<% if group.joined_accounts.include?(current_account) %>
　　参加者あり
　<% else %>
　　参加者なし
　<% end %>
<% end %>

Comments

[zin2]

ありがとうございました！
全く知らなかった方法でしたので、大変勉強になりました。

N＋１問題については、ずっと引っかかっていましたが、解決できました！

Answer 2

<% if Groups.joins(:group_accounts).where(group_accounts: {account_id: current_account.id}).where(group_accounts: {join_switch: "true"}).present? %>
　　参加者あり
　<% else %>
　　参加者なし
　<% end %>
<% end %>

でいけないかと思いましたが、上手く取り出せませんでした。