FC2ブログ

リファクタリング:重複した条件記述の断片の統合

条件式のすべての分岐に同じコードの断片がある場合、それを式の外側に移動します。

if (item.Type == Items.Potion)
{
_hp += PotionRecoveryAmount;
Update();
}
else if (item.Type == Items.Potion)
{
_hp += ElixirRecoveryAmount;
Update();
}

if (item.Type == Items.Potion)
{
_hp += PotionRecoveryAmount;
}
else if (item.Type == Items.Potion)
{
_hp += ElixirRecoveryAmount;
}

Update();
ときどき、条件記述のアクション部で実行されるすべてのコードが同じであることに気づくことがあります。
この場合は、そのコードを条件記述の外に出しましょう。
これによって、変わる部分と変わらない部分を明確に出来ます。

リファクタリング―プログラムの体質改善テクニック (Object Technology Series)

Comment

Comment Form
公開設定

Trackback


→ この記事にトラックバックする(FC2ブログユーザー)
上記広告は1ヶ月以上更新のないブログに表示されています。新しい記事を書くことで広告を消せます。