尝试从列表中获取元素时出现 ConcurrentModificationException
ConcurrentModificationException when attempting to get an element from a List
所以我正在尝试创建一个简单的纸牌游戏,为此我创建了一个玩家 class,其中包含一个名字和玩家的手。
public class Player {
private List<Card> hand;
private String name;
public Player(String name){
this.name = name;
}
public void drawHand(List<Card> hand){
System.out.println(this.name + " draws a hand.");
this.hand = hand;
}
public String throwingError(){
Card c = this.hand.get(0);
return c.getRank();
}
}
这是引擎 运行 游戏
public static void main(String[] args) {
int numberOfPlayers = Integer.parseInt(args[0]);
players = new Player[numberOfPlayers];
deck = createDeck();
for(int i = 0; i < numberOfPlayers; i++){
players[i] = new Player("Player" + (i+1));
}
for(Player player : players){
List<Card> hand = deck.dealHand();
player.drawHand(hand);
}
String cardRank = players[turn - 1].throwingError();
}
代码在到达主函数的最后一行时抛出 ConcurrentModificationException,这让我感到困惑,因为没有元素被迭代或从 Player class.[=14 的手牌字段中删除=]
getRank() 方法是另一个 class Card 的一部分。很简单getter
public String getRank() {
return rank;
}
这里是堆栈跟踪:
Exception in thread "main" java.util.ConcurrentModificationException
at java.util.ArrayList$SubList.checkForComodification(ArrayList.java:1231)
at java.util.ArrayList$SubList.listIterator(ArrayList.java:1091)
at java.util.AbstractList.listIterator(AbstractList.java:299)
at java.util.ArrayList$SubList.iterator(ArrayList.java:1087)
at java.util.AbstractCollection.toString(AbstractCollection.java:454)
at java.lang.String.valueOf(String.java:2981)
at java.io.PrintStream.println(PrintStream.java:821)
at Player.getRank(Player.java:22)
at GameEngine.nextTurn(GameEngine.java:32)
at GameEngine.main(GameEngine.java:27)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:483)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147)
在此代码中:
for(Player player : players){
List<Card> hand = deck.dealHand();
player.drawHand(hand);
}
您的 dealHand()
方法是在每次迭代中 return 创建一个子列表,同时添加到后备列表。这是支持列表的结构修改,正如 sublist()
的 Javadoc 所说,子列表方法的行为在支持列表的结构修改后未定义。因此,子列表上的 get() 操作抛出。
要么一次创建整个列表,然后将其分成子列表,要么 return 每个子列表的副本。
所以我正在尝试创建一个简单的纸牌游戏,为此我创建了一个玩家 class,其中包含一个名字和玩家的手。
public class Player {
private List<Card> hand;
private String name;
public Player(String name){
this.name = name;
}
public void drawHand(List<Card> hand){
System.out.println(this.name + " draws a hand.");
this.hand = hand;
}
public String throwingError(){
Card c = this.hand.get(0);
return c.getRank();
}
}
这是引擎 运行 游戏
public static void main(String[] args) {
int numberOfPlayers = Integer.parseInt(args[0]);
players = new Player[numberOfPlayers];
deck = createDeck();
for(int i = 0; i < numberOfPlayers; i++){
players[i] = new Player("Player" + (i+1));
}
for(Player player : players){
List<Card> hand = deck.dealHand();
player.drawHand(hand);
}
String cardRank = players[turn - 1].throwingError();
}
代码在到达主函数的最后一行时抛出 ConcurrentModificationException,这让我感到困惑,因为没有元素被迭代或从 Player class.[=14 的手牌字段中删除=]
getRank() 方法是另一个 class Card 的一部分。很简单getter
public String getRank() {
return rank;
}
这里是堆栈跟踪:
Exception in thread "main" java.util.ConcurrentModificationException
at java.util.ArrayList$SubList.checkForComodification(ArrayList.java:1231)
at java.util.ArrayList$SubList.listIterator(ArrayList.java:1091)
at java.util.AbstractList.listIterator(AbstractList.java:299)
at java.util.ArrayList$SubList.iterator(ArrayList.java:1087)
at java.util.AbstractCollection.toString(AbstractCollection.java:454)
at java.lang.String.valueOf(String.java:2981)
at java.io.PrintStream.println(PrintStream.java:821)
at Player.getRank(Player.java:22)
at GameEngine.nextTurn(GameEngine.java:32)
at GameEngine.main(GameEngine.java:27)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:483)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147)
在此代码中:
for(Player player : players){
List<Card> hand = deck.dealHand();
player.drawHand(hand);
}
您的 dealHand()
方法是在每次迭代中 return 创建一个子列表,同时添加到后备列表。这是支持列表的结构修改,正如 sublist()
的 Javadoc 所说,子列表方法的行为在支持列表的结构修改后未定义。因此,子列表上的 get() 操作抛出。
要么一次创建整个列表,然后将其分成子列表,要么 return 每个子列表的副本。