このメソッドをどのようにリファクタリングするのですか？

Question

私は、間隔の開始が他の間隔の終わりと等しい場合、日付間隔を別の日付間隔とマージする方法を書いています。

def merge 
    slots.each do |slot| 
    if self.start == slot.end 
     slot.end = self.end 
     slot.save 
     return self 
    elsif self.end == slot.start 
     slot.start = self.start 
     slot.save 
     return self 
    end 
    end 
    self.save 
end 

selfは、私がslots配列に存在する間隔のいずれかでそれをマージする間隔です。 この方法をリファクタリングするにはどうすればよいでしょうか？

ノー成功でこのような何かにそれを回すことを試みた：

def merge 
    slots.each do |slot| 
    if slot.one_bound == slot.other_bound 
     slot.other_bound = self.other_bound 
     slot.save 
     return self 
    end 
    end 
    self.save 
end 

Answer 1

私は、これは、より読みやすいと思う：

def merge 
    slots.each do |slot| 
    if start_the_same_as_end_for?(slot) 
     slot.end = self.end 
    elsif end_the_same_as_start_for?(slot) 
     slot.start = self.start 
    end 

    slot.save 
    end 
end 

private 

def start_the_same_as_end_for?(slot) 
    self.start == slot.end 
end 

def end_the_same_as_start_for?(slot) 
    self.end == slot.start 
end 

Answer 2

あなたが範囲に演算子を追加するには、このgemを使用することができます。

# gem install range_operators 
require 'date' 
require 'range_operators' 

range1 = Date.new(2014, 5, 10) .. Date.new(2014, 8, 10) 
range2 = Date.new(2014, 8, 10) .. Date.new(2014, 8, 20) 
range3 = Date.new(2014, 8, 11) .. Date.new(2014, 8, 20) 

puts range1 + range2 
#=> 2014-05-10..2014-08-20 

ボーナスとして、2番目の範囲が正しい場合にも有効ですr最初の範囲：

puts range1 + range3 
#=> 2014-05-10..2014-08-20 

Range#は範囲の配列を返します。この配列に要素が1つしかない場合は、範囲がマージされたことを意味します。

result = range1 + range2 
if result.size == 1 then 
    # save result.first 
end