Как правильней сделать рефакторинг?

Question

[WonderMetal]

class Reservation < ActiveRecord::Base

  validates :table, presence: true
  validates :start, presence: true
  validates :finish, presence: true


  validate :checking_the_time, :uniqueness_reservation

  scope :tables, ->(table) { where("'reservations'.'table' = ? ", table) }

 
  def checking_the_time
    if start >= finish
      errors.add(:finish, "Invalid time range")
    end
  end

  def uniqueness_reservation
    unless Reservation.diapazone(start, finish).tables(table).where.not(id: id).empty?
      errors.add(:booked_from, 'Invalid period.')
    end
  end


  private

  def self.diapazone(start, finish)
    where("(start >= :s AND finish <= :f) OR (start <= :s AND finish >= :s)
             OR (start <= :f AND finish >= :f)",
          {s: start, f: finish})
  end


end

помогите с рефакторингом валидации uniqueness_reservation, валидация на то что б новый заказ никак не пересекался с существующим

Answer 1

[Viktor Vsk]

Вообще, программирование уже давно не в машинных кодах производится. Немного бизнес-логики не помешало б. Что такое tables? Вы сдаете квартиру и проверяете, что б в одном временном интервале был только один заказ?
Тогда все проще

Comments

[WonderMetal]

да, именно так.

[Viktor Vsk]

Тогда, думаю, как-то так:

def single_reservation_for_period
  if Reservation.where("start BETWEEN #{start} AND #{finish} OR finish BETWEEN #{start} AND #{finish}").present?
     errors.add(:reserved, 'Invalid period of time. Already reserved')
  end
end

То есть, например, я хочу забронировать на период с 10 по 15 августа и хоть какой-то один из этих дней (10, 11, 12, 13, 14, 15) занят - у меня не получится

[Денис Кривощёков]

Виктор Ablebeam Можете объяснить, как сюда добавить бизнес-логику?