1-Bad Smell (代码的坏味道)

Bad Smell (代码的坏味道)

如果一段代码是不稳定或者有一些潜在问题的，那么代码往往会包含一些明显的痕迹。
正如食物要腐坏之前，经常会发出一些异味一样, 我们管这些痕迹叫做 “代码异味”。
今天让我们一起来熟悉开发中经常出现的22种坏味道情形和解决方法。

1.Duplicated Code(重复代码)

不良影响	解决方法
重复代码,难维护	提取公共函数

2.Long Method(过长的代码)

不良影响	解决方法
函数长, 难理解	拆分成若干函数

3.Large Class(过大类)

不良影响	解决方法
类大, 难理解	拆分成若干类

4.Long Parameter List(过长参数列)

不良影响	解决方法
参数多,难用，难理解	将参数封装成结构或者类

5.Divergent Change（发散式变化）

不良影响	解决方法
万能类,发散试修改,改好多需求，都会动他	拆,将总是一起变化的东西放在一块儿

6.ShotgunSurgery（散弹式修改）

不良影响	解决方法
天女散花的逻辑,散弹式修改,改某个需求的时候，要改很多类	将各个修改点，集中起来，抽象成一个新类

7.Feature Envy（依恋情结）

不良影响	解决方法
红杏出墙的函数,使用了大量其他类的成员	将这个函数挪到那个类里面

8.Data Clumps（数据泥团）

不良影响	解决方法
数据团,常一起出现的一坨数据	他们那么有基情，就在一起吧，给他们一个新的类

9.Primitive Obsession（基本偏执型）

不良影响	解决方法
偏爱基本类型,热衷于使用int,long,String等基本类型	反复出现的一组参数，有关联的多个数组换成类吧

10.Switch Statements（Switch惊悚现身）

不良影响	解决方法
switch语句过多, 难以理解和维护	state/strategy 或者只是简单的多态

11.Parallel Inheritance Hierarchies（平行继承体系）

不良影响	解决方法
平行继承,增加A类的子类ax，B类也需要相应的增加一个bx	应该有一个类是可以去掉继承关系的。

12.Lazy Class（冗余类）

不良影响	解决方法
冗赘类,如果他不干活了，炒掉他吧	把这些不再重要的类里面的逻辑，合并到相关类，删掉旧的

13.SpeculativeGenrerality（夸夸其谈未来性）

不良影响	解决方法
夸夸其谈未来性	删掉

14.Temporary Field（令人迷惑的暂时值域）

不良影响	解决方法
临时字段,仅在特定环境下使用的变量	将这些临时变量集中到一个新类中管理

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

不良影响	解决方法
消息链,过度耦合的才是坏的	拆函数或者移动函数

16.Middle Man（中间转手人）

不良影响	解决方法
中介,大部分都交给中介来处理了	用继承替代委托

17.Inappropriate Intimacy（不正当关系）

不良影响	解决方法
太亲密,两个类彼此使用对方的私有的东西	划清界限拆散，或合并，或改成单项联系

18.Alternative Classes With Different Interfaces（异曲同工的类）

不良影响	解决方法
相似的类，有不同接口	重命名，移动函数，或抽象子类

19.Incomplete Library Class(不完美的程序库类)

不良影响	解决方法
不完善的类库	包一层函数或包成新的类

20.Data Class（纯稚的数据类）

不良影响	解决方法
纯数据类,类很简单，仅有公共成员变量，或简单操作函数	将相关操作封装进去，减少public成员变量

21.Refused Bequest（被拒绝的遗赠）

不良影响	解决方法
继承过多,父类里面方法很多，子类只用有限几个	用代理替代继承关系

22.Comments（过多的注释）

不良影响	解决方法
太多注释,这里指代码太难懂了，不得不用注释解释	避免用注释解释代码，而是说明代码的目的，背景等。好代码会说话