代码坏味道 - OO 滥用
2016-03-04 13:31 yoogo 阅读(571) 评论(0) 编辑 收藏 举报OO 滥用
switch 语句
症状：
复杂的 switch 或 一系列的if
成因：
switch/case 语句在 OO 的代码中一般少见，一旦出现往往会在多个地方重复。 据经验，遇见 switch/case , 往往意味着应该考虑多态了。
治疗：
条件与属性相关的：抽子类或使用策略/状态模式
条件与参数相关的：把case 分解为多个具体方法
null值条件：考虑 NullObject
收益：
提高代码的组织性
什么时候忽略：
工厂模式会使用 if/else 生成不同的产品。
case 的行为太简单
临时字段
症状：
某个字段有时需要，有时不需要-空置不用
成因：
某个算法需要很多数据，为了避免参数太多，把它们提取成了字段。这些字段只有在特定算法时才有用。
治疗：
把这些字段和其操作抽取到单独的类中。
与Null Object 联合起来使用。
收益：
更清晰的类划分
拒绝继承
症状：
仅仅部分属性与方法继承自父类，其他的则与父类不一致。 违反了李氏替换原则。
成因：
仅仅为了重用部分代码而采用继承
治疗：
采用委托而不是继承。
如果采用继承，则把相同的部分代码抽一个公共基类。
收益：
代码组织更好
功能重叠
症状：
两个类都有提供同一功能的方法。
成因：
重复发明轮子，不知其已存在。
治疗：
收益：
去重
