SCUEL開発者ブログ

医療機関および介護事業所などのデータベース事業『SCUEL』を作っているエンジニア達のブログです。エンジニア募集中です!ご興味がありましたら、以下のフォームからお問い合わせください。 http://goo.gl/forms/8fPwBHDL2H

"Move `%s` out of conditional." の対処方法

Rubocopで、こんなエラーが出た時の対処方法。

Move `%s` out of the conditional.

%sには、メソッドの呼び出しなどが入る。


対処方法は、以下のように%sで指摘されているメソッド呼び出しが冗長になっているので、条件分岐のブロックの外にだすことで解決する。

Class: RuboCop::Cop::Style::IdenticalConditionalBranches — Documentation for rubocop (0.39.0)

@bad
if condition
  do_x
  do_z
else
  do_y
  do_z
end

@good
if condition
  do_x
else
  do_y
end
do_z