Java新手求助：Field Goal命中率计算始终输出0问题排查

首先看构造方法里的核心逻辑错误：
在if(newnumAttempt < newnumMade)分支中，你先将numMade赋值为newnumAttempt，紧接着又把numAttempt赋值为numMade，这会导致numAttempt被覆盖成newnumAttempt的值。如果输入的newnumAttempt是0，最终numMade和numAttempt都会变成0，后续计算命中率时，因为numAttempt > 0不成立，直接返回默认的fgPercentage（0.0）。同时这个分支的逻辑本身不合理——当命中数超过出手数时，正确的修正应该是把命中数调整为出手数，出手数保持原输入值，而非将两者设为同一个数。

其次，构造方法存在冗余代码：
开头已经给numAttempt和numMade赋过值，else分支里的重复赋值完全没必要，可直接删除。

最后，命中率计算方法的潜在问题：
如果numAttempt为0，方法直接返回fgPercentage，建议在类中显式初始化该变量（比如private double fgPercentage = 0.0;），避免依赖Java默认初始化的隐式行为。

修正后的代码 #

构造方法 #

public Player(String newName, int newnumMade, int newnumAttempt) {
    name = newName;
    // 修正数据合理性：命中数不能超过出手数
    if (newnumMade > newnumAttempt) {
        numMade = newnumAttempt;
    } else {
        numMade = newnumMade;
    }
    numAttempt = newnumAttempt;
}

命中率计算方法 #

// 类成员变量显式初始化
private double fgPercentage = 0.0;

public double getfgPercentage() {
    if (numAttempt > 0) {
        double first = (double) numMade / numAttempt;
        fgPercentage = first * 100;
    }
    return fgPercentage;
}

测试时注意传入合理的参数，比如newnumMade=3、newnumAttempt=10，此时命中率应返回30.0。

内容的提问来源于stack exchange，提问作者marior21