如何清理这个非常简单的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