Question

我试图参加战争，每当你比较两张牌if(c.getValue() > Player2.get(x).getValue()){...时，我找到了胜利者，并将玩家的牌和玩家二张牌重新分配给保持桩，ArrayList<Card> Pile = new ArrayList<Card>();，然后根据布尔值将该桩分配给获胜者，但是控制台正在抛出＆＃34; concurrentModificationException＆＃34;我不知道任何其他方法来解决这个问题。在比较两个对象的arraylists之后，如何添加一个不断变化的大小的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;
}

这是用于删除对象，但添加它们的实现也非常简单。

使用增强的for循环时更改arraylist大小

3 个答案: