Изменение размера arraylist при использовании улучшенного цикла для цикла

Question

Я пытаюсь играть в войну, и всякий раз, когда вы сравниваете две карты if(c.getValue() > Player2.get(x).getValue()){... Я нахожу победителя и перераспределяю карту игрока и карту двух игроков на стопку, ArrayList<Card> Pile = new ArrayList<Card>();, а затем распространять эту кучу на победителя, основываясь на булевом, но консоль бросает «concurrentModificationException». Я не знаю другого способа подойти к этому. Как добавить к arraylist постоянно меняющегося размера после сравнения двух аррайалистов объектов?

import java.util.ArrayList; 

public class Deck { 
    ArrayList<Card> deck = new ArrayList<Card>(); 
    ArrayList<Card> player1 = new ArrayList<Card>(); 
    ArrayList<Card> player2 = new ArrayList<Card>(); 
    ArrayList<Card> pile = new ArrayList<Card>(); 
    boolean winner; 

    public Deck(){ 

    } 
    public void createDeck(){ 
     for(int i =0; i<4; i++){   
      for(int j =1; j<15;j++){    
       if(i==0) deck.add(new Card("Spades", j));     
       if(i==1) deck.add(new Card("Hearts", j)); 
       if(i==2) deck.add(new Card("Clubs", j)); 
       if(i==3) deck.add(new Card("Diamonds",j)); 

      } 
     } 
    } 
    public void splitDeck(){ 
     for(int i = 0; i< 52;i++){ 
      int r = (int) (Math.random()*(51-i)); 
      if(i%2==0){ 
       player1.add(deck.get(r)); 
      } 
      else{ 
       player2.add(deck.get(r)); 
      } 
      deck.remove(r); 
     } 

    } 
    public int [] war(){ 

     int[]score = {0,0}; 

     for(Card c : player1){ 
      int x = player1.indexOf(c); 
      if(c.getValue() > player2.get(x).getValue()){ 
       winner = true; 
       score[0]++; 
       pile.add(player1.get(x)); 
       pile.add(player2.get(x)); 
       distributeCards(); 
      } 
      else if(c.getValue() <player2.get(x).getValue()) score[1]++; 
      distributeCards(); 
     } 

     return score; 
    } 

    public void distributeCards(){ 

      if(winner==true){ 
       for(int i = 0; i<pile.size(); i++){ 
        player1.add(pile.get(i)); 
       } 
      } 
      else{ 
       for(int i = 0; i<pile.size(); i++){ 
        player2.add(pile.get(i)); 
       } 
      } 
     } 


    public String warGame(){ 
     String print= ""; 
     for(Card c : player1){ 
      int x = player1.indexOf(c); 
      if(c.getValue() > player2.get(x).getValue()){ 

       print+= "\n"+"Player1 won"; 
      } 
      else if(c.getValue() <player2.get(x).getValue()) 
       print+="\n"+"Player2 won"; 

      else if(c.getValue() ==player2.get(x).getValue()) 
       print+= "\n"+"Its a tie"; 
     } 

     return print; 

    } 

} 

ошибка в консоли

Exception in thread "main" java.util.ConcurrentModificationException 
    at java.util.ArrayList$Itr.checkForComodification(Unknown Source) 
    at java.util.ArrayList$Itr.next(Unknown Source) 
    at Deck.war(Deck.java:69) 
    at War.main(War.java:6) 

Answer 1

Что вам нужно, это CopyOnWriteArrayList см http://docs.oracle.com/javase/tutorial/collections/implementations/list.html. Суть его заключается в том, что CopyOnWriteArrayList не бросает исключения одновременной модификации (поэтому вы можете добавлять к ним, итерации по ним), однако это происходит за счет копирования базовых данных каждый раз, когда есть запись. Так как ваш вариант использования - это военная игра, эта работа, но, вероятно, не будет такой заметной.

Answer 2

Вы также можете попробовать ListIterator, который поддерживает метод удаления/добавления во время самой итерации.

Answer 3

Если вы хотите изменить ArrayList вы итерация, решение довольно просто:

private List<YourObject> list = new ArrayList<>(); 
public void removeFromList(){ 
    List<YourObject> tempList = new ArrayList<>(); 
    for (Object o: list) { 
     o.doSomething(); 
     if (yourCondition){ 
      tempList.add(o); 
     } 
    } 
    list = tempList; 
} 

Это для удаления объектов, но реализация для добавления их также очень просто.