rails includesとwhereを使用した検索について 1対多対1

Question

現在rails apiを使用してアプリを作成しています。

userが所持するfree_time達に、hostがhost_requestを送れます。
しかし、host_request作成時に同userが同hostに同期間(start_timeとfinish_timeの差)をすでに作成していた場合は失敗するようvalidateを書いたのですがエラーが発生して実現できていません。

User	UserRequest	RecruitmentTime	Host
name :strint	user_id :int	host_id :int	name :string
	recruitment_time_id :int	start_time :datetime
	start_time :datetime	finish_time :datetime
	finish_time :datetime

FreeTime	HostRequest
user_id :int	host_id :int
start_time :datetime	free_time_id :int
finish_time :datetime	start_time :datetime
	finish_time :datetime

それぞれのリレーションは以下になっています。

# user.rb
  has_many :free_times
  has_many :user_requests
  has_many :host_requests, through: :free_times
  has_many :recruitment_times, through: :user_requests

#host.rb
  has_many :recruitment_times
  has_many :host_requests
  has_many :user_requests, through: :recruitment_times
  has_many :free_times, through: :host_requests

#free_time.rb
  belongs_to :user
  has_many :host_requests

#recruitment_time.rb
  belongs_to :host
  has_many :user_requests

#user_request.rb
  belongs_to :user
  belongs_to :recruitment_time
  has_one :host, through: :recruitment_time

#host_request.rb
  belongs_to :free_time
  belongs_to :host
  has_one :user, through: :free_time

以下はvalidateのメソッドです

#host_request.rb
validate :duplication_of_user_request

def duplication_of_user_request
    free_time = FreeTime.find(free_time_id)
    user = free_time.user
    if UserRequest.includes(:host).where('user_id = ? && host.id = ? && user_request.finish_time >= ? && ? >= user_request.start_time ', user.id, host_id, start_time, finish_time).exists?
      errors.add(:start_time, "同じ時間帯でお相手から申請が来ています。")
    end
  end

user_requestに has_one :host, through: :recruitment_timeを記入しているため、
直感的にuser_requestとhostは1対1のような関係になり、上記の書き方で実現できるかと考えていたのですが、、、
書き方に問題はありますでしょうか？

#host_requests_controller.rb
 def create
    host_request = HostRequest.new(host_request_params)
    if host_request.save
      render json: host_request, status: 201
    else
      render json: host_request.errors, status: 400
    end
  end

  private
  def host_request_params
    params.permit(:free_time_id).merge(host_id: current_api_host.id, start_time:Time.zone.parse(params[:start_time]), finish_time: Time.zone.parse(params[:finish_time]))
  end

#エラー
 ActiveRecord::StatementInvalid:
       Mysql2::Error: Unknown column 'host.id' in 'where clause

Answer 1

二つ問題が見当たりました。
まず、whereの中で、SQLを直書きしている以上、それはSQL文法にそっていなければなりません。モデルHostのテーブル名は、きっとhostsですよね? 実際のSQL文中では、このような場合は、単にテーブルをJOINしています。だから、where()文中に現れるべきはテーブル名であるhostsであって、hostではありません。同様に、user_requestも複数形でなければなりません。

このようなSQL直書き文をデバッグする時のtipとして、to_sql文をログなどに出力してみて、それがSQL文として正当かどうか、Rails consoleあるいは MySQLを直接使って、検討してみると良いでしょう。

もう一点、Rails APIマニュアルのincludesの項目によれば、

If you want to add string conditions to your included models, you'll have to explicitly reference them.

とされています。端的には、このような場合は、references を使う必要があります。引数はテーブル名です。この場合であれば、def duplication_of_user_request において、

if UserRequest.includes(:host).where('user_id = ? && host.id #中略
  ', user.id, host_id, start_time, finish_time).
  .references(:hosts, :user_requests).exists?

などとすることは、最低限必要でしょう。

以上、元投稿のコードに誤りの部分が多いこともあり、これだけで動くかどうかはわかりません。たとえば、includes文に :user_requests も必要かもしれません(必要ないかも?)。
しかし、上記の二つのポイントを念頭に試行錯誤すると解決に近づくのでは、と思います。

Comments

[IRIESS]

回答ありがとうございました！SQL文法等々知識がなさすぎました。

最終的には
UserRequest.includes(recruitment_time: :host).where('user_id = ? && hosts.id = ? && user_requests.finish_time >= ? && ? >= user_requests.start_time ', user.id, host_id, start_time, finish_time).references(:hosts, :recruitment_times).exists?

で実現できました。