代码的味道--烟熏鲱鱼3

烟熏鲱鱼（臭臭）

指那些已经被添加了太多属性或者方法的，难以维护的的函数或类。通常这种情况不是突然出现的，而是随着代码进化而不断积累，特别是在没有人专门维护代码的时候。

对原始类型的迷恋

原文链接

症状

• 在简单的任务中（比如currency,ranges,电话号码的字符串等等），使用原始类型替换小的对象。
• 使用静态变量保存信息（比如：使用USER_ADMIN_ROLE = 1来标识用户是否有管理权限)
• 在数据数组中使用静态变量作为字段名

原因

就像其他的坏味道一样，对原始类型的迷恋产生自不靠谱的言论。“就是一个字符串数据而已”。添加一个原始类型的数据要比创建一个新的 class 要简单的多，对吧？所以一个新的属性被添加了，然后类似的情况一再发生，类慢慢变得又臭又长。

解决办法

类中包含有独特行为和关联数据的属性

image

创建新类，将该属性及相关的行为和数据从用来的类中抽离

image

你经常从同一个对象中获取数据，然后传递给某个方法

int low = daysTempRange().getLow();
int high = daysTempRange().getHigh();
boolean withinPlan = plan.withinRange(low, high);

以传递整个对象的的方式替代之前的方式

boolean withinPlan = plan.withinRange(daysTempRange());

多个方法接受相同的一组参数

image

使用这些参数构建新的对象

image

类中经常会包含类型代码。这些类型的值不会出现在条件变短中，也不会影响程序的行为

image

创建一个新类，使用这个类来替换之前那些类型代码

image

还有一些类型编码会参与条件判断，从而影响程序行为

image

为每个编码类型创建一个子类。然后将相关的行为从原来的类中分离出来。软后用多态替换之前的条件判断。

image

如果没办法使用子类进行拆分

image

用一个状态对象替换类型编码，根据类型编码的不同，将不同的状态对象插入到对象中

image

收益

• 用对象替代原始类型，使代码变得更加的灵活
• 代码被更好的组织，变得更容易理解。对特定数据的处理集中到了一个地方，而不是分散在代码各处。再也不用考猜测来判断那些常量为什么出现在数组中。
• 容易发现重复的代码