2. 内联函数(Inline Method)
2.1 动机
(1)在函数调用点插入函数本体,然后移除该函数。
(2)有时遇到某些函数,其内部代码和函数名称同样清晰易读,采用内联函数可以提供代码的执行速度(因为少掉了函数调用的开销)
(3)在Replace Method withd Method Object之前,将一些组织不合理的函数内联到一个大型函数,再从中提炼组织合理的小型函数时效果很好。
(4)使用太多的间接层时,特别是一部分函数只是对另一个函数的简单委托时,可同去掉一些无用的间接层。
2.2 做法
(1)检查函数,确定它不具有多态性。如果子类继承了这个函数,就不要将此函数内联,因为子类无法覆写一个根本不存在的函数。
(2)找出这个函数的所有被调用点。
(3)将这个函数的所有被调用点都替换为函数本体
(4)删除该函数的定义
2.3 范例
//重构前 int getRating() { return moreThanFiveLateDeliveries() ? 2 : 1; } bool moreThanFiveLateDeliveries() { return _numberOfLateDeliveries > 5; } //重构后 int getRating() { return (_numberOfLateDeliveries > 5) ? 2 : 1; }
2.4 思考
(1)对于递归、多返回点不适合用内联函数来重构
(2)内联至另一个对象中,而该对象并无提供访问函数。这种情况也不适合使用这个重构手法。
3. 内联临时变量
3.1 动机
(1)内联临时变量多半是作为Replace Temp with Query的一部分使用,所以真正的动机出现在后者那儿。
(2)如果某个临时变量被赋值为某个函数的返回值。一般这个临时变量不会有任何危害,可以保留下来。但这样的变量会妨碍重构,使重构变成比较困难,所以可以将它内联化。
3.2 做法
(1)检查临时变量赋值的语句,确保等号右边的表达式没有副作用
(2)将该变量声明改为const变量,以确定该变量真的只被赋值一次。
(3)找到该临时变量的所有引用点,将它们替换为“为临时变量赋值的”表达式
(4)删除该临时变量的声明和赋值语句。
3.3 范例
//内联临时变量 //重构前 double basePrice = anOrder.basePrice(); //basePrice只被赋值一次 return (basePrice > 1000) //重构后 return (anOrder.basePrice() > 1000);
4. 以查询取代临时变量(Replace Temp with Query)
4.1 动机
(1)临时变量的问题在于它们是暂时的,而且只能在所属函数使用。因此会导致写出来的函数更长,因为只有这样才能访问到需要的临时变量。可以将临时变量替换为一个查询,这将帮助我们编写更清晰的代码。
(2)Replace Temp with Query往往是ExtractMethod之前必不可少的步骤。因为局部变量会使代码难以被提炼,所以应尽可能将它们替换为查询函数。
4.2 做法
(1)找出只被赋值一次的临时变量。如果某个临时变量被赋值超过一次,考虑使用Split Temporary Variable将它们分割成多个变量。
(2)将该临时变量声明为const变量。然后编译,以确保只会被赋值一次
(3)将“对该临时变量赋值”之语句的等号右侧部分提炼到一个独立函数中。
(4)在临时变量身上实施Inline Temp。
4.3 范例
//重构前
//用查询替换临时变量 //重构前(目标:basePrice和discountFactor替换掉) double getPrice() { int basePrice = _quantity * _itemPrice; //_quantity和_itemPrice为成员变量 double discountFactor; if(basePrice > 1000) discountFactor = 0.95 else discountFactor = 0.98; return basePrice * discountFactor; }
//重构后
//重构后 double getPrice() { return basePrice() * discountPrice(); } //计算底价 double basePrice() { return _quantity * _itemPrice; } //打折因子(类中声明为private函数) double discountFactor() { //此处的basePrice()也是个查询函数,由于替换去除了临时变量 //使得discountFactor函数的重构成为可能! if(basePrice() > 1000) return 0.95; else return 0.98; }
4.4 思考
(1)如果临时变量保存循环中的累加信息,则整个循环可以被提炼到一个独立函数中去。
(2)这种手法会导致性能问题,但这种影响一般很小。