関連モデルを含めたwhere検索について

Question

現在Railsを勉強中のものです。

Rate(評価)モデルから、userが投稿した特定のhostへの評価一覧を検索して取得したいと思っています。

class Rate < ApplicationRecord
  belongs_to :agreement
  has_one :user, through: :agreement
  has_one :host, through: :agreement

class Agreement < ApplicationRecord
  belongs_to :user
  belongs_to :host
  has_one :rates
end

class Host < ApplicationRecord
  has_many :agreements, dependent: :destroy
  has_many :rates, through: :agreements
end

class User < ApplicationRecord
  has_many :agreements, dependent: :destroy
  has_many :rates, through: :agreements
end

恥ずかしながら私が調べながら書いてみたのは以下のような感じです

  def show
    host = Host.find_by(myid: params[:id])
    rates = Rate.left_joins(:agreement).includes(agreement: :user).where(:agreement => {host_id: host.id})

    render rates.as_json(
      only: %i[id star comment created_at],
      include: {
        agreement: {
          only: [],
          include: {
            user: {
              only: %i[name age]
            }
          }
        }
      }
    )
  end

left_joinsで関連モデルを含めた検索を可能にして、
includesで必要な情報(関連しているagreement.userのname, age等)を取得するためのN + 1対策
whereで関連しているagreementのhost_idがパラメータから事前に検索しておいたhostのidに一致しているものを検索
というイメージでかきました。

しかしこれでは

Mysql2::Error: Unknown column 'agreement.host_id' in 'where clause'

うまく関連モデルを紐づけられていないのでしょうか。

先輩エンジニアの皆様ご教授よろしくお願い致します。

Answer 1

model を直したほうがよさげです。
class Agreement < ApplicationRecord の has_one :rates をやめ、rate は class Agreement の属性にする。

そうしますと
rates = host.agreements
です。
Agreement にrateとつなげる以上の機能がないなら、なくして

class Rate < ApplicationRecord
  belongs_to :user
  belongs_to :host
end

で rates = host.rates

Comments

[IRIESS]

まずモデル設計がよくないのですね。相談せず自分の感でつくってしまいました。今から変更すると骨が折れそうです
agreementには他にもstatusなど機能があります！
回答ありがとうございました！

[winterboum]

ということですと、もし修正するなら 前者 ですね

Answer 2

個人的には他テーブルで絞り込みをする時は、mergeを使うのが好きです。他テーブルを参照しているのが、一目で分かるので。(勿論whereでも出来ますが）

Rate.left_joins(:agreement).includes(agreement: :user).merge(Agreement.where(host_id: host.id))

Comments

[IRIESS]

ありがとうございました！mergeを使ってみます