Ява - как создать колоду карт (с акцентом на статические массивы)? - AP Computer Science Project

Это мой первый вопрос. Просто чтобы уточнить, я проверил, есть ли какие-либо вопросы, которые могли бы мне помочь, прежде чем задавать это. Заранее извиняюсь, если я что-то делаю неправильно, я новичок.

В любом случае, для моего класса AP CS я должен сделать колоду карт a и распечатать ее в текстовом окне. Мне кажется, я очень близок к завершению. Как говорится в названии, в общем, как создать колоду карт с акцентом на статические массивы? Но кроме того (после ошибки I спросите о том, что ниже решено), глядя на код, который я создал до сих пор, это правильный способ сделать это?

вот данный код (это означает, что это не может быть изменено):

public class Card
{
   private String suit;
   private String rank;
   private int value;

   public Card(String s, String r, int v)
   {
      suit = s;
      rank = r;
      value = v;
   }

   public String getSuit()       { return suit; }
   public String getRank()       { return rank; }
   public int getValue()         { return value; }

   public void setSuit(String s) { suit = s; }
   public void setRank(String r) { rank = r; }
   public void setValue(int v)   { value = v; } 

   public String toString()
   {
      return "[" + suit + ", " + rank + ", " + value + "]";
   }
}

и вот то, что я закодировал до сих пор:

public class Lab11bst
{
    public static void main(String[] args)
    {
      Deck deck = new Deck();
      System.out.println(deck);
    }
}


class Deck
{
   private int numberOfCards;
   private Card [] cards;
   private String [] suits = {"Clubs","Diamonds","Hearts","Spades"};
   private String rank;
   private int value;


   public Deck() // This creates a deck of 52 playing cards.
   {
      numberOfCards = 52;
      cards = new Card[52];
      for ( int suit = 0; suit <= 3; suit++ )
      {
         String [] ranks = {"Two","Three","Four","Five","Six","Seven","Eight","Nine","Ten"};
         for ( int rank = 1; rank <= 13; rank++ )
         {
            if (rank == 1)
            {
               this.rank = "Ace";
            }
            else if (rank == 11)
            {
               this.rank = "Jack";
            }
            else if (rank == 12)
            {
               this.rank = "Queen";
            }
            else if (rank == 13)
            {
               this.rank = "King";
            }
            else
            {
               this.rank = "" + ranks[rank];
            }
            for ( int value = 1; value <= 10; value++ )
            {
               if (this.rank == "Ace")
               {
                  value = 1;
               }
               else if (this.rank == "Jack")
               {
                  value = 10;
               }
               else if (this.rank == "Queen")
               {
                  value = 10;
               }
               else if (this.rank == "King")
               {
                  value = 10;
               }
               else
               {
                  this.value = value;
               }
               cards [numberOfCards] = new Card(suits[suit],this.rank,value);
               numberOfCards ++;
            }
         }
      }
   }
}

глядя на мой код, я уверен, что есть лучший способ избавиться от всех этих операторов if и сделать все это более коротким и кратким. Мой вид-третий вопрос (который может быть необходим / может помочь решить раньше отвечая на два основных вопроса), как исправить эту ошибку при запуске программы?:

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 52
    at Deck.<init>(Lab11bst.java:89)
    at Lab11bst.main(Lab11bst.java:5)

большое спасибо заранее.

5 ответов


Я, к сожалению, нашел несколько логических ошибок, особенно с numberOfCards int, который используется в качестве индекса для cards массив. В результате я оптимизировал и исправил ваш class Deck, он работает с основным методом и class Card вы создали:

class Deck {
    private int numberOfCards = 52;
    private Card[] cards;

