Remove Assignments to Parametersはパラメーターへの代入をなくすというリファクタリングです。
メソッドはいくつかパラメーターをとることも多いですが、そのパラメーターにメソッド内で新たな値を代入するのは、不注意なバグを引き起こしかねないためです。
以下の例のように、パラメーターへの代入は避け、新しい一時変数を利用します。
リファクタリング前:
public int IncludeConsumerTax(int price)
{
price = price * 1.05;
Console.WriteLine("税込価格" + price.ToString());
return price;
}
リファクタリング後:
public int IncludeConsumerTax(int price)
{
int includingConsumerTaxPrice = price * 1.05;
Console.WriteLine("税込価格" + includingConsumerTaxPrice.ToString());
return includingConsumerTaxPrice;
}
|
Copyright(c) 2012 WoodenSoldier Software |