重新组织函数(4)

前言

今天我们继续来讨论重新组织函数的重构手法.

正文

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

示例

if((platform.toUpperCase().indexOf("MAC") > -1) && 
(browser.toUpperCase().indexOf("IE") > -1) && resize >0){
  // do something
}
//我们可以看到上面的表达式特别长, 也特别难以理解, 所以我们可以引入解释性变量来改写
final boolean isMacOs = platform.toUpperCase().indexOf("MAC") > -1;
final boolean isIEBrowser = browser.toUpperCase().indexOf("IE") > -1;
final boolean wasResized = resize >0;
if(isMacOs && isIEBrowser && wasResized){
  //do something
}

解释

通过上面的示例就能发现, 当我们的表达式特变长会变得很难理解, 这时候引入解释性变量就能提高我们对程序的理解. 所以这是很重要的重构手法.

但是大家有没有发现, 如果我们把每个表达式语句变为函数是不是更好呢? 也就是运用 Extract Method. 示例如下:

private boolean isIEBrowser(){
  return browser.toUpperCase().indexOf("IE") > -1;
}
private boolean isMacOs(){
  return platform.toUpperCase().indexOf("MAC") > -1;
}
private boolean wasResized(){
  return resize >0;
}
if(isMacOs() && isIEBrowser() && wasResized()){
  //do something
}

我们发现使用Extract Method, 使得表达式能被整个类使用, 更具有通用性. 所以我认为, 能使用Extract Method就尽量使用, 当不能使用Extract Method, 才使用Introduce Explaining Variable, 这种方式最好!

Split Temporary Variable(分解临时变量)

示例

double temp = 2*(_heigh+_width);
System.out.println(temp);
temp = _height*_width;
System.out.println(temp);

//我们发现上面的temp变量含糊不清, 不够语义化, 对temp进行了两次赋值很容易让我们摸不着头脑, 所以我们应该更加语义化
final double perimeter = 2*(_heigh+_width);
System.out.println(temp);
final double area = _height*_width;
System.out.println(temp);

解释

当我们使用临时变量时, 应该只有两种情况可以无意义, 那就是:循环变量,结果收集变量.

如果你发现临时变量用来保存一段代码的运算结果, 以便稍后使用. 那么这种临时变量应该只能被赋值一次, 并且起一个语义化的名字. 如果不这么做就会使阅读者糊涂, 理解起来更费劲.

总结

今天我们又介绍了两种函数重构手法, 一个是Introduce Explaining Variable, 这个手法可以作为Extract Method的补充, 因为Extract Method更具有通用性. 另一个是Split Temporary Variable, 对临时变量取名应该语义化, 方便阅读者理解.