Mutator方法问题及骰子求和功能异常技术咨询

看起来你的骰子求和Mutator方法踩了两个常见的逻辑坑，我来帮你拆解问题并给出具体的修复方案：

一、为什么会累加随机数而非当前骰子点数？ #

大概率是你的求和方法在计算时重新生成了随机点数，而不是读取dieList中存储的当前值。比如你可能写了类似这样的错误代码：

public void addToTotal() {
    int sum = 0;
    for (int i = 0; i < 5; i++) {
        // 错误：每次求和都重新生成随机数，忽略dieList里的现有值
        sum += (int)(Math.random() * 6) + 1;
    }
    total += sum;
}

修复方案： #

直接遍历dieList，取出每个骰子的当前点数进行累加：

public void addToTotal() {
    int sum = 0;
    for (Integer currentDieValue : dieList) {
        // 正确：读取dieList中已存储的骰子点数
        sum += currentDieValue;
    }
    total += sum;
}

二、为什么未掷骰子时点击按钮仍会生成数值？ #

这是因为你的求和方法没有做前置校验，不管骰子有没有被掷过都会执行累加逻辑。我们可以通过两种方式解决：

方式1：用状态标记控制 #

添加一个布尔标记（比如hasRolled），用来记录是否已经完成掷骰子操作：

private boolean hasRolled = false; // 初始状态为未掷骰子

// 掷骰子的方法
public void rollDice() {
    // 遍历dieList，为每个骰子赋值1-6的随机数
    for (int i = 0; i < dieList.size(); i++) {
        dieList.set(i, (int)(Math.random() * 6) + 1);
    }
    hasRolled = true; // 掷完后标记为已掷骰子
    // 这里同步更新GUI显示骰子点数
}

// 求和按钮的点击事件处理
sumButton.addActionListener(e -> {
    if (hasRolled) {
        addToTotal(); // 只有掷过骰子才执行累加
        nextRound(); // 进入下一轮
        hasRolled = false; // 重置标记，下一轮需要重新掷骰子
    } else {
        // 提示用户先掷骰子，比如弹出对话框
        JOptionPane.showMessageDialog(null, "请先掷骰子再求和！");
    }
});

方式2：直接校验dieList的有效性 #

如果不想用标记，也可以通过检查dieList中的数值是否在骰子的合法范围（1-6）内来判断是否已掷骰子：

private boolean areDiceValid() {
    if (dieList == null || dieList.isEmpty()) return false;
    for (int value : dieList) {
        // 骰子点数不在1-6范围内，说明未被正确掷过
        if (value < 1 || value > 6) {
            return false;
        }
    }
    return true;
}

// 修改求和方法，先做校验
public void addToTotal() {
    if (!areDiceValid()) {
        // 未掷骰子时直接返回，不执行累加
        JOptionPane.showMessageDialog(null, "请先掷骰子！");
        return;
    }
    // 正常求和逻辑...
    int sum = 0;
    for (Integer currentDieValue : dieList) {
        sum += currentDieValue;
    }
    total += sum;
}

额外调试建议 #

• 在求和方法里打印dieList的内容，比如System.out.println("当前骰子点数：" + dieList);，这样能快速确认dieList里的数值是否是你预期的当前点数，定位问题根源。
• 确保你的rollDice方法是在修改dieList中的现有元素，而不是重新创建一个新的ArrayList覆盖原有的dieList（这会导致求和时读取的是旧列表）。

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