历时一个多月的时间,代码重构系列终于写完了,代码的坏味道大家一看都不会陌生,绝对是在我们的编程中如影随形的,现在把相应的重构方法和设计模式总结出来,以供大家参考.因为不论是重构还是应用设计模式,一切的目的都是为了软件构架的“优雅”,而不是炫耀技术。
步步为营 .NET 代码重构学习笔记系列
代码的坏味道 | 一般重构方法 | 使用模式重构 |
重复代码 | 提炼方法 提取类 方法上移 替换算法 链构造方法 | 构造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统一接口 |