[Ruby on Rails]日付範囲による複数条件式をシンプルに書きたい

Question

現在、あるコードの書き方で悩んでおります。

一例なのですが、

• start_at
• end_at

のDateTime型の２つのカラムを持ったモデルから、「sample」というインスタンスを生成したとします。

ここで、以下のような値チェックを行いたいと思っています。

if sample.start_at.nil? && sample.end_at.nil?
  return
elsif sample.start_at.present? && sample.end_at.nil?
  return if Time.now.to_i > sample.start_at.to_i
elsif sample.start_at.nil? && sample.end_at.present?
  return if Time.now.to_i < sample.end_at.to_i
elsif Time.now.to_i.between?(sample.start_at.to_i, sample.end_at.to_i)
  return
else
  raise
end

やりたいことは以下となります。

• start_atとend_atが共にnilだった場合はreturn
• start_atだけが存在するとき、その値が現在の日付以前だったらreturn
• end_atだけが存在する時、その値が現在の日付以降だったらreturn
• 両方存在するとき、それらの値の間に現在の日付が入っていたらreturn
• 上記のどれにも含まれない場合、例外発生

ただ、上記のコードだと非常に冗長というか、もう少しシンプルにかける気がしておりまして・・・。
どなたかご教授いただけないでしょうか。

Answer 1

DateTime型とお書きですが、コードを見るとTime型の間違いのようなので、Time型としておきます。
また、比較する前にto_iする意味がわからないので、とりあえずto_iは省いて書いています。
必要なら追加してください。

MIN = Time.new(-9999)
MAX = Time.new(9999)

if (sample.start_at||MIN) < Time.now &&
   Time.now < (sample.end_at||MAX)
  return
else
  raise
end

注：上記コードには、西暦10000年問題があります。

Answer 2

それでいいような気がしますけど...
行数を減らすのであればこんな感じに書けなくもないですけど、可読性が悪いのでオススメはしません。

return if sample.start_at.nil? && sample.end_at.nil?
return raise unless (sample.start_at.present? && sample.end_at.nil?) && (Time.now.to_i > sample.start_at.to_i) \
  || (sample.start_at.nil? && sample.end_at.present?) && (Time.now.to_i < sample.end_at.to_i) \
  || Time.now.to_i.between?(sample.start_at.to_i, sample.end_at.to_i)

どうしてもというなら、条件式をメソッドにまとめれば多少見やすくなるかと

def both_times_nil?
  sample.start_at.nil? && sample.end_at.nil?
end

def start_at_faster_than_now?
  (sample.start_at.present? && sample.end_at.nil?) && (Time.now.to_i > sample.start_at.to_i)
end

def end_at_later_than_now?
  (sample.start_at.nil? && sample.end_at.present?) && (Time.now.to_i < sample.end_at.to_i)
end

def within_range?
  Time.now.to_i.between?(sample.start_at.to_i, sample.end_at.to_i)
end