我收到 PMD 红色违规
避免重新分配诸如“bankRequest”之类的参数
这是我的方法
@Override
public BankDTO loadTariff(BankDTO bankRequest, int[] executionLevels) {
double[] fee = null;
for (int level : executionLevels) {
// Check the tariff availability from execution level one to .....
fee = loadCokaAndBankFee(bankRequest,level);
if (fee != null) { // if fee found reload the bank request with new
// amount
bankRequest = reloadBankRequest(bankRequest, fee);
break; // no need to go for any other level deep level cover //
// here.
} // if tariff not found use the esb provided amounts
}
return bankRequest;
}
有人可以解释一下这段代码有什么问题。如果我忽略它有什么影响。
最佳答案
由于已经给出的原因,您应该避免重新分配变量。无论如何,与其分配新变量,不如考虑在找到正确值时从循环中返回。这也会使“中断”变得多余。
关于java - 避免重新分配参数,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/13507025/