我无法弄清楚为什么抛出ArrayIndexOutOfBoundsException

Question

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

线程"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());
  }
 }
} 

Answer 1

看看你的嵌套循环:

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

你为循环的每次迭代增加了a 很多次while ...这意味着你很容易a在循环中得到52,这时你会得到一个异常.你在其他方法中也遇到了同样的问题,例如giveSuit和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对象呢？只需将值/ suit/name传递给构造函数,并将它们存储在final字段中.

(另外,我建议你解决您的名称cardDeck类遵循Java的命名约定,当你有哪些是特定实例的状态不是逻辑部分领域-比如你的values,suits和names域-你让他们静.)