Bad Smell (代码的坏味道)

sourcemaking

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

Duplicated Code

重复代码

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

Long Method

函数长

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

Large Class

类大

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

Long Parameter List

参数多

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

Divergent Change

万能类

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

Shotgun Surgery

天女散花的逻辑

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

Feature Envy

红杏出墙的函数

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

Data Clumps

数据团

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

Primitive Obsession

偏心基本类型

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

Switch Statements

switch语句过多

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

Parallel Inheritance Hierarchies

平行继承

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

Lazy Class

冗赘类

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

Speculative Generality

夸夸其谈将来性

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

Temporary Field

临时字段

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

Message Chains

消息链

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

Middle Man

中介

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

Inappropriate Intimacy

太亲密

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

Alternative Classes with Different Interfaces

类似的类

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

Incomplete Library Class

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

[https://sourcemaking.com/data-class]()

纯数据类

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

Refused Bequest

继承过多

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

Comments

太多注释

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

参考资料

• 22种代码的坏味道，一句话归纳
• Code Smells