0

我正在尝试制作一个创建一副纸牌的小程序。所有代码都可以编译，但是当我尝试运行测试器类时，它会说：

线程“main”中的异常 java.lang.ArrayIndexOutOfBoundsException: 52
at cardDeck.giveValue(cardDeck.java:23)
at CardTest.main(CardTest.java:6)

我只使用 java 几个星期，所以我不知道问题出在哪里，或者如何解决它。我会很感激任何帮助。
卡类：

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

 public void setValue(int v){
  value = v;
 }
 public int getValue(){
  return value;
 }

 public void setName(String n){
  name = n;
 }
 public String getName(){
  return name;
 }

 public void setSuit(String s){
  suit = s;
 }
 public String getSuit(){
  return suit;
 }

}

卡组类：

public class cardDeck {

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

 public Card[] giveName (Card[] d){
 int j = 0;
 while(j < 52){
  for(String k: names){
   d[j] = new Card();
   d[j].setName(k);
    j++;
   }
  }
  return d;
 }

 public Card[] giveValue(Card[] e){
  int a = 0;
  while(a < 52){
   for(int m: values){
     if((e[a].getName().equals("King")) || (e[a].getName().equals("Ten")) || (e[a].getName().equals("Queen")) || (e[a].getName().equals("Jack"))){
      e[a].setValue(10);
      a++;
     }else if(e[a].getName().equals("Ace")){
      e[a].setValue(11);
      a++;
     }else{
      e[a].setValue(m);
      a++;
     }
    }
   }
   return e;
  }

 public Card[] giveSuit(Card[] f){
  int b = 0;
  while(b < 52){
   for(String l:suits){
    f[b].setSuit(l);
    b++;
    }
   }
   return f;
 } 
}

测试类：

class CardTest {
 public static void main(String[] args){
  Card[] deck = new Card[52];
  cardDeck playingCards = new cardDeck();
  playingCards.giveName(deck);
  playingCards.giveValue(deck);
  playingCards.giveSuit(deck);




  for(int q = 0; q < 52; q++){
   System.out.println("A "+deck[q].getName()+ " of " +deck[q].getSuit()+ " " +deck[q].getValue());
  }
 }
}
4

3 回答 3

5

看看你的嵌套循环：

while(a < 52) {
  for(int m: values) {
    // Various things which increment a
  }
}

对于循环的每次迭代，您都会增加a 很多次while......这意味着您很容易a在循环期间以 52 结束，此时您会遇到异常。您在其他方法中也遇到了同样的问题，例如giveSuitand giveName。其他方法几乎可以工作，因为您的其他数组的长度恰好分为 52 - 而values只有 10 个元素，所以在循环的 5 次迭代之后，while您将继续a = 50......所以在嵌套循环的 3 次迭代之后，您会爆炸的。但是为什么你有嵌套循环呢？

虽然可以修复您现有的giveValue方法，但将其重写为：

public Card[] giveValue(Card[] e) {
  for (int i = 0; i < 52; i++) {
    int rawValue = (i % 13) + 2; // Value 2-14
    if (rawValue == 14) {
      e[i].setValue(11); // The ace
    } else if (rawValue >= 11 && rawValue <= 13) {
      e[i].setValue(10); // Jack to King
    } else {
      e[i].setValue(rawValue);
    }
  }
  return e;
}

我实际上建议重新设计 - 我建议你让你的Card类不可变。想想现实生活——一张卡片在创建后就永远不会改变，那么为什么你的一个Card对象应该改变呢？只需将值/西装/名称传递给构造函数，并将它们存储在final字段中。

（此外，我建议您修改cardDeck类的名称以遵循 Java 命名约定，并且当您的字段在逻辑上不属于特定实例的状态时 - 例如您的values和suits字段names- 您将它们设为静态。）

于 2013-09-28T14:56:15.030 回答
0

关于循环结构的其他答案是正确的，但你真正的问题是：

int [] values = {2, 3, 4, 5, 6, 7, 8, 9, 10, 11};

...

int a = 0;
 while(a < 52) {
  for(int m: values) {
    // stuff
    a++;
  }
}

所以发生的是第一个循环：

Loop # -> a
0 -> 0 to 10
1 -> 11 to 20
2 -> 21 to 30
3 -> 31 to 40
4 -> 41 to 50
5 -> 51 to 60

所以循环 5 51 小于 52 所以它继续设置值循环另外 10 次。

作为一个简单的修复，我会将 13 个值放入值中。

于 2013-09-28T17:09:30.883 回答
0

用这样的 for 重写你的 while 循环：

for(int a=0; a < 52; a++) {
   //
}

并避免a++在多个地方。

于 2013-09-28T14:57:07.620 回答