[リファクタリングを行いやすい状況]
・ 一度だけ代入される一時変数
・ 代入される値をつくる式が副作用を起こしにくい。
[Example: from Refactoring by Martin Fowler]
double basePrice = _itemprice * _quantity;
if (basePrice > 10000)
return basePrice * 0.95
else
return basePrice * 0.98
↓
if (baseProce() > 1000)
return basePrice * 0.95
else
return basePrice * 0.98
double basePrice() {
return _itemprice * _quantity;
}
0 件のコメント:
コメントを投稿