Question

我有一组变量，这些变量是由古代遗留代码中的一个巨型方法传递的......

public List<type> check (String required, String sales, String report,
 Long passId, Long seatId, String capName, String vCapName, 
 String attName, Long vid) {

        if(required != null) {
            goodA = method(required);
            goodB = methodTwo(required);
            goodC = methodThree(required);
        }
        if(sales != null) {
            goodA = method(sales);
            goodB = methodTwo(sales);
            goodC = methodThree(sales);
        }
        if(report != null) {
            goodA = method(report);
            goodB = methodTwo(report);
            goodC = methodThree(report);
        if(passId != null)
        ... you got the point....


}

传递给check的变量只能是1个有效值，所有其他变量都将变为null。例如检查（＆＃34;是＆＃34;，NULL，NULL，NULL，NULL，NULL ...）要么检查（NULL，NULL，NULL，13212L，NULL，NULL，NULL，NULL）

现在我正在尝试将其重写为不那么重复和干净的东西我想知道是否有人可以就如何做到这一点提供一些想法。

Answer 1

这样的事情怎么样？

List<Object> items = Lists.newArrayList(required, sales, report,
 capName, vCapName, attName);

for(Object item : items) {
    if(item != null){
        methodOne(item);
        methodTwo(item);    
        methodThree(item);
    }
}

如何提高代码质量（主要是重复）

1 个答案: