代码重构-寻找坏味道

代码重构-寻找坏味道

本文参考了一下书籍：

《重构-改善既有代码的设计》

《重构手册》

1. 重复的代码

a) 封装成类，使用组合方式。

b) 封装成方法，通常是在同一类中。

c) 参数少的方法调用参数多的代码，并赋予默认值。

d) 提取公共基类，把共同的代码放到父类。

2. 过长

a) 方法

i. 提取子方法。

ii. 移动这个方法动作外的函数到外面。

b) 类

i. 提取类（一起变化的，逻辑上有依赖的变量封装在一起）

ii. 考虑move method来搬移不属于该类的方法。

c) 参数列

i. 封装成一个代表参数的对象。

ii. 传递这个参数对象代替过长的参数序列。

3. 名不达意

a) 类名

i. Rename 类名。

ii. 封装提取其他类。

iii. Move method。

b) 方法名

i. Rename method name。

ii. 封装提取方法。

iii. 搬移方法。

c) 变量名

i. Rename 

4. 过多的Null检查

a) 使用Null Object模式。

5. 多重职责

a) 类

i. 拆分类

b) 方法

i. 拆分方法

c) 接口

i. 拆分接口

d) 变量

6. 注释

a) 提取封装方法

b) 给方法提供一个表意的名字。

c) 给涉及到得变量提供一个表意的名字。

7. 方法中颗粒失衡

a) 方法中有很粗的方法调用，又有很细粒度的细节操作。

b) 应该为细粒度的细节操作封装成方法，以供调用。

8. 不一致的变化：某个类因为不同的原因在不同的方向上发生变化。

a) 拆分类。

9. 散弹式修改：一个变化，多处修改。

a) 把相同依赖的变化，封装到一个类里。

10. 依恋情结：函数对某个类的兴趣高过于其本身。

a) 搬移方法。

11. 复杂的逻辑运算表达式

a) 提取方法。

b) 给一个合适的名字。

12. 数据泥团：多处函数引用类似的参数序列

a) 可以把相关参数提取封装类。

13. 暴漏过多的细节

a) 一个类中过多的public方法。

b) 在一个类里过多的操作了另一个类。（应该把给另一个类创建一个方法，以供前面的类调用。）

14. 子类并非父类的一种情况：本质不同或子类不想拥有父类的实现

a) 子类发生了畸变。

b) 应该使用组合的方法。

15. 逻辑与UI操作混搭。

a) 把逻辑运算和UI操作分离。

16. 基本类型偏执：类中有很多关系紧密的基本类型成员变量

a) 提取内部类。

17. 复杂的switch 表达式。

a) 使用策略模式，状态模式改善。