重构
一.何谓重构？
重构：使用一系列重构手法，在不改变软件可观察行为的前提下，调整其结构，提高其可理解性，降低其修改成本。
二.为什么要重构？
- 重构改进软件设计
- 重构使软件更容易理解
- 重构帮助找到BUG
- 重构提高编程速度
三.何时重构？
- 三次法则：事不过三，三则重构
- 添加功能时重构
- 修复BUG时重构
- Review代码时重构
四.何时不该重构？
- 无法稳定运行直接重写不用重构
- 项目已经接近最后期限，不应该重构，虽然重构能够提高生产力，但是你没有足够的时间，这通常标示你其实早该进行重构了。
五.代码的坏味道
- Duplicated Code 重复代码
- Long Method 过长函数
- Large Class 过大的类
- Long Parameter List 过长参数列表
- Divergent Change 发散式变化
- Shotgun Surgery 散弹式修改
- Feature Envy 依恋情结
- Data Clumps 数据泥团
- Primitive Obsession 基本类型偏执
- Switch Statements switch惊悚现身 （使用多态性替换）
- Parallel Inheritance Hierarchies 平行继承体系
- Lazy Class 冗赘类
- Speculative Generality 夸夸其谈未来性
- Temporary Field 令人迷惑的暂时字段
- Message Chains 过渡耦合的消息链
- Middle Man 中间人
- Inappropriate Intimacy 狎昵关系
- Alternative Classes with Different Interfaces 异曲同工的类
- Incomplete Library Class 不完美的库类
- Data Class 纯稚的数据类
- Refused Bequest 被拒绝的遗赠
- Comments 过多的注释
五.常用的重构手法
5.1 Extract Method（提炼函数）
func printOwing(amount: Double) {
printBanner()
//print details
print("name:", self.name)
print("amount:", amount)
}
func printOwing(amount: Double) {
printBanner()
printDetails(amount: amount)
}
func printDetails(amount: Double) {
//print details
print("name:", self.name)
print("amount:", amount)
}
动机
当一个函数过长或者需要注释才能看懂的代码，可以将这段代码放入一个独立的函数中，并给他一个合适的命名。
简短而命名良好的函数的优点：函数粒度越小，它被复用的几率越大；其次，这会使高层函数读起来像一系列注释；再次，如果函数都是细粒度，子类重载父类函数也会容易得多。
做法
- 创建一个新函数，根据这个函数的意图来给他命名（以“做什么”来命名，而不要以“怎么做命名”）。
2.将要提炼的代码从源函数复制到新建的目标函数中。
3.将被提炼代码中需要读取的局部变量，作为参数传给目标函数。
4.处理完所有局部变量后，进行编译。
5.在源函数中，将被提炼的代码段替换为对目标函数的调用。
6.编译，测试。
5.2 Inline Method（内联函数）
func getRating() -> Int {
return valueMoreThanFive() ? 2 : 1
}
func valueMoreThanFive() -> Bool {
return self.value > 5
}
func getRating() -> Int {
return (self.value > 5) ? 2 : 1
}
动机
某些函数，其内部代码和它的名称一样清晰易读。
你手上有一群组织不合理的函数，你可以先把它们内联到一个大的函数里，在从中提炼出组织合理的小型函数。
使用了太多的中间层，使得系统中所有的函数只是对另外一个函数的简单调用，造成在这些调用中晕头转向。
做法
- 检查函数，确定它不具有多态性。
如果有子类重载它，不要将它内联，因为不能重载一个不存在的函数
2.找到这个函数所有被调用的点。
3.将这个函数所有被调用点都替换为函数本体。
4.编译，测试。
5.删除该函数的定义。
5.2 Move Method（搬移函数）
搬移函数
动机
“搬移函数”是重构理论的支柱。如果有一个类有太多行为，或如果一个类与另一个类有太多合作而形成高度耦合，就应该使用搬移函数。
一个函数使用另一个对象的次数比使用自己所驻的对象还要多，考虑是否把这个函数搬移到另一个对象所属的类中。
做法
- 检查源类中被源函数使用的一切特性（包括字段和函数），考虑他们是否也该被搬移。
如果某个特性只被你打算搬移的函数用到，就应该将它一并搬移。如果有另外的函数使用了这个特性，你可以考虑将该特性的所有函数全都一并搬移。有时候，搬移一组函数比逐一搬移简单些。
- 检查源类的子类和父类，看看是否有该函数的其它声明。
- 在目标类中声明这个函数。
你可以为此函数选择一个新名称——对目标类更有意义的名称
- 将源函数的代码复制到目标函数中。调整目标函数，使其能在新家正常运行。
- 编译目标类。
- 决定如何从源函数正确引用目标对象。
- 修改源函数，使之成为一个纯委托函数。
- 编译，测试。
- 决定是否删除源函数，或者将它作为一个委托函数保留下来。
- 如果要移除源函数，将源类中对源函数的所有调用，替换为对目标函数的调用。
- 编译，测试。
5.2 Rename Method（函数改名）
Rename Method
动机
函数的名称应该准确地表达它的用途。给函数命名有一个好方法：首先考虑应该给这个函数写上一句怎样的注释，然后想办法将注释变成函数名称。
如果你看到一个函数名称不能很好地表达它的用途，应该马上加以修改。好的函数名可以让其它人更容易读懂你的代码。要想成为一个编程高手，起名的水平是至关重要的。
做法
- 检查函数是否被父类或子类实现过。如果是，需要针对每个实现分别进行下列步骤。
- 声明一个新函数，将它命名为你想要的新名称。将旧函数的代码复制到新函数中，并进行适当调整。
- 编译。
- 修改旧函数，让它掉用新函数。
- 编译，测试。
- 找出旧函数所有被引用点，将它们全部修改为对新函数的引用。每次修改后，编译并测试。
- 删除旧函数。
如果旧函数是该类public接口的一部分，你可能无法安全地删除它。这种情况下，将它保留在原地，并将它标示为deprecated。
- 编译，测试。
5.2 Add Parameter（添加参数）
为此函数添加一个对象参数，让该对象带进函数所需信息。
Add Parameter
动机
你必须修改一个函数，而修改后的函数需要一些过去没有的信息，因此你需要给该函数添加一个参数。
做法
- 检查函数是否被父类或子类实现过。如果是，需要针对每个实现分别进行下列步骤。
- 声明一个新函数，名称与原函数相同，只是加上新添参数。将旧函数的代码复制到新函数中。
- 编译。
- 修改旧函数，让它掉用新函数。
- 编译，测试。
- 找出旧函数所有被引用点，将它们全部修改为对新函数的引用。每次修改后，编译并测试。
- 删除旧函数。
如果旧函数是该类public接口的一部分，你可能无法安全地删除它。这种情况下，将它保留在原地，并将它标示为deprecated。
- 编译，测试。
5.2 Remove Parameter（移除参数）
函数本体不再需要某个参数，将该参数去除
Remove Parameter
动机
程序员可能经常添加参数，确往往不愿意去掉它们。因为多余的参数不会引起任何问题，而且以后还可能用上它。
但是对于多态函数，情况有所不同。可能多态函数的另一个实现会使用这个参数，此时你就不能去除它。
做法
- 检查函数是否被父类或子类实现过。如果是，需要针对每个实现分别进行下列步骤。
- 声明一个新函数，名称与原函数相同，只是去除不必要的参数。将旧函数的代码复制到新函数中。
- 编译。
- 修改旧函数，让它调用新函数。
- 编译，测试。
- 找出旧函数所有被引用点，将它们全部修改为对新函数的引用。每次修改后，编译并测试。
- 删除旧函数。
如果旧函数是该类public接口的一部分，你可能无法安全地删除它。这种情况下，将它保留在原地，并将它标示为deprecated。
- 编译，测试。
5.2 Parameterize Method（令函数携带参数）
若干函数做了类似的工作，但在函数名称中却包含了不同的值。建立单一函数，以参数表达那些不同的值
Parameterize Method
动机
你可能会发现两个函数，它们做着类似的工作，但因少数几个值致使行为略有不同。
做法
- 新建一个带参数的函数，使它可以替换先前所有的重复性函数。
- 编译。
- 将调用旧函数的代码改为调用新函数。
- 编译，测试。
- 对所有旧函数重复上述步骤，每次替换后，修改并测试。
- 删除旧函数。
如果旧函数是该类public接口的一部分，你可能无法安全地删除它。这种情况下，将它保留在原地，并将它标示为deprecated。
- 编译，测试。
5.2 Replace Parameter with Explicit Methods（以明确函数取代参数）
你有一个函数，其中完全根据参数值不同而采取不同的行为。针对该参数的每一个可能值，建立一个独立函数
重构前：
func setValue(name: String, value: CGFloat) {
if name == "height" {
self.height = value
} else if name == "width" {
self.width = value
}
}
重构后：
func setHeight(value: CGFloat) {
self.height = value
}
func setWidth(value: CGFloat) {
self.width = value
}
动机
如果某个参数有多种可能的值，而函数内又以条件表达式检查这些参数值，并根据不同参数值做出不同的行为，那么就应该使用本项重构。
接口更清晰，相比之下，Switch.setOn()比Switch.setState(true)要清楚得多。
做法
- 针对参数的每一种可能值，新建一个明确函数。
- 修改条件表达式的每个分支，使其调用合适的新函数。
- 修改每个分支后，编译并测试。
- 修改原函数的每一个被调用点，让它们调用合适的新函数。
- 编译，测试。
- 所有调用端都修改完毕后，删除原函数。
5.2 Preserve Whole Objcet（保持对象完整）
你把某个对象的若干值作为函数调用时的参数，改为传递整个对象
重构前：
let low = daysTempRange.low
let high = daysTempRange.high
withinPlan = plan.withinRange(low, high)
重构后：
withinPlan = plan.withinRange(daysTempRange)
动机
有时候，你会将同一对象的若干数据项作为参数，传递给某个函数。这样做的问题在于：万一将来被调用函数需要新的数据项，你就必须查找并修改对此函数的所有调用。
使参数列表变短，提高代码可读性。
弊端：如果你传的是数值，被调用的函数只依赖于这些数值。如果你传递的是整个对象，被调用的函数所在的对象就需要依赖参数对象。
5.2 Replace Parameter with Methods（以函数取代参数）
对象调用某个函数，并将所得结果作为参数，传递给另一个函数。而另一个函数本身也能调用前一个函数。让参数接受者去除该项参数，并直接调用前一个函数
重构前：
let basePrice = quantity * itemPrice
let discountLevel = self.getDiscountLevel()
let finalPrice = discountedPrice(basePrice, discountLevel)
重构后：
let basePrice = quantity * itemPrice
let finalPrice = discountedPrice(basePrice)
动机
如果函数可以通过其它途径获得参数值，那么它就不应该通过参数取得该值。
做法
- 如果有必要，使用Extract Method将参数的计算过程提炼到一个独立函数中。
- 将函数本地内引用该参数的地方改为调用新建的函数。
- 每次替换后，编译并测试。
- 全部替换完成后，使用Remove Parameter将该参数去掉。
5.2 Introduce Parameter Object（引入参数对象）
某些参数总是很自然地同时出现
Introduce Parameter Object
动机
你常会看到特定的一组参数总是一起被传递，可以运用一个对象包装所有这些数据。
当你把这些参数组织到一起后，往往很快可以发现一些行为可被移至新建的类中。
做法
- 新建一个类，用以表现你想替换的一组参数。
- 编译。
- 针对使用该组参数的所有函数，使用Add Parameter，传入新建类的实例对象，并将此参数值设置为null。
- 对于这些参数中的每一项，从函数名中移除掉，并修改调用端和函数本体，让它们都通过新的参数对象取得该值。
- 每去除一个参数，编译并测试。
- 将原先的参数全部去掉之后，观察有无适当的函数可以运用Move Method搬移到参数对象之中。
被搬移的可以是整个函数，也可以是某个函数的部分代码。如果是后者，需要先用Extract Method将该部分代码提炼到一个新的函数中。
5.2 Hide Method（隐藏函数）
有一个函数，从来没有被其他任何类用到，将这个函数修改为private
image.png
动机
一个类，暴露给其他类的接口越多，越容易产生耦合。在一个函数确实需要被其他类访问之前，应该让它对其他类不可见。
做法
- 经常检查有没有可能降低某个函数的可见度。
- 尽可能降低所有函数的可见度。
- 每完成一组函数的隐藏之后，编译并测试。
5.2 Replace Constructor With Factory Method（以工厂函数取代构造函数）
你希望在创建对象时不仅仅是简单的构建动作，将构造函数替换为工厂函数
重构前：
Employee(type: Int) {
self.type = type
}
重构后：
static func create(type: Int) -> Employee {
return Employee(type: type)
}
动机
最明显的动机就是，在派生子类的过程中以工厂函数取代类型码。
做法
- 新建一个工厂函数，让它调用现有的构造函数。
- 将调用构造函数的代码改为调用工厂函数。
- 每次替换后，编译并测试。
- 将构造函数声明为private。
- 编译。
5.2 Pull Up Field（字段上移）
两个子类拥有相同的字段，将该字段移至父类
Pull Up Field
动机
如果子类是分别开发的，或者是在重构过程中组合起来的，你常会发现它们拥有重复性，特别是字段更容易重复。如果它们重复了，你就可以将它们归纳到父类中去。
做法
- 针对待提升的字段，检查它们的所有被使用点，确认它们以同样的方式被使用。
- 如果这些字段的名称不同，先将它们改名，使每一个名字都和你想为父类字段取的名字相同。
- 在父类中新建一个字段，并移除子类中的字段。
4.编译，测试。
5.2 Pull Up Method（函数上移）
有些函数，在各个子类中产生完全相同的结果。将该函数移至超类中。
Pull Up Method
动机
如果函数在各个子类中的函数体都相同，这就是最显而易见的适用场合。
Pull Up Mehod常常紧随其它重构而被使用。也许你能找出若干个身处不同子类中的函数，而它们又可以通过其它重构手法调整成为相同的函数。这时候，你就可以先调整函数，然后再将它们上移到父类中去。当然，如果你足够自信，也可以一次完成这两个步骤。
做法
- 检查待上移的函数，确定它们是完全一致的。
- 如果待上移的函数的名字不同，先使用Rename Method将它们都修改为你想要在父类中使用的名字。
- 在父类中新建一个函数，将某一个待上移函数的代码复制到其中，做适当的调整，然后编译。
- 移除一个待上移的子类函数， 编译，测试。
- 逐一移除待上移的子类函数，直到只剩下父类中的函数为止。每次移除之后都需要测试。
- 观察该函数的调用者，看看是否可以改为使用父类类型的对象。
5.2 Pull Up Constructor Body（构造函数本体上移）
在父类中新建一个构造函数，并在子类构造函数中调用它
重构前：
class Manager: Employee {
init(name: String, id: String, grade: Int) {
self.name = name
self.id = id
self.grade = grade
}
}
重构后：
class Employee {
init(name: String, id: String) {
self.name = name
self.id = id
}
...
}
class Manager: Employee {
init(name: String, id: String, grade: Int) {
self.grade = grade
super.init(name: name, id: id)
}
...
}
动机
如果你看到各个子类中的函数有共同行为，第一个念头就是将共同行为提炼到一个独立函数中，然后将这个函数上移到父类。对于构造函数而言，它们彼此的共同行为往往就是“对象的构建”。
做法
- 在父类中定义一个构造函数。
- 将子类构造函数中的共同代码搬移到父类构造函数中。
- 将子类构造函数中的共同代码删掉，改成调用新建的父类构造函数。
- 编译，测试。
5.2 Pull Down Method（函数下移）
父类中的某个函数只与部分子类有关，将这个函数移到相关子类去
Pull Down Method
动机
Pull Down Method与Pull Up Method刚好相反。当我们有必要把某些行为从父类移至子类时，就使用Pull Down Method。使用Extract SubClass之后你可能会需要它。
做法
- 在所有子类中声明该函数，将父类中的函数本体复制到每一个子类函数中。
- 删除父类中的函数。
- 编译，测试。
- 将该函数从所有不需要它的那些子类中删掉。
- 编译，测试。
5.2 Pull Down Field（字段下移）
父类中的某个字段只被部分子类用到，将这个字段移到需要它的那些子类中去
Pull Down Field
动机
与Pull Up Field刚好相反。如果只有部分子类需要父类中的一个字段，可以使用本项重构。
做法
- 在所有子类中声明该字段。
- 将该字段从父类中移除。
- 编译，测试。
- 将该字段从所有不需要它的那些子类中删掉。
- 编译，测试。
5.2 Extract Subclass（提炼子类）
类中的某些特性只被某些实例用到，新建一个子类，将那一部分特性移到子类中
Extract Subclass
动机
与Pull Up Field刚好相反。如果只有部分子类需要父类中的一个字段，可以使用本项重构。
做法
- 在所有子类中声明该字段。
- 将该字段从父类中移除。
- 编译，测试。
- 将该字段从所有不需要它的那些子类中删掉。
- 编译，测试。
5.2 Extract Superclass（提炼父类）
两个类有相似的特性。为两个类建立一个父类，将相同特性移至父类。
Extract Superclass
动机
重复代码是系统中最糟糕的东西之一。如果你在不同的地方做同一件事情，一旦需要修改那些动作，你就需要修改每一份代码。
如果两个类以相同的方式做类似的事情，你就需要使用继承机制来去除这些重复。
做法
- 为原本的类创建一个空白父类。
- 运用Pull Up Filed、Pull Up Method、 Pull Up Constructor Body逐一将子类中的共同元素上移到父类。
- 每次上移后，编译并测试。
- 检查留在子类中的函数，看它们是否还有共通成分。如果有，可以使用Extract Method将共同部分提炼出来，然后使用Pull Up Method将提炼出的函数上移到父类中。
- 将所有共通元素上移到父类后，检查子类的所有用户。如果它们只使用了共同接口，你就可以把调用它们的对象类型改为父类类型。
5.2 Replace Inheritance with Delegation（以委托代替继承）
在子类中新建一个字段持有父类的对象；调整子类函数，令它改而委托父类；然后去掉两者的继承关系
Replace Inheritance with Delegation
动机
设计模式中讲到，优先使用组合而不是继承。
但是你常常会遇到这样的情况：一开始继承了一个类，随后发现父类中的许多操作并不真正适用于子类。这是你可以使用组合来代替继承。
做法
- 在子类中新建一个字段，使其引用父类的一个对象，并将它初始化为self。
- 修改子类中的所有函数，让它们不再使用父类，转而使用上面新加的那个字段。
- 去除两个类之间的继承关系，新建一个受托类的对象赋给受托字段。
- 对于父类中的每一个函数，在子类中添加一个简单的委托函数。
- 编译，测试。
5.2 Extract Class（提炼类）
某个类做了应该由两个类做的事。建一个新类，将相关字段和和函数搬移到新类。
Extract Class
动机
单一职责原则说到，一个类应该只有一个职责。
如果一个类职责过于多，过于复杂，就应该把它们分离到一个单独的类中。
做法
- 决定如果分解类所负的责任。
- 建立一个新类，用以表现从旧类中分离出的职责。
- 建立从旧类访问新类的连接关系。（大部分情况是让旧类持有新类类型的变量）
- 对于每一个你想搬移的字段，使用Move Filed进行搬移。
- 每次搬移后编译，测试。
- 使用Move Method将必要函数搬移至新类。先搬较低层的函数，后搬较高层的函数。
- 每次搬移后编译，测试。
- 检查，精简每个类的接口。
- 决定是否公开新类。如果你确实需要公开它，就需要决定让它成为引用类型还是值类型。
六、重构示例（心动日常查看距离功能重构）
心动日常iOS的查看距离会根据用户所处的位置使用合适的地图，如果双方都在国内使用高德地图，如果有一方在国外使用苹果地图。
旧版本双方都有位置信息时才会展示地图，新版本接到新的需求：默认展示地图，只要一方有位置信息在地图上对应的位置展示人物头像，人物头像上增加用户位置权限标识。
接到需求后，我先熟悉代码。经过一段时间的熟悉，我发现了一些问题：
1.控制器类持有MAMapView和YLAppleMapView类的对象，他们的功能类似并且是互斥的，可以抽象出相同的接口。
2.一些实例变量和函数很明显应该封装到地图类内部，而不是放在ViewController类里。
3.需要调用对应地图类对象时，都要使用if语句来判断
因为这些问题，如果我要修改现有代码，我必须要修改对应的高德地图和苹果的图部分的代码。但是因为他们所在函数的函数名和具体的业务逻辑可能很不一致，找起来可能会很麻烦。
如果我要往地图上添加头像，需要判断当前是什么地图，并分别调用不同的方法。想想就很头大，于是我决定重构。
重构前后代码结构对比：
UML图-重构前
UML图-重构后
ViewController成员-重构前
ViewController成员-重构后
地图相关的成员变量-重构前
地图相关的成员变量-重构后
具体重构过程通过git代码演示。
网友评论