Ruby/Rails 从一行扩展 if 块 - rubocop

Question

我必须从一行修改我的 if 块。

def filename
 "#{model.external_id}-inquiry_process_summary_#{timestamp}.pdf" if original_filename
end

像这样：

def filename
  if original_filename
    result = model.inquiry_field_responses.joins(:inquiry_field).where(inquiry_fields: { name: 'company_name' })
    "#{result.first&.value}-inquiry_process_summary_#{timestamp}.pdf"
  end
end

这很简单，但我的 rubocop 尖叫Use a guard clause instead of wrapping the code inside a conditional expression中。 如果我在最后end之前添加Success ，我的一些规范会因错误而失败

期望响应具有状态代码（422）但它是：好的（200）

没有Success ，一切都会如其所愿地过去。

Answer 1

AFAIK，Rubocop 抱怨不早点回来。 这应该可以解决 Rubocop 问题：

def filename
  return unless original_filename

  result = model.inquiry_field_responses.joins(:inquiry_field).where(inquiry_fields: { name: 'company_name' })
  "#{result.first&.value}-inquiry_process_summary_#{timestamp}.pdf"
end

阅读有关保护条款的更多信息。