我收到 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/

10-11 04:03