重构手法-重新组织函数

最新推荐文章于 2021-10-17 15:57:28 发布

长江七号zzj

最新推荐文章于 2021-10-17 15:57:28 发布

阅读量405

点赞数

分类专栏：重构-改善既有代码的设计

本文链接：https://blog.csdn.net/u012867699/article/details/77606201

版权

重构-改善既有代码的设计专栏收录该内容

3 篇文章 0 订阅

订阅专栏

重构手法中，很大一部分是对函数进行处理，问题大部分来源于Long Methods（过长函数）。

下面介绍一下针对在函数中使用到的重构手法

一、Extract Method（提炼函数）

1、定义

把一段代码从原先函数中提取出来，放进一个单独函数中，并让函数名称解释该函数的用途。

2、动机

有过长的函数或者一段需要注释才能让人理解用途的代码，这时候需要将这段代码放进一个独立函数中。

如果每个函数的粒度都很小，那么函数被复用的机会就更大；

使高层函数读起来就像一系列注释；

如果函数都是细粒度，那么函数的覆写就会更容易些。

3、做法

Extract Method最大的困难就是处理局部变量，包括传进源函数的参数和源函数所声明的临时变量，而临时变量则是其中一个主要的困难源头。

创造新函数，根据这个函数的意图命名；
将提炼出的代码从源函数复制到新建的目标函数中去；
查看提炼出来的代码是否引用了“作用域限于”源函数的变量；
若被提炼代码段，有局部变量被修改，看看是否可以将被提炼代码处理成一个查询，并将结果赋值给相关变量，或需要将临时变量消灭掉；
将被提炼代码段中需要读取的局部变量，当做参数传给目标函数；
处理完所有局部变量，在源函数中，将被提炼代码段替换为对目标函数的调用。

被提炼代码段：

case1：无局部变量->直接提炼

case2：有局部变量，被提炼代码段只是读取这些变量，并不修改->局部变量作为参数传给目标函数

case3：对局部变量赋值

变量只在被提炼代码中使用->将临时变量的声明移到被提炼代码段中，一起提炼
被提炼代码段之外的代码也使用了这个变量，之后未使用->直接在目标函数中修改；之后使用->目标函数返回该变量改变后的值

4、其他问题

如果返回的变量不止一个，安排多个函数，每个函数只返回一个值；

如果临时变量很多，会使提炼工作举步维艰，应该先减少临时变量，使用方法（Replace Temp with Query、Replace Method with Method Object）

二、Inline Method（内联函数）

1、定义

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

2、动机

内部代码和函数名称同样清晰易读
一群组织不甚合理的函数。可以将它们都内联到一个大型函数中，再从中提炼出组织合理的小型函数

3、做法

检查函数，确定不具有多态性：如果子类继承了这个函数，就不要将此函数内联，否则导致子类无法覆写
找出这个函数的所有被调用点，并替换为函数本体
删除该函数的定义

4、其他问题

对于递归调用、多返回点、内联至另一个对象中而该对象并无提供访问函数。。。对于这些复杂情况，不要使用这种重构手法。

三、Inline Temp（内联临时变量）

1、定义

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

2、动机

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

3、做法

检查给临时变量赋值的语句，确保等号右边的表达式没有副作用
如果这个临时变量并未被声明为final，那就将它声明为final，检查该临时变量是否真的只被赋值一次
找到临时变量的所有引用点，替换为为临时变量赋值的表达式
删除临时变量的声明和赋值语句

四、Replace Temp with Query（以查询取代临时变量）

1、定义

以一个临时变量保存某一表达式的运算结果。将这个表达式提炼到一个独立函数中，将这个临时变量的所有引用点替换为对新函数的调用。此后，新函数就可被其他函数调用。

2、动机

临时变量是暂时的，只能在所属函数内使用，并且为了访问临时变量，需要写出更长的函数
局部变量会使代码难以被提炼，所以尽可能替换为查询式

3、做法

找出只被赋值一次的临时变量。如果被赋值多次，考虑使用Split Temporary Variable将它分割成多个变量
将临时变量声明为final，检查该临时变量是否真的只被赋值一次
将对该临时变量赋值等号的右侧提炼到一个独立函数中。 1、可先将函数声明为private，以后有更多类需要，再放开访问限制。2、确保该函数无副作用，不修改任何对象