读完《重构——改善既有代码的设计》和《重构与模式》,有了些许感想,先与大家分享一下。当我们已经对设计模式倒背如流时,却往往发现在实际代码编写中有生搬硬套的感觉。设计模式是前人经验的总结,直接拿来用合不合适呢?这让我想起了大学一位老师告诉我们的一条学习的道路“知识,理论,智慧”。设计模式是很一种优雅的“智慧”,但对于我们初学者来说还仅仅是留存于文字的“知识”。把“知识”融合到自己的开发中,在不断探索和总结中形成自己“理论”,再应用到实际中,那么这才是是真正属于我们自己的“智慧”。重构恰恰是由“知识”到“理论”的必经之路,而书中的各种重构方法无疑是这条路上清晰的路标。
代码的坏味道大家一看都不会陌生,绝对是在我们的编程中如影随形的,现在把相应的重构方法和设计模式总结出来,方便自己参考查阅,也希望对大家有所帮助。
友情提示:下面所列出的不是公式,不是别的重构方法不能用,也不是非要重构到相应的设计模式,因为不论是重构还是应用设计模式,一切的目的都是为了软件构架的“优雅”,而不是炫耀技术。另外,两位作者在描述重构步骤的时候,都不断重复着“编译并通过测试”的步骤,这无疑是在强调测试的重要性,和重构的递进性,切不可一措而蹴。
代码的坏味道 | 一般重构方法 | 使用模式重构 |
重复代码 | 提炼方法 提取类 方法上移 替换算法 链构造方法 |
构造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统一接口 |
作者:Angelo Lee
本文版权归作者和博客园共有,欢迎转载,但未经作者同意必须保留此段声明,且在文章页面明显位置给出原文连接,否则保留追究法律责任的权利.