烟熏鲱鱼（臭臭）

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

超长参数列表

原文链接

症状

方法的参数超过3到4个

原因

多个算法被合并到一个方法之后，往往会产生参数列表过长的问题。这些参数通常用来决定使用哪种算法，以及这种算法如何执行。

超长的参数列表可能会使类之间的耦合变得越来越紧。比如，一个方法中创建某个对象的代码，从这个方法中被迁移到方法的调用者中，然后调用者将创建好的对象作为参数传递给原来的方法。这导致原方法不知道对象之间的关系，同时降低了耦合度。但是，如果这样的情况比较多的话，那么参数列表必然会膨胀。

当参数列表不断膨胀的时候，方法变得难于使用，难于理解。对象可以使用自身的数据，来代替参数的传递，如果自身数据无法满足需求的时候再通过参数传递。

解决办法

在调用方法之前，通过其他对象的方法获得数据，这些将作为方法调用的参数。问题是这些数据可以通过对象的方法获得

int basePrice = quantity * itemPrice;
double seasonDiscount = store.getSeasonalDiscount();
double fees = store.getFees();
double finalPrice = discountedPrice(basePrice, seasonDiscount, fees);

直接将对象作为参数传递，而不是传递对象中某些方法的结果。

int basePrice = quantity * itemPrice;
double finalPrice = discountedPrice(basePrice, store);

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

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

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

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

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

image

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

image

收益

• 更可读的代码、更短的代码
• 可以发现之前没有注意到的重复的代码