尝试从List获取元素时出现ConcurrentModificationException

时间:2016-10-07 23:48:30

标签: java concurrentmodification

所以我正在尝试创建一个简单的Card游戏,为此我创建了一个拥有名称和玩家手牌的Player类。

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

代码在到达main函数的最后一行时抛出一个ConcurrentModificationException,这让我很困惑,因为在Player类的hand字段中没有迭代或删除任何元素。

getRank()方法是名为Card的另一个类的一部分。这是一个简单的吸气剂

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)

1 个答案:

答案 0 :(得分:0)

在此代码中:

for(Player player : players){
    List<Card> hand = deck.dealHand();
    player.drawHand(hand);
}

您的dealHand()方法在每次迭代中返回一个子列表,同时添加到支持列表中。这是对支持列表的结构修改,并且正如sublist()的Javadoc所说,在对支持列表进行结构修改之后,子列表的方法的行为是未定义的。因此,子列表上的get()操作会抛出。

一次性创建整个列表,然后将其划分为子列表,或者返回每个子列表的副本。