    private String[] suits = { "Clubs", "Diamonds", "Hearts", "Spades" };
    String[] ranks = { "Ace", "Two", "Three", "Four", "Five", "Six", "Seven", "Eight", "Nine", "Ten", "Jack", "Queen",
            "King" };
    int[] values = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 10, 10, 10 };

    private String rank;
    private int value;

    public Deck() // This creates a deck of 52 playing cards.
    {
        cards = new Card[numberOfCards];
        int currentCardId = 0;
        for (int suit = 0; suit <= 3; suit++) {
            for (int rank = 0; rank <= 12; rank++) {
                this.rank = "" + ranks[rank];
                this.value = values[rank];

                cards[currentCardId] = new Card(suits[suit], this.rank, value);
                System.out.println(cards[currentCardId].toString()); //print out the cards .toString() as it was added to the array
                currentCardId++;
            }
        }
    }
}

он заполняет Card[] cards, не стесняйтесь проверить его с помощью геттера и systemout. Он очень адаптивен к cardnaming и valuesetting, так как вы можете заполнить соответствующие массивы любой информацией, которую хотите. Если вы хотите дальнейшие объяснения, дайте мне знать.


первая проблема, как указано CodeMatrix-неправильная начальная точка, вы должны начать индекс массива с 0. С другой стороны-вы увеличиваете индекс массива элементов 52 4*13*10 (3 петли) раз - это 520 приращений. Вот почему вы всегда будете выпрыгивать за пределы досягаемости.


это довольно просто. У тебя переменная numberOfCards что 52 в начале. Если вы проходите через цикл for, вы пытаетесь получить индекс 52 из cards массив. Так что будет бросать ваш ArrayIndexOutOfBoundsException.

первым элементом массива является а не .

вместо 52 вы должны начать с 51, чтобы вы не получили эту ошибку. Есть еще одна вещь, если вы увеличиваете numberOfCards переменная все равно бросит это исключение.

чтобы распечатать каждую карту после создания, я хотел бы внести некоторые изменения:

...
Card createdCard  = new Card(suits[suit], this.rank, value);
System.out.println(createdCard); //you can also do System.out.println(createdCard.toString());
cards[numberOfCards] = createdCard;
...

чтобы распечатать все созданные карты после их создания, вы можете сделать следующее:

for(Card card : cards) {
    System.out.println(card);
}

System.out.println() вызовет overriden toString() метод Card класс


вы установили "numberOfCards = 52;" при инициализации колоды. После этого, когда вам нужна новая карта, вы вызываете numberOfCards++ как индекс массива. Но массив только начальный с 52. Итак, вы сталкиваетесь с этой проблемой.


спасибо всем за ответы, они действительно помогли реализовать некоторые понятия для меня. Хотел бы я принять больше одного ответа на свой вопрос. Вот полная программа, с которой я закончил (с добавленным методом "перетасовки", который я оставил ранее для простоты).

public class Lab11bst
{
    public static void main(String[] args)
    {
      Deck deck = new Deck();
    }
}

class Deck {
   private String[] suits = { "Clubs", "Diamonds", "Hearts", "Spades" };
   String[] ranks = { "Ace", "Two", "Three", "Four", "Five", "Six", "Seven", "Eight", "Nine", "Ten", "Jack", "Queen", "King" };
   int[] values = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 10, 10, 10 };

   private int numberOfCards = 52; // Variable name "size" was changed to "numberOfCards" to clarify self identifier further for programmer
   private Card[] cards;
   private String rank;
   private int value;

   public Deck()
   {
      cards = new Card[numberOfCards];
      for (int suit = 0; suit <= 3; suit++) 
      {
         for (int rank = 0; rank <= 12; rank++) 
         {
            for (int values = 0; values <=12; values++)
            this.rank = ranks[rank];
            this.value = values[rank];
            numberOfCards--;
            cards[numberOfCards] = new Card(suits[suit], this.rank, value);
         }
      }

      shuffle(); // Shuffle method call. Comment out to obtain an unshuffled, ordered deck of cards.

      for(Card card : cards) 
      {
         System.out.println(card);
      }
   }

   private void shuffle()
   { 
      for (int k = 1; k < 1000; k++)
      {
         int random1 = (int) (Math.random() * 52);
         int random2 = (int) (Math.random() * 52);
         Card temp = cards[random1];
         cards[random1] = cards[random2];
         cards[random2] = temp;
      }
   }
}