FC2ブログ

リファクタリング:一時変数の分離

複数回代入される一時変数があるが、それはループ変数でも
一時変数を集める変数でもない場合、代入ごとに別の一時変数に分けます。

1つの変数が2つ以上のことに使われていると、コードを読む人が非常に混乱します。
複数の責任を担う変数は、それぞれの責任を担う変数に置き換えられるべきです。

int temp = _strength + _weapon;
Debug.WriteLine("攻撃力:{0}", temp);
temp = _defence + _armor;
Debug.WriteLine("防御力:{0}", temp);

int totalStrength = _strength + _weapon;
Debug.WriteLine("攻撃力:{0}", totalStrength);
int totalDefence = _defence + _armor;
Debug.WriteLine("防御力:{0}", totalDefence);
リファクタリング―プログラムの体質改善テクニック (Object Technology Series)

Comment

Comment Form
公開設定

Trackback


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