Java二十一点计分问题

Question

我在二十一点游戏中的得分有问题。它可以找到正确的分数，但是当用户绘制一张新卡片时，它会错误地添加分数。

例如： 原始手牌是：4 和 5（所以得分 9） 用户抽到 10。 分数不是 19，而是 19+9 或 28。

这是我的代码： 计分方式：

public int getHandValue() {
    boolean ace = false;
    for (int i = 0; i < this.hand.size(); i++) {
        if (this.hand.get(i).getRank().value > 10) {
            points += 10;
        } else if (this.hand.get(i).getRank().value == 1) {
            ace = true;
        } else {
            points += this.hand.get(i).getRank().value;
        }
        if (ace == true && points + 11 <= 21) {
            points += 11;
        }

    }
    return points;
}

玩法：

public void play(Deck deck) {
    boolean isDone = false;
    if (this.getHandValue() > 21){
        System.out.println("You have busted!");
        isDone = true;
        this.lose();
    }
    takeCard(deck.drawCard());
    takeCard(deck.drawCard());
    System.out.println("Here are your cards and your score:");
    System.out.println(this.hand.toString());
    System.out.println("Score: " + getHandValue());
    ListItemInput hitOrPass = new ListItemInput();
    hitOrPass.add("h", "hit");
    hitOrPass.add("p", "pass");
    while (!isDone){
        System.out.println("Hit or pass?");
        hitOrPass.run();
        if (hitOrPass.getKey().equalsIgnoreCase("h")) {
            String result = "";
            this.takeCard(deck.drawCard());
            result += "You hand is now " + this.hand.toString() + "\n";
            result += "Your score is now " + this.getHandValue();
            System.out.println(result);
        } else {
            System.out.println("You have chosen to pass.");
            isDone = true;
        }
    }
}

Answer 1

每次调用你的方法时，你都在手上循环，所以你的积分应该在这样做之前重置。否则分数增加 2 倍 + 手中的额外牌。在你循环你的手之前重置值

public int getHandValue() {
    boolean ace = false;
    points = 0; //<--- reset the point total
    for (int i = 0; i < this.hand.size(); i++) {
        if (this.hand.get(i).getRank().value > 10) {
            points += 10;
        } else if (this.hand.get(i).getRank().value == 1) {
            ace = true;
        } else {
            points += this.hand.get(i).getRank().value;
        }
        if (ace == true && points + 11 <= 21) {
            points += 11;
        }

    }
    return points;

Answer 2

我假设 points 是在此方法之外声明的。

由于您要返回积分，因此最好不要为此使用类范围的变量。你最终会得到这样的意想不到的结果。相反，在方法范围内使用变量，就像这样。

public int getHandValue() {
    boolean ace = false;
    int value = 0;

    for (int i = 0; i < this.hand.size(); i++) {
        if (this.hand.get(i).getRank().value > 10) {
            value += 10;
        } else if (this.hand.get(i).getRank().value == 1) {
            ace = true;
        } else {
            value += this.hand.get(i).getRank().value;
        }
        if (ace == true && points + 11 <= 21) {
            value += 11;
        }
    }

    return value;
}