FC2ブログ

リファクタリング:パラメータへの代入の除去

引数への代入が行われている場合、一時変数を使います。
渡されたものを表現するためだけにパラメータを使うことで、混乱を防ぐことができます。

private void Damage(int monsterStrength)
{
if (999 < monsterStrength)
{
monsterStrength = 999;
}

_hp -= monsterStrength;
}

private void Damage(int monsterStrength)
{
int damage = monsterStrength;

if (999 < monsterStrength)
{
damage = 999;
}

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

Comment

Comment Form
公開設定

Trackback


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