干燥 Ruby 三元

Question

我经常遇到这样的情况：我想做一些条件逻辑，然后返回一部分条件。如何在不重复 true 或 false 表达式中的条件部分的情况下执行此操作？

例如：

ClassName.method.blank? ? false : ClassName.method

有没有办法避免重复ClassName.method？

这是一个现实世界的例子：

PROFESSIONAL_ROLES.key(self.professional_role).nil? ? 
    948460516 : PROFESSIONAL_ROLES.key(self.professional_role)

Answer 1

假设您同意将 false 视为与 nil 相同的方式，则使用 ||：

PROFESSIONAL_ROLES.key(self.professional_role) || 948460516

这将返回 948460516 code> 如果 key 返回 nil 或 false，否则返回调用 key 的返回值。

请注意，如果 key 返回 nil 或 false，则仅返回 948460516，而不是返回空数组或字符串。由于您在第二个示例中使用了 nil? ，我认为这是可以的。但是，您在第一个示例中使用了 blank? （并且 blank? 对于空数组和字符串返回 true），所以我不确定。

Answer 2

如果你只想 DRY，那么你可以使用临时变量：

x = ClassName.method
x.blank? ? false : x

x = PROFESSIONAL_ROLES.key(self.professional_role)
x.nil? ? 948460516 : x

如果你不想使用临时变量，你可以使用块：

Proc.new do |x| x.blank? ? false : x end.call(ClassName.method)
Proc.new do |x| x.nil? ? 948460516 : x end.call(PROFESSIONAL_ROLES.key(self.professional_role))

对于你描述的情况（当你只想使用原始值时）默认检查失败），编写一个辅助方法很简单：

def x_or_default(x, defval, checker = :nil?)
  if x.send(checker) then defval else x end
end
x_or_default(ClassName.method, false, :blank?)
x_or_default(PROFESSIONAL_ROLES.key(self.professional_role), 94840516)

它与所描述的 || 方法非常相似，但也适用于您的 blank?例子。

我通常使用临时变量来处理这类事情。

Answer 3

我知道这看起来不太漂亮，但它确实让事情变得有点干燥。

a = "ABC"
b = (temp = a.downcase).length < 3 ? "---" : temp

如果您出于某种原因不想创建 temp 变量，则可以重用已存在的内容，例如 $_。