《重构改善既有代码的设计》之重构列表--重新组织函数（一）

一、Extract Method(提炼函数)

你有一段代码可以被组织在一起并独立出来。

将这段代码放进一个独立函数中，并让函数名称解释该函数的用途。

Void printOwing(double amount){

printBanner();

//print details

System.out.println("name:" + _name);

System.out.println("amount:" + amount);

}

提炼后：

Void printOwing(double amount){

printBanner();

printDetails();

}

 

void printDetails(){

//print details

System.out.println("name:" + _name);

System.out.println("amount:" + amount);

}

动机

Extract Method是最常用的重构手法之一。当看到一个过长的函数或者一段需要注释才能让人理解用途的代码，就会将这段代码放到一个独立函数中。

有几个原因造成我喜欢简短而命名良好的函数。首先，如果每个函数的粒度都很小，那么函数被复用的机会就更大；其次，这会使高层函数读起来就像一系列注释；再次，如果函数都是细粒度，那么函数的覆写也会更容易些。

做法

1、创造一个新函数，根据这个函数的意图来对它命名（以它“做什么”来命名，而不是以它“怎样做”命名）。

2、将提炼出来的代码从源函数中复制到新建的目标函数中。

3、仔细检查提炼出的代码，看看其中是否引用了“作用域限于源函数”的变量（包括局部变量和临时变量）。

4、检查是否有“仅用于被提炼代码段”的临时变量。如果有，在目标函数中将它们声明为临时变量。

5、检查被提炼代码段，看看是否有任何局部变量的值被它改变。如果一个临时变量值被修改了，看看是否可以将被提炼代码段处理为一个查询，并将结果赋值给相关变量。如果很难这样做，或如果被修改的变量不止一个，你就不能仅仅将这段代码原封不动地提炼处理。你可能需要先运用Split Temporary Variable，然后再尝试提炼。也可以使用Replace Temp with Query将临时变量消灭掉。

6、将被提炼代码段中需要读取的局部变量，当做参数传给目标函数。

7、处理完所有局部变量之后，进行编译。

8、在源函数中，将被提炼代码段替换为对目标函数的调用。

9、编译、测试。

二、Inline Method(内联函数)

    一个函数的本体与名称同样清晰易懂。

在函数调用点插入函数本体，然后移除该函数。 

int Rating(){

    Return(moreThanFiveLateDliveries())?2:1;

}

Boolean moreThanFiveLateDeliveries(){

Return _numberOfLateDeliveries > 5;

}

转换后：

int Rating(){

    Return (_numberOfLateDeliveries > 5)?2:1;

}

动机

本书经常以简短的函数表现动作意图，这样会使代码更清晰易懂。但有时候你会遇到某些函数，其内部代码函数名称一样清晰易读。也可能你重构了函数，使得其内容和其名称变得同样清晰。果真如此，你就应该去掉这个函数，直接使用其中的代码。间接性可能带来帮助，但非必要的间接性总是让人不舒服。

另一种需要使用Inline Method 的情况是：你手上有一群组织不甚合理的函数。你可以将它们都内联到一个大型函数中，再从中提炼出组织合理的小型函数。

做法

1、检查函数，确定它不具多态性。

?  如果子类继承了这个函数，就不要将此函数内联，因为子类无法覆写一个根本不存在的函数。

2、找出这个函数的所有引用点。

3、将这个函数的所有被调用点都替换为函数本体。

4、编译、测试。

5、删除该函数的定义。

三、Inline Temp(内联临时变量)   

你有一个临时变量，只被一个简单表达式赋值一次，而它妨碍了其他重构手法。

将所有对该变量的引用动作，替换为对它赋值的那个表达式本身。

Double basePrice = anOrder.basePrices();

Return (basePrice > 1000);

转换后：

Return ( anOrder.basePrices() > 1000);

动机

Inline Temp多半是作为Replace Temp with Query的一部分使用，所以真正的动机出现在后者那儿。唯一单独使用Inline Temp的情况是：你发现某个临时变量被赋予某个函数调用的返回值。一般来说，这样的临时变量不会有任何危害，可以放心地把它留在那儿。但如果这个临时变量妨碍了其他重构手法，例如Extract Method，你就应该将它内联化。

做法

1、检查给临时变量赋值的语句，确保等号右边表达式没有副作用。

2、如果这个临时变量并未声明为final ，那么就将它声明为final ，然后编译。（这可以检查该临时变量是否真的只被赋值一次）

3、找到该临时变量的所有引用点，将它们替换为“为临时变量赋值”的表达式。

4、每次修改后，编译并测试。

5、修改完所有引用点后，删除该临时变量的声明和赋值语句。

6、编译、测试。