分类： Java
2011-12-11 23:08:15
读完《重构——改善既有代码的设计》和《重构与模式》，有了些许感想，先与大家分享一下。当我们已经对设计模式倒背如流时，却往往发现在实际代码编写中有生搬硬套的感觉。设计模式是前人经验的总结，直接拿来用合不合适呢？这让我想起了大学一位老师告诉我们的一条学习的道路“知识,理论,智慧”。设计模式是很一种优雅的“智慧”，但对于我们初学者来说还仅仅是留存于文字的“知识”。把“知识”融合到自己的开发中，在不断探索和总结中形成自己“理论”，再应用到实际中，那么这才是是真正属于我们自己的“智慧”。重构恰恰是由“知识”到“理论”的必经之路，而书中的各种重构方法无疑是这条路上清晰的路标。
代码的坏味道大家一看都不会陌生，绝对是在我们的编程中如影随形的，现在把相应的重构方法和设计模式总结出来，方便自己参考查阅，也希望对大家有所帮助。
友情提示：下面所列出的不是公式，不是别的重构方法不能用，也不是非要重构到相应的设计模式，因为不论是重构还是应用设计模式，一切的目的都是为了软件构架的“优雅”，而不是炫耀技术。另外，两位作者在描述重构步骤的时候，都不断重复着“编译并通过测试”的步骤，这无疑是在强调测试的重要性，和重构的递进性，切不可一措而蹴。
| 代码的坏味道 | 一般重构方法 | 使用模式重构 |
| 重复代码 | 提炼方法 提取类 方法上移 替换算法 链构造方法 | 构造Template Method 以Composite取代一/多之分 引入Null Object 用Adapter统一接口 用Fatory Method引入多态创建 |
| 过长方法 | 提取方法 组合方法 以查询取代临时变量 引入参数对象 保持对象完整 | 转移聚集操作到Vistor 以Strategy取代条件逻辑 以Command取代条件调度程序 转移聚集操作到Collecting Parameter |
| 过长参数列 | 以方法取代参数 引入参数对象 保持对象完整 | |
| 条件逻辑过度复杂 | 分解条件式 合并条件式 合并重复的条件片段 移除控制标记 以卫语句取代嵌套条件式 以多态取代条件式 引入断言 | 以Strategy取代条件逻辑 转移装饰功能到Decorator 以State取代状态改变条件语句 引入Null Object |
| 分支语句 | 提取方法 转移方法 以子类取代类型代码 以多态取代条件式 已明确方法取代参数 | 以State/Strategy取代类型代码 引入Null Object 以Command替换条件调度程序 转移聚集操作到Visitor |
| 基本类型迷恋 程序代码过于依赖基本类型（int，string，double，array等低层次语言要素） | 以对象取代数据值 以类型取代类型代码 以子类取代类型代码 提取类 引入参数对象 以对象取代数组 | 以State取代状态改变条件语句 以Strategy取代条件逻辑 以Composite取代隐含树 以Interpreter取代隐式语言 转移装饰功能到Decorator 用Builder封装Composite |
| 数据泥团 在类的字段和参数列中，总是一起出现的数据 | 提取类 引入参数对象 保持对象完整 | |
| 令人迷惑的临时字段 | 提取类 | 引入Null Object |
| 组合爆炸 许多段代码使用不同种类或数量的数据或对象做同样的事情（例如使用特定条件和数据库查询） | 以Interpreter取代隐式语言 | |
| 过大类 | 提取类 提取子类 提取接口 复制被监视数据 | 以Command取代条件调度程序 以State取代状态改变条件语句 以Interpreter取代隐式语言 |
| 冗赘类 不再做很多工作或没有用的类 | 折叠继承关系 内联Singleton | |
| 不恰当的暴露 在客户代码中不应看到类的字段和方法，却是公开可见的 | 封装字段 封装群集 移除设置方法 隐藏方法 | 用Factory封装类 |
| 发散式变化 类经常因为不同的原因在不同方向上发生变化，显然是违反了单一职责原则 | 提取类 | |
| 霰弹式修改 如果遇到变化，必须在不同的类中作出相应的修改 | 转移方法 转移字段 内联类 | 将创建知识搬移到Factory |
| 依恋情结 方法对于某个类的兴趣高过对自己所处的宿主类 | 转移方法 提取方法 | 引入Strategy 引入Visitor |
| 平行继承体系 当为一个类增加一个子类时，也必须在另一个类中增加一个相应的子类 | 转移方法 转移字段 | |
| 夸夸其谈未来性 | 折叠继承关系 内联类 移除参数 移除方法 | |
| 过度耦合的消息连 不断的向一个对象索求另一个对象 | 隐藏委托 提取方法 转移方法 | 使用抽象引入Chain Of Responsibility |
| 中间转手人 类接口中有很多方法都委托给其他类 | 移除中间转手人 内联方法 以继承取代委托 | |
| 狎昵关系 类之间彼此依赖于其private成员 | 转移方法 将双向关联改为单向 提取类 隐藏委托 以继承取代委托 | |
| 异曲同工的类 | 重命名方法 转移方法 提取超类 | 用Adapter统一接口 |
| 不完善的程序库类 | 引入外加方法 引入本地扩展 | 用Adapter统一接口 用Facade封装类 |
| 纯稚的数据类 只拥有字段的数据类 | 封装字段 封装集合 移除设置方法 转移方法 隐藏方法 | |
| 被拒绝的遗赠 继承父类时，子类想要选择继承的成员 | 以委托取代继承 | |
| 过多的注释 为糟糕的代码写大量的注释 | 使用一起重构方法，使方法本身达到自说明的效果，让注释显得多余 | |
| 怪异解决方案 在同一系统中使用不同的方式解决同一问题 | 替换算法 | 用Adapter统一接口 |
