リファクタリングする上でのポイント（if文）

リファクタリングとは

リファクタリングとは、冗長なコードを削除したり、汎用性のある効率のいいコーディングに書き換えたりして、ソースを改善すること。

if文のリファクタリングを行うためのポイント

• if・elseの順番
• 条件式は肯定形を使う
• 単純な条件を先に書く
• ネストを浅くする

if・elseの順番
左側には、値が変化する「調査対象」の式を右側には、値が変化しない「比較対象」の式を書くのがおすすめ。

サンプル

 if value >= 5
  # 処理内容
 end

if/elseの場合

• 条件式は肯定形を使う
  • 「!=」 より 「==」 を使う
• 単純な条件を先に書く

# bad
if  a && b == 0
  if c
    # 処理
  end
end

# good
if c
  if a && b == 0
    # 処理
  end
end

• ネストを浅くする

ネストとは入れ子構造のこと
if文の中にif文を入れるなどして、入れ子構造になるケースがあるが、ネストが深いほどコードが読みにくくなるため、浅くするようにできるだけ心がけるのが大切。