ホテル予約システムのオーバーラップメソッドRuby

Question

私はホテル予約システムのコードをリファクタリングしています。具体的には、学校プロジェクトのためにホテル内のすべての空室を一覧表示する方法です。私たちのインストラクターは彼のソリューションを提供しました。残念なことに、彼と残りの学校は月曜日（私たちのリファクタリングが予定されています）まで休憩しています。 StackOverflow、私はあなたの助けを利用することができます。

私の質問は、なぜ次のコードでbang（！）が使われたのかということになります。

def overlap?(other) 
    return !(other.checkout <= @checkin || other.checkin >= @checkout) 
end 

このコードは、Module HotelとクラスDateRange内のメソッドです。特定の日付範囲の空席を確認するために、別のクラスで呼び出されます。以下を参照してください。

overlap_reservations = @reservations.select do |res| 
    res.overlaps(dates) 
end 

私の考えでは、私はバングを利用しませんでした。しかし、私は初心者であり、ここでの相互作用の目隠しです。あなたが提供できるどんな助けもありがとうございます。

Answer 1

このコードはいくつかの深刻なコードの匂いやグリッチに悩まされていますが、それは決して良いルビーコードには現れません。

def overlap?(other) 
    return !(other.checkout <= @checkin || other.checkin >= @checkout) 
end 

1. 1はルビー方式の最後の行にreturn明示的に使用しないでください。 Rubyは値を返し、最後の行は自動的に評価されます。
2. 人間の脳は否定的な二重/三重の状態に脆弱であるため、できるだけ条件を簡素化するようにしてください。
3. 否定と否定は、結合に展開されるため、否定と組み合わせて使用​​しないでください。
4. 私たちが一般的に使用される「チェックイン」と話している場合は、上記のコードは、さえ正しけれ疑う

合算「チェックアウト。」：

def overlap?(other) 
    # boolean for whether other checkout overlaps 
    co = other.checkout >= @checkin && other.checkout <= @checkout 
    # boolean for whether other checkin overlaps 
    ci = other.checkin >= @checkin && other.checkin <= @checkout 
    # return a result 
    ci || co 
end 

または、全体の電力を使用してルビーの：あなたの今後のクリスマスのレッスンから

def overlap?(other) 
    ci_co_range = @checkin..@checkout 
    ci_co_range.cover?(other.checkout) || ci_co_range.cover?(other.checkin) 
end 

または（:)

def overlap?(other) 
    [other.checkout, other.checkin].any? do |co_ci| 
    (@checkin..@checkout).cover?(co_ci) 
    end 
end 

ドキュメント：Range、Enumerable#any?。