甲板交易Java

Question

我正在尝试为类编写代码，该类执行以下操作：

• 询问要使用多少甲板
• 每次按enter键时，程序都会从卡片组中生成一张未使用的卡片
• 当没有更多的卡可以交易时，程序会通知用户
• 程序允许用户再次播放
• 方法被广泛使用

到目前为止，我的代码如下所示：

import java.util.*;
public class IA3 {

@SuppressWarnings({ })
public static void play () {
}
@SuppressWarnings("resource")
public static void main(String[] args) {
    boolean draw = true;
    boolean pa = true;
    Scanner console = new Scanner(System.in);
    // TODO Auto-generated method stub
    System.out.println("Hello! Please input the number of decks you would like to use:");
    int decks = console.nextInt();

    int t = decks * 52;
    System.out.println("Your total amount of cards to use are " +t);
    int a = 0;
    do {
        int[] deck = new int[t];
        //declaration of suits and ranks
        String[] suits = {"Spades", "Hearts", "Diamonds", "Clubs"};
        String[] ranks = {"Ace", "2", "3", "4", "5", "6", "7", "8", "9", "10", "Jack", "Queen", "King"};

        //Initializing of the cards
        for (int i = 0; i < t; i++) {
        deck[i] = i;
                }

            System.out.println("Press enter to draw a random card : "); //allows player to prompt the system for the next card

            String input = console.nextLine();
            a++;
            if (input.equals("")) {
                // Shuffles the cards
                for (int i = 0; i < 1; i++) {
                  int index = (int)(Math.random() * t);
                  int temp = deck[i];
                  deck[i] = deck[index];
                  deck[index] = temp;
                }

                      for (int i = 0; i < 1; i++) {
                      String suit = suits[deck[i] / 13];
                      String rank = ranks[deck[i] % 13];
                      System.out.println(rank + " of " + suit);
                      }
             if (a>t) {
                 System.out.println ("No more cards available");
                 break;
             }
                      System.out.println("Draw another card? (Yes or No) : ");
                      String again = console.nextLine();
                      if (again.equals("Yes")) {
                          continue;
                      }
                      if (again.equals("No")) {
                          draw = false;
                          System.out.println("Game over!");
                      }
                        System.out.println("Play again? (Yes or No) : ");
                          String plag = console.nextLine();
                          if (plag.equals("Yes")) {
                              continue;
                          }
                          if (plag.equals("No")) {
                              pa = false;
                              System.out.println("Thank you for playing!");
                              break;
                          } while (pa == true);
                  }
        } while (draw == true);
    }
}

当我尝试运行超过1套牌时，问题就出现了，有时会立即失败，而有时运行可能4个交易然后失败。我似乎也无法让它再次运行；每当我输入“是”而不是再次播放时，它就会终止。如您所见，我在那里没有方法（我在方法上迷失了方向）。任何帮助都将不胜感激，谢谢！

Answer 1

现在还不能发表评论，所以这里有一些想法可以让这段代码更容易理解，特别是当你的问题需要“任何帮助”时

删除如下所示的循环：

for (int i = 0; i < 1; i++) {
 // code
}

上面的for循环将i设置为0，然后执行内部代码，然后增加i（因此i现在等于1），然后检查i是否小于1（这不是因为它现在等于1），因此它不会再次执行内部代码。换句话说，这种类型的for循环是不必要的，因为它只能执行其内部代码一次。应移除周围的for循环，并保持内部代码完整。

将数据组存储为ArrayList可能更简单，而不是数组。这样做的好处是，您可以利用.remove（int）和.size（）等操作，而不必担心洗牌（您可以随机删除一个条目并打印出来）。

希望这能给你一些正确方向的建议。继续努力！

这是一个关于创建方法的有用教程，我通过谷歌搜索“java方法”找到了它。http://www.tutorialspoint.com/java/java_methods.htm

Answer 2

您应该发布完整的错误消息。没有它很难猜测发生了什么。我看到至少一个数组索引可能超出边界的地方：

String suit = suits[deck[i] / 13];

如果您有多个牌组，则此除法的结果将大于3。您可以使用以下方法进行修复：

String suit = suits[(deck[i] / 13) % 4];

这样可以确保只使用索引为[0,1,2,3]的元素。