为什么我的程序总是处理重复的卡片？ (JAVA)

Question

因此，对于家庭作业任务，我必须编写一个程序来处理 N（命令行输入）数量的扑克牌（5 张牌）。我有一个 for 循环来检查卡片是否已被使用，但仍然打印了重复的卡片。任何帮助将不胜感激。

检查重复项的代码

do {///check for dupes
                    bUsed = false;
                    randS = (int) (Math.random() * 4);
                    randV = (int) (Math.random() * 13);
                   
                    for (int k = 0; k < 52; k++) {
                    value[randV]);
                        if ((used[k] == (suit[randS] + value[randV])) && (used[k] != null)) {

                            bUsed = true;
                        }
                    }


                } while (bUsed);///end check

完整代码

public class Deal {
public static void main(String[] args) {
    String[] suit = {"Hearts", "Diamonds", "Spades", "Clubs"};
    String[] value = {"A", "2", "3", "4", "5", "6", "7", "8", "9", "10", "J", "Q", "K"};

    String[] used = new String[52];
    boolean bUsed = false;
    int usedC = 0;

    int N = Integer.parseInt(args[0]);
    // int N = 4;

    if (N <= 10) {
        /////////check for max
        for (int i = 0; i < N; i++) {/////////print N amount of decks
            System.out.println("Deck " + (i + 1));
            for (int j = 0; j < 5; j++) {/////////print 5 cards

                int randV = 0;
                int randS = 0;

                do {///check for dupes
                    bUsed = false;
                    randS = (int) (Math.random() * 4);
                    randV = (int) (Math.random() * 13);
                    
                    for (int k = 0; k < 52; k++) {
                       
                        if ((used[k] == (suit[randS] + value[randV])) && (used[k] != null)) {

                            bUsed = true;
                        }
                    }


                } while (bUsed);///end check

                used[usedC] = suit[randS] + value[randV];
                System.out.println(suit[randS] + " " + value[randV]);

                usedC++;
            }////end print 5 cards

            System.out.println();
        }///end print amount of decks
    } else {
        System.out.println("Too many decks requested");
    }
}

}

输出示例

java Deal.java 5 甲板 1 梅花 6 梅花 10 梅花 J 红心 Q 黑桃 J

牌组 2 红心 4 红心 7 红心 A 梅花 J 钻石 3

甲板 3 黑桃 K 方块 3 方块 Q 方块 4 梅花 2

牌组 4 黑桃 7 方块 7 方块 3 梅花 A 方块 9

牌组 5 梅花 2 黑桃 4 方块 4 方块 3 黑桃 7

Answer 1

您正在使用 == 而不是 .equals() 来比较字符串。

这是有问题的行：

if ((used[k] == (suit[randS] + value[randV])) && (used[k] != null))

首先，您需要翻转空检查和值检查，以防止 NullPointerException，因为 .equals() 是一种方法调用（对于运算符 == 这并不重要）。然后使用该方法进行比较，例如：

if ((used[k] != null) && used[k].equals(suit[randS] + value[randV]))

除此之外，如果您使用 List 代替 used 而不是数组，您的生活会变得更加轻松，因为然后你可以简单地使用 .contains 方法而不是你自己的循环 - 但我不知道你的作业是否允许这样做。

为了使程序更快一点，您可以在 if 块内的末尾添加一个 break; 语句，因为一旦发现重复项，您就可以中断 for 循环。