如何重构这个 if/else 语句？

Question

下面是 if/else 语句：

if number.events.blank?
  Resque.enqueue(ProcessNumbers, number.id)
elsif number.events && !number.events.ordered.first.status.downcase.include? 'delivered'
  Resque.enqueue(ProcessNumbers, number.id)
end

最终想说的是：“如果 number.events 为空，则运行 enqueue。或者 if number.events.ordered.first。 status.downcase.include 包含“已交付”（但要确保 number.events 实际上有某些内容）。”

Answer 1

我不是 Ruby 开发人员，但您在 elsif 中的 number.events 检查似乎是多余的，因为您的代码只会“落入”该 elsif 在 number.events 不为空的情况下。由于您遇到了 if...or 情况，更简洁的表达方式可能是：

if number.events.blank? || !number.events.ordered.first.status.downcase.include? 'delivered'
    Resque.enqueue(ProcessNumbers,number.id)

Answer 2

或者，

if !number.events.blank? && !number.events.ordered.first.status.downcase.include? 'delivered'
    Resque.enqueue(ProcessNumbers, number.id)

它将确保 number.events 实际上有一些东西。