Bad Smell (代码的坏味道)
Posted herelsp
tags:
篇首语：本文由小常识网(cha138.com)小编为大家整理，主要介绍了Bad Smell (代码的坏味道)相关的知识，希望对你有一定的参考价值。
sourcemaking 如果一段代码是不稳定或者有一些潜在问题的，那么代码往往会包含一些明显的痕迹。
正如食物要腐坏之前，经常会发出一些异味一样, 我们管这些痕迹叫做 “代码异味”。
今天让我们一起来熟悉开发中经常出现的22种坏味道情形和解决方法。
重复代码
函数长
类大
参数多| 不良影响 | 解决方法 |
|---|
| 参数多,难用，难理解 | 将参数封装成结构或者类 |
万能类| 不良影响 | 解决方法 |
|---|
| 万能类,发散试修改,改好多需求，都会动他 | 拆,将总是一起变化的东西放在一块儿 |
天女散花的逻辑| 不良影响 | 解决方法 |
|---|
| 天女散花的逻辑,散弹式修改,改某个需求的时候，要改很多类 | 将各个修改点，集中起来，抽象成一个新类 |
红杏出墙的函数| 不良影响 | 解决方法 |
|---|
| 红杏出墙的函数,使用了大量其他类的成员 | 将这个函数挪到那个类里面 |
数据团| 不良影响 | 解决方法 |
|---|
| 数据团,常一起出现的一坨数据 | 他们那么有基情，就在一起吧，给他们一个新的类 |
偏爱基本类型| 不良影响 | 解决方法 |
|---|
| 偏爱基本类型,热衷于使用int,long,String等基本类型 | 反复出现的一组参数，有关联的多个数组换成类吧 |
switch语句过多| 不良影响 | 解决方法 |
|---|
| switch语句过多, 难以理解和维护 | state/strategy 或者只是简单的多态 |
平行继承| 不良影响 | 解决方法 |
|---|
| 平行继承,增加A类的子类ax，B类也需要相应的增加一个bx | 应该有一个类是可以去掉继承关系的。 |
冗赘类| 不良影响 | 解决方法 |
|---|
| 冗赘类,如果他不干活了，炒掉他吧 | 把这些不再重要的类里面的逻辑，合并到相关类，删掉旧的 |
夸夸其谈未来性
临时字段| 不良影响 | 解决方法 |
|---|
| 临时字段,仅在特定环境下使用的变量 | 将这些临时变量集中到一个新类中管理 |
消息链| 不良影响 | 解决方法 |
|---|
| 消息链,过度耦合的才是坏的 | 拆函数或者移动函数 |
中介| 不良影响 | 解决方法 |
|---|
| 中介,大部分都交给中介来处理了 | 用继承替代委托 |
太亲密| 不良影响 | 解决方法 |
|---|
| 太亲密,两个类彼此使用对方的私有的东西 | 划清界限拆散，或合并，或改成单项联系 |
相似的类| 不良影响 | 解决方法 |
|---|
| 相似的类，有不同接口 | 重命名，移动函数，或抽象子类 |
| 不良影响 | 解决方法 |
|---|
| 不完善的类库 | 包一层函数或包成新的类 |
[https://sourcemaking.com/data-class]()
纯数据类| 不良影响 | 解决方法 |
|---|
| 纯数据类,类很简单，仅有公共成员变量，或简单操作函数 | 将相关操作封装进去，减少public成员变量 |
继承过多| 不良影响 | 解决方法 |
|---|
| 继承过多,父类里面方法很多，子类只用有限几个 | 用代理替代继承关系 |
太多注释| 不良影响 | 解决方法 |
|---|
| 太多注释,这里指代码太难懂了，不得不用注释解释 | 避免用注释解释代码，而是说明代码的目的，背景等。好代码会说话 |
参考资料
以上是关于Bad Smell (代码的坏味道)的主要内容，如果未能解决你的问题，请参考以下文章
代码的坏味道
code smell
我的重构识别代码的坏味道
代码的坏味道
22 种代码的坏味道
21种代码的坏“味道”