最近的任务是维护一套很久远的QT代码,由于是很久之前的代码,理解起来就比较麻烦,而且写这套代码的人大多是写C的来写的,夹杂了一些面向过程的写法,整个继承体系就比较混乱,刚好就想起之前老师介绍的一本书,《重构-改善既有代码的设计》,也想借此机会把这套代码重构一下,深入的去学习一下面向对象,学习一下怎样的代码才是好的代码,这本书最经典的一句话就是“闻到代码的坏味道”,那究竟什么是“代码的坏味道”呢?稍微整理了一下

1.Duplicated Code  重复代码

2.Long Method  过长函数

3.Large Class  过大的类

4.Long Parameter List 过长参数列

5.Divergent Change  发散式变化

6.Shotgun Surgery 散弹式变化

每遇到某种变化,都必须在许多不同的类内做出许多小修改

7.Feature Envy 依恋情节

函数对某个类的兴趣高过对自己所处类的兴趣

8.Data Clumps 数据泥团

两个类中相同的字段,函数签名中相同的参数

9.Primitive Obsession 基本类型偏执

将原本单独存在的数据值替换成对象

10.Switch Statement

少用switch 语句,考虑以多态替换

11.Speculative Generality 夸夸其谈未来性

12.Parall Inheritance Hierarchies 平行继承体系

某个继承体系的类名称前缀和另一个继承体系的类名称前缀完全相同

13.Lazy Class 冗赘类

一个类的所得不值其身价,就应该消失

14.Temporary Field 令人迷惑的暂时字段

其内某个实例变量仅为某种特定情况而设

15.Message Chains 过度耦合的消息链

先观察消息链最终得到的对象是用来干什么的,看看能否以Extract Method,把使用该对象的代码提炼到一个独立函数中,再运用Move Method 把这个函数推入消息链

16.Middle Man 中间人

过度使用委托

17.Inappropriate Intimacy 狎昵关系

两个类过于亲密

18.Alternative Classes with Different Interface 异曲同工的类

两个函数做同一件事情,却有不同的签名

19.Incomplete Library Class 不完美的库类

如果想修改库类的一两个函数,可以运用Introduce Foreign Method

20.Data Class 纯稚的数据类

作为一个起点很好,但是要像成熟的对象那样参与整个系统的工作,就要承担一定的责任

21.Refused Bequest 被拒绝的遗赠

22.Comments 过多的注释

posted on 2017-08-01 09:08  Honic  阅读(395)  评论(0编辑  收藏  举报