《重构改善既有代码的设计》之重构列表--重新组织函数(二)

来源:互联网 发布:java架构师工资 编辑:程序博客网 时间:2024/05/16 01:26

四、Replace Temp with Query(以查询取代临时变量)

你的程序以一个临时变量保存某一表达式的运算结果。

将这个表达式提炼到一个独立函数中。将这个临时变量的所有引用点为对新函数的调用。此后,新函数就可被其他函数使用。

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;

}

动机

临时变量的问题在于:它们是暂时的,而且只能在所属函数内使用。由于临时变量只在所属函数内可见,所以它们会驱使你写出更长的函数,因为只有这样你才能访问到需要的临时变量。如果把临时变量替换为一个查询,那么同一个类中的所有函数都将可以获得这份信息。这将带给你极大的帮助,使你能够为这个类编写更清晰的代码。

这个重构手法较为简单的情况是:临时变量只被赋值一次,或者赋值给临时变量的表达式不受其他条件影响。其他情况比较棘手,但也有可能发生。你可能需要先运用Split Temporary VariableSeparate Query form Modifier使情况变得简单一些,然后在替换临时变量。如果你替换的临时变量是用来收集结果的(例如循环中的累加值),就需要将某些程序逻辑(如循环)复制到查询函数中。

做法

首先是简单情况:

1、找出只被赋值一次的临时变量。

2、将该变量声明为final

3、编辑

4、将“对该临时变量赋值”之语句的等号右侧部分提炼到一个独立函数中国。

5、编译、测试。

6、在该临时变量身上实施Inline Temp

五、Introduce Explaining Variable(引入解释性变量)

你有一个复杂的表达式。

将该复杂表达式(或其中一部分)的结果放进一个临时变量,以此变量名称来解释表达式用途。

If(platform.toUpperCase().indexOf("MAX") > -1) && (browser.toUpperCase().indexOf("IE") > -1) && wasInitialized() && resize > 0){

//do something

}

转换后:

Final boolean isMacOs = platform.toUpperCase().indexOf("MAX") > -1;

Final boolean isIEBrowser = browser.toUpperCase().indexOf("IE") > -1;

Final boolean wasResize = resize > 0;

If(isMacOs  && isIEBrowser && wasInitialized() && wasResize ){

//do something

}

动机

    表达式可能非常复杂而难以阅读。这种情况下,临时变量可以帮助你将表达式分解为比较容易管理的形式。

在条件逻辑中,Introduce Explaining Variable特别有价值:你可以用这项重构将每个子句提炼出来,以一个良好命名的临时变量来解释对应条件子句的意义。使用这项重构的另一种情况是,在较长的算法中,可以运用临时变量来解释每一步运算的意义。

Introduce Explaning Variable是一个很常见的重构手法,但我得承认,我并不常用它。我几乎总是尽量使用Extract Method来解释一段代码的意义。毕竟临时变量只在它所处的那个函数中才有意义,局限性较大,函数则可以在对象的整个生命周期中都有用,并且可被其他对象使用。但有时候,当局部变量使Extract Method难以进行时,我就使用Introduce Explaining Variable

做法

1、声明一个final临时变量,将待分解之复杂表达式的一部分动作的运算结果赋值给它。

2、将表达式中"运算结果"这一部分,替换为上述临时变量。

3、编辑、测试。

4、重复上述过程,处理表达式的其他部分。

六、Split Temporary Variable(分解临时变量)

你的程序有某个临时变量被赋值超过一次,它既不是循环变量,也不被用于收集计算结果。

针对每次赋值,创造一个独立、对应的临时变量。

Double temp = 2 * (_height + _width);

System.out.println(temp);

temp = _height * _width;

System.out.println(temp);

转换后:

Final double perimeter = 2 * (_height + _width);

System.out.println(perimeter);

Final double area = _height * _width;

System.out.println(area);

动机

临时变量有各种不同用途,其中某些用途会很自然地导致临时变量被多次赋值。“循环变量”和“结果收集变量”就是两个典型例子:循环变量(loop variable)会随循环的每次运行而改变(例如for(int i=0;i<10;i++));结果收集变量(collecting temporary variable)负责将“通过整个函数的运算而构成的某个值收集起来”。

除了这两种情况,还有很多临时变量用于保存一段冗长代码的运算结构,以便稍后使用。这种临时变量应该只被赋值一次。如果它们被赋值超过一次,就意味它们在函数中承担了一个以上的责任。如果临时变量承担多个责任,它就应该被替换为多个临时变量,每个变量只承担一个责任。同一个临时变量承担两件不同的事情,会令代码阅读者糊涂。

做法

1、在待分解临时变量的声明及其第一次被赋值处,修改其名称。

2、将新的临时变量声明为final

3、以该临时变量的第二次赋值动作为界,修改此前对该临时变量的所以引用点,让它们引用新的临时变量。

4、在第二次赋值处,重新声明原先那个临时变量。

5、编译、测试

6、逐次重复上述过程。每次都在声明处对该临时变量命名,并修改下次赋值之前的引用点。

原创粉丝点击