将整数值分配给ArrayList中的字符串

Question

我正在尝试制作纸牌游戏。 它大多数都能工作，但是我很难弄清楚将整数值分配给数组中的字符串元素的有效方法。 从本质上讲，我有一个ArrayList ，每张卡在标准甲板上，每转一圈抓两张。 即使它们是字符串（黑桃王牌等），我也需要它们以某种方式具有数值，以便可以将两张牌进行比较以确定赢家（杰克击败7位，等等）。 我尝试过尝试，但应该不会。

boolean checkAce = deckList.get(0).startsWith("Ace");
boolean checkTwo = deckList.get(0).startsWith("Two");
boolean checkThree = deckList.get(0).startsWith("Three");
boolean checkFour = deckList.get(0).startsWith("Four");
boolean checkFive = deckList.get(0).startsWith("Five");
boolean checkSix = deckList.get(0).startsWith("Six");
boolean checkSeven = deckList.get(0).startsWith("Seven");
boolean checkEight = deckList.get(0).startsWith("Eight");
boolean checkNine = deckList.get(0).startsWith("Nine");
boolean checkTen = deckList.get(0).startsWith("Ten");
boolean checkJack = deckList.get(0).startsWith("Jack");
boolean checkQueen = deckList.get(0).startsWith("Queen");
boolean checkKing = deckList.get(0).startsWith("King");

boolean checkAceP2 = deckList.get(1).startsWith("Ace");
boolean checkTwoP2 = deckList.get(1).startsWith("Two");
boolean checkThreeP2 = deckList.get(1).startsWith("Three");
boolean checkFourP2 = deckList.get(1).startsWith("Four");
boolean checkFiveP2 = deckList.get(1).startsWith("Five");
boolean checkSixP2 = deckList.get(1).startsWith("Six");
boolean checkSevenP2 = deckList.get(1).startsWith("Seven");
boolean checkEightP2 = deckList.get(1).startsWith("Eight");
boolean checkNineP2 = deckList.get(1).startsWith("Nine");
boolean checkTenP2 = deckList.get(1).startsWith("Ten");
boolean checkJackP2 = deckList.get(1).startsWith("Jack");
boolean checkQueenP2 = deckList.get(1).startsWith("Queen");
boolean checkKingP2 = deckList.get(1).startsWith("King");

int playerOneScore = 0;
int playerTwoScore = 0;

if (checkAce = true) {
    playerOneScore = 14;
}else if (checkTwo = true) {
    playerOneScore = 2;
}else if (checkThree = true) {
    playerOneScore = 3;
}else if (checkFour = true) {
    playerOneScore = 4;
}else if (checkFive = true) {
    playerOneScore = 5;
}else if (checkSix = true) {
    playerOneScore = 6;
}else if (checkSeven = true) {
    playerOneScore = 7;
}else if (checkEight = true) {
    playerOneScore = 8;
}else if (checkNine = true) {
    playerOneScore = 9;
}else if (checkTen = true) {
    playerOneScore = 10;
}else if (checkJack = true) {
    playerOneScore = 11;
}else if (checkQueen = true) {
    playerOneScore = 12;
}else if (checkKing = true) {
    playerOneScore = 13;
}

if (checkAceP2 = true) {
    playerTwoScore = 14;
}else if (checkTwoP2 = true) {
    playerTwoScore = 2;
}else if (checkThreeP2 = true) {
    playerTwoScore = 3;
}else if (checkFourP2 = true) {
    playerTwoScore = 4;
}else if (checkFiveP2 = true) {
    playerTwoScore = 5;
}else if (checkSixP2 = true) {
    playerTwoScore = 6;
}else if (checkSevenP2 = true) {
    playerTwoScore = 7;
}else if (checkEightP2 = true) {
    playerTwoScore = 8;
}else if (checkNineP2 = true) {
    playerTwoScore = 9;
}else if (checkTenP2 = true) {
    playerTwoScore = 10;
}else if (checkJackP2 = true) {
    playerTwoScore = 11;
}else if (checkQueenP2 = true) {
    playerTwoScore = 12;
}else if (checkKingP2 = true) {
    playerTwoScore = 13;
}

if (playerOneScore > playerTwoScore) {
    System.out.println("Player One Wins!");
}else if (playerTwoScore > playerOneScore) {
    System.out.println("Player Two Wins!");
}else if (playerOneScore == playerTwoScore) {
    System.out.println("Tie");
}

其背后的想法是，我执行发牌的每个循环都将检查ArrayList元素，以查看开头是否与面值的字符串匹配。 然后，一条if语句检查看哪一个是正确的，并为每个玩家的得分值分配适当的分数，并进行比较。 出于某种原因，每次我运行checkAce和checkAceP2都评估为true而不管实际上在deckList.get(0)和.get(1)有什么牌。 这导致每次打平。 谁能看到我代码中的漏洞？ 还是有人可以推荐一种更好的方法来实现我的目标？

Answer 1

(checkAce = true)将true分配给checkAce并且（作为副作用） 评估为true。 您需要==或仅布尔值（对于所有if ）。

if (checkAce == true)

或者更短

if (checkAce)

Answer 2

回答

我在寻找一种有效的方法来将整数值分配给数组中的字符串元素时遇到麻烦。

如果可能的话，我将使用枚举而不是字符串，因为每张卡的值将保持不变。 然后，您将拥有一个ArrayList of Cards，而不是拥有一个ArrayList的字符串。

public enum Card {
    ACE(14),
    ONE(1),
    TWO(2),
    ...
    KING(13);

    /** The point value associated to each card. */
    private int pointValue;

    private Card(int pointValue) {
        this.pointValue = pointValue;
    }

    public int getPointValue() {
        return this.pointValue;
    }

}

Answer 3

您的问题是将卡存储为字符串。 卡通常被表示为具有两个属性的对象：等级和西服。

卡片的String表示很容易从Rank and Suit建立。 但是，从字符串中提取等级和花色并不是那么容易，尤其是当您的纸牌游戏变得流行并且需要翻译成其他语言时。 它也比其他方法慢得多。

因此，我要说的是，您的麻烦是您开始做错了。 创建一个带有“等级”字段和“西服”字段的Card类。 这两个领域都可以从枚举中受益：

public enum Rank { TWO, THREE, ... , JACK, QUEEN, KING, ACE }
public enum Suit { SPADES, HEARTS, DIAMONDS, CLUBS }
public class Card {
    public final Rank rank;
    public final Suit suit;
    public Card(Rank rank, Suit suit) {
        this.rank = rank;
        this.suit = suit;
    }
    @Override
    public String toString() {
        // code here
    }
}

或将字段设为private并添加getter方法。

这样，您的代码变得更容易：

if (deckList.get(0).rank == Rank.ACE) {

并且打印卡片（或整个deckList ）仍将显示完整文本，例如toString()方法。