Rails での指定した日付期間の重複チェックについて

Question

###指定した日時の期間重複がないかチェックしたい
Ruby on Rails4にて、期間を指定して予約を登録するシステムを作成しております。
１つの予約につき、複数の予約期間を設定出来るようにしており、
登録をする際にこれらの期間に重複がないかのチェックを実装中です。

【例】
reservation ID=1
reservation_details [
reservation_detail ID.1 2016/6/1～2016/6/5
reservation_detail ID.2 2016/6/10～2016/6/20
reservation_detail ID.3 2016/6/18～2016/6/20 ]

→reservation_detail ID.2とID.3の期間が重複しているため、登録時にバリデーションエラーを出したい。

###該当のソースコード

models/reservation.rb

class Reservation < ActiveRecord::Base
  has_many :reservation_details

  accepts_nested_attributes_for :reservation_details, allow_destroy: true

models/reservation_detail.rb

class ReservationDetail < ActiveRecord::Base
  has_many :reservation_details

  belongs_to :schedule

controller/reservations_controller.rb

class ReservationsController < ApplicationController

  private
    def reservation_params
      params.require(:reservation).permit(:name, :description, 
                     reservation_details_attributes: [:id, :start_date_on,
                     :end_date_on, :start_time_at, :end_time_at, :_destroy ])
    end

###現状
EachValidatorを使用したカスタムバリデーションを作成し、ネストしているパラメータを渡す事で実現できないかと考えておりました。
EachValidatorに対して、配列の１項目だけを渡して重複チェックは行えましたが、
配列その物を渡し、Validator内で期間のチェックを行う事は実現出来ておりません。

そこで、この様な仕様の場合のValidationの手法が存在するのか、そもそもEachValidatorを使用する方向で間違っていないのかをお教え頂けると幸いです。
EachValidatorに拘らず、その他の手法でもあると助かります。

よろしくお願い致します。

Answer 1

複数項目同士の関連を見る場合、EachValidatorではなくActiveModel::Validatorのほうが向いています。こちらにはdef validate(record)のようなメソッドを定義してバリデーションを行い、モデルからはvalidates_withで参照します。

Comments

[DaisukeJP]

maisumakun様

お返事ありがとうございます。
詳しく調べてみましたが、確かにおっしゃる通りValidatorの方が私の要件を満たせそうです。
動作確認を行いながら、少しずつ使い方が見えてきました。
ActiveModel::Validatorにて実装出来るかチャレンジしてみます。

ありがとうございます。

Answer 2

次のテーブルを導入しては如何でしょう。
*テーブル名 reserved_days (予約で埋まっている日を保持する)
列： day (属性は not nil, uniq: true とする)

予約の追加、削除の度に この reserved_days も適宜 行を追加、削除していきます。
(transaction にして、両方のテーブルが整合性を保つようにして処理することが必要です)

ある期間 [from .. to] が、予約可能か否かは、
SELECT count
FROM reserved_days
WHERE day BETWEEN form AND to
で判定が可能になります。

あるいは、つぎの方法でもよいはずです。

• 期間重複を判定する https://yone64.wordpress.com/2013/07/03/%E6%9C%9F%E9%96%93%E9%87%8D%E8%A4%87%E3%82%92%E5%88%A4%E5%AE%9A%E3%81%99%E3%82%8B/

...
データAとデータBがあって、それぞれFromとToを持つとき、期間が被るものを検索するためには、
WHERE A.From <= B.To AND A.TO >= B.From
....

Comments

[DaisukeJP]

katoy様
返信ありがとうございます。
確かに、DBに登録されているデータと比べて重複判断を行うのであれば、
頂いた内容にて実現できそうです。
しかし、今回は登録するデータに複数の予約期間レコードを保持しており、
それら同士を登録前に重複チェックしなければなりません。
has_manyでネストされたデータ同士の重複チェックです。

その為、データベース側に保持されているデータと、新たに追加するデータとのチェックになると、重複データが出来る可能性が高くなってしまいます。

今後、DB側との重複チェックも必要になる可能性がありますので、その際は参考にさせて頂きます。

ありがとうございます。