是的，我们有一条干净的红宝石路。

discount.to_f.zero?

这个检查处理了大量的情况，即折扣可能是nil，折扣可能是int 0，折扣可能是浮动0.0，折扣可能是字符串“0.0”，“0”。

unless discount.nil? || discount == 0
  # ...
end

if (discount||0) != 0
  #...
end

你可以把discount初始化为0，只要你的代码保证在初始化之前不尝试使用它。我想这只会删除一张支票，我想不出还有什么。

您可以将空行转换为整数值并检查零?。

"".to_i.zero? => true
nil.to_i.zero? => true

unless [nil, 0].include?(discount) 
  # ...
end