如何清理這個非常簡單的Rails函數？

Question

有沒有辦法讓這個Rails代碼漂亮起來？

def function 
    if new_record?      
    thing 
    else 
    thing + yet_another_thing 
    end 
end 

我不喜歡的thing重複這裏，所以我不知道是否有一個更清潔的方式。

感謝您的任何幫助。

Answer 1

這適用於支持+任何對象， （甚至是字符串。）

[thing, (yet_another_thing unless new_record?)].compact.inject(:+) 

它乾燥而可怕，就像被困在沒有水的沙漠中一樣。

---

您可能還能夠與閃避：

thing.dup.tap{|t| t << yet_another_thing unless new_record?} 

如果事情是一個整數這是不行的（你不能DUP它），也需要支持< <運營商。

也幹，但以不同的方式嚇人。

Answer 2

如果你不想重複thing，那麼這可能是一個解決方案。

def function 
    result = thing 
    result += yet_another_thing unless new_record? 
    result 
end 

Answer 3

你可以使用一個在線如果

def function 
    return thing if new_record? 
    thing + yet_another_thing 
end 

Answer 4

如果thing和yet_another_thing是字符串，你可以這樣做：

thing + (yet_another_thing unless new_record?).to_s 

Answer 5

三元算子呢？

def function 
    new_record? ? thing : (thing + yet_another_thing) 
end 

如果我們知道你在哪裏使用它或者包含在變量中，這將會更有幫助。

Answer 6

如果事情和yet_another_thing一些方法你打電話：

def function 
    thing 
    yet_another_thing if new_record? 
end