烟熏鲱鱼(臭臭)
指那些已经被添加了太多属性或者方法的,难以维护的的函数或类。通常这种情况不是突然出现的,而是随着代码进化而不断积累,特别是在没有人专门维护代码的时候。
超长参数列表
症状
方法的参数超过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());
多个方法接受相同的一组参数
使用这些参数构建新的对象
收益
- 更可读的代码、更短的代码
- 可以发现之前没有注意到的重复的代码