これはN+1問題ですか？

Question

あけましておめでとうございます！

この前、初めてN+1問題というのを知り、何も考えずに打ったコードを改善しようとしている初心者です。よろしくお願いいたします！

has_manyの子モデルから特定のカラムを取得する場合にSQLが発生することは、致し方がないことなのですか？

前提

rails 5.1.3
ruby 2.4.1
gem active_admin

正規化前

User	男	女	Gap	NIKE	Supreme
User1	○		1	7
User2	○	4	4

正規化後

### Model User
belongs_to :user_sex => 性別が入るSEXモデルも別途あり
has_many :user_brands => ブランドが入るBrandモデルも別途あり

一覧

ActiveAdmin.register User do
  column :user_name
  Sex.order('id').each do |sex|
    column sex.sex_name do |user|
      "○" if user.user_sex.find_by(sex_id: sex.id)
    end
  end
  Brand.order('id').each do |brand|
    column brand.brand_name do |user|
      user.user_brands.find_by(brand_id: brand.id).number if user.user_brands.find_by(brand_id: brand.id)
    end
  end
end

def scoped_collection
      User.includes(:user_sex, :user_brands)
    end

問題点　has_manyの子モデルから特定カラムを取得する方法にはSQLが発生する？

性別判定の○は、if user.user_sex.pluck(:sex_id).include?(sex.id)という記述で、複数のSQLを減らすことが出来ました。
が、UserBrandモデルから個数を取り出す際には、どうしてもSQLが発生してしまいます。
調べた所、where find find_byを使った場合にはSQLが発生してしまうようなので、複数関係のモデルから特定カラムの値を取得する場合にSQLが発生することは致し方がないことなのですか？

よろしくお願いいたします！

Answer 1

そもそも論ですが、通常入れるべきデータを考えれば、性別は1つしか取らないので、正規化するにしても中間テーブルは過剰です。user.sex_idを用意して、use belongs_to :sexで十分です（もちろん、両性具有、性同一性障害など、男女より詳細に性別を管理する必要のある状況であればまた別ですが）。

そして、.find_byは呼ぶたびにデータベースを検索します。同じリレーションについて特定の条件のものを探したい、という状況であれば、includesした上でEnumerableにある方のメソッドを使えば、取ってきたデータからの検索となるので、繰り返しデータベースを呼ぶことはなくなります。

  Brand.order('id').each do |brand|
    column brand.brand_name do |user|
      user.user_brands.find { |item| item.brand_id == brand.id }
          .try(:number)
    end
  end

Comments

[PartyKids]

あけましておめでとうございます！
昨年は、大変お世話になりありがとうございました！
本年もよろしくお願い申し上げます。

＞中間テーブルは過剰です
中間テーブルから値を取る場合と取らない場合の場合分けを試してみたかったので、user_sexという変な中間テーブルを作成したのですが、やりすぎでした。。。

＞Enumerableにある方
丁寧に説明していただきありがとうございます。これが無かったら、ActiveRecord::BaseのFind？なのにSQLが発生しないのは何で？と勘違いしてました。
tryメソッドでnilになってもエラーが発生しないので、真偽を省略出来るんですね。

ありがとうございました！