Remove Assignments to Parameters(パラメーターへの代入の削除)


HOME > プログラミング > リファクタリング > Remove Assignments to Parameters リファクタリング プログラムの体質改善テクニック

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