若有临时变量只用个表达式赋值了一次,并且这个变量妨碍其它重构。将对这个变量的所有引用换成这个表达式。
大多数内联临时变量是作为“以查询方法替换临时变量”重构方法的一部分,这才是真正目的。一般临时变量是无害的,如果它妨碍其它重构方法才需要内联。
这个重构可能会有很大的负作用,如果这个表达式的计算很费时,并且变量被引用多次,就有效率问题,如果这个表达式每次计算结果都不同,这就更麻烦。
示例:
double basePrice = anOrder.basePrice();
return (basePrice > 1000)
return (anOrder.basePrice() > 1000)
若使用一个临时变量保存了一个表达式的结果,把这个表达式提练成新函数,把这个临时变量所有的引用替换成新函数,这个新函数可以被它函数使用。
临时变量是暂时的,只能在所属的函数内使用,往往导致代码过长;将表达式抽取成函数,可以使代码更加清晰。
示例:
double basePrice = _quantity * _itemPrice;
if (basePrice > 1000)
return basePrice * 0.95;
else
return basePrice * 0.98;
if (basePrice() > 1000)
return basePrice() * 0.95;
else
return basePrice() * 0.98;
...
double basePrice()
{
return _quantity * _itemPrice;
}