【转载】关于重构，代码的坏味道，应该重构的代码

转载地址:http://my.oschina.net/u/572987/blog/94140


1.重复的代码：

重复代码在同一个类中的不同方法中，则直接提炼为一个方法

如果重复代码在两个互为兄弟的子类中，则将重复的代码提到父类中

如果代码类似，则将相同部分构成单独函数，或者用 Template Method 设计模式

重复代码出现在不相干的类中，则将代码提炼成函数或者放在独立的类中


2.过长的函数：


降低了可读性，应该将独立的功能提炼成新函数


3. 过大类

使得责任不清晰，容易造成重复代码，混乱，应该将过大类的功能拆分成多个功能单一的小类


4.过长的参数列

过长的参数列难以理解，而且容易传错参数。应该将参数列表用参数对象替换

5.发散式变化：

一个类由于不同的原因而被修改。应该将类拆分成多个，每个类只因为一种变化而修改。

6.霰弹式修改：

与发散式变化相反，遇到变化时需要修改许多不同的类。应该将类似的功能放到一个类中


7.依恋情结：

函数对某个类的兴趣高过对自己所处的类，通常是为了取其他类中的数据。应该将函数部分功能移到它感兴趣的类中


8.数据泥团：

在多个地方看到相同的数据项。例如：

      多个类中相同的变量，多个函数中相同的参数列表，并且这些数据总是一起出现。应该将这些数据项放到独立的类中


9.基本类别偏执：

对象技术的新手通常不原因在小任务上运用小对象，比如结合数值和币别的money class，等，应该Replace Data Value with Object。


10.分支语句：

大量的分支、条件语句导致过长的函数，并且可读性差。应将它变成子类或者使用 State和 Strategy模式


11.平行继承体系

是霰弹式修改的特殊情况。一般是当你为某个类增加了一个子类，必须也为另一个类相应的增加一个子类。如果你发现某个继承体系的class名称前缀和另一个继承体系的class名称前缀完全相同。变素有问题了。 应该让一个继承体系的实体（instance）指涉（参考，引用，refer to）另一个继承体系的实体。


12 冗赘类（lazy class）

几乎没有用的组建 应该进行inline class


13.夸夸其谈未来性

现在用不到，觉得未来可以用到的代码，要警惕。应该将用不上的代码去掉


14.过度耦合的消息链

一个对象请求另一个对象，后者又请求另外的对象，然后继续。。。。，形成耦合的消息链。应该公布委托对象供调用


15 纯粹的数据类

  将数据类中数据以Public方式公布，没对数据访问进行保护。应该 将数据封装起来，提供Get/Set方法。

16 过多的注释

代码有着长长的注释，但注释之所以多是因为代码很糟糕。先重构代码，再写上必要的注释

17 令人迷惑的暂时值域

如某个instance变量仅为某特定情况而设，在变量未被使用的情况下猜测当初设置目的非常困难。应该建立一个新的class，把所有和这个变量相关的代码都放到里面。