若有临时变量只用个表达式赋值了一次,并且这个变量妨碍其它重构。将对这个变量的所有引用换成这个表达式。
大多数内联临时变量是作为“以查询方法替换临时变量”重构方法的一部分,这才是真正目的。一般临时变量是无害的,如果它妨碍其它重构方法才需要内联。
这个重构可能会有很大的负作用,如果这个表达式的计算很费时,并且变量被引用多次,就有效率问题,如果这个表达式每次计算结果都不同,这就更麻烦。
示例:
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;
}
本文介绍两种重构技巧:一是将仅使用一次的临时变量直接替换为表达式,以简化代码并提高可读性;二是将频繁使用的表达式提炼为独立函数,减少代码冗余,提升维护效率。
292

被折叠的 条评论
为什么被折叠?



