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