ruby：rubocop 抱怨将 if else 转换为 case 语句

发布于04月14日

我有下面的if-else声明，rubocop抱怨了Style/CaseLikeIf: Convert if-elsif to case-when个

if obj.is_a? Rest::Headers
  obj
elsif obj.is_a? Hash
  Rest::Headers.new(**obj)
elsif obj.is_a? Array
  Rest::Headers.new(**obj.to_h)
else
  raise 'Invalid object type for Headers!'
end

我把它转换成了案件陈述

case
when (obj.is_a? Rest::Headers)
  obj
when (obj.is_a? Hash)
  Rest::Headers.new(**obj)
when (obj.is_a? Array)
  Rest::Headers.new(**obj.to_h)
else
  raise 'Invalid object type for Headers!'
end

但现在鲁博科普抱怨了Do not use empty case condition, instead use an if expression次.有人知道我的案件陈述有什么问题吗？

推荐答案

Ruby希望您将其转换为与类型本身匹配的case.Ruby中的case不只是平等匹配.表格的陈述

case a
when b
  ...
end

case 将调用b === a(注意这里的三重相等，not表示相等)，如果是真的，则触发when块.

===在Object上定义为等同于==，但许多类会覆盖它.

正则表达式使用它来匹配右边的模式
范围判断右侧是否包含在范围内
类判断右侧是否有is_a?个类实例.

这是Rubocop建议你使用的第三个要点.

case obj
when Rest::Headers
  obj
when Hash
  Rest::Headers.new(**obj)
when Array
  Rest::Headers.new(**obj.to_h)
else
  raise 'Invalid object type for Headers!'
end

请注意，我从来没有明确地在这里调用is_a?；case街区自动知道该怎么做.