Ява - как создать колоду карт (с акцентом на статические массивы)? - 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;
}
}
}