Удаление элементов из ArrayList

Question

В этой программе я веду список дуэлянтов, которые убивают лучший дуэлянт, когда их случайное значение int = 0. У меня проблемы с циклом в нижней части, и я получаю эту ошибку:

Exception in thread "main" java.util.ConcurrentModificationException
at java.util.AbstractList$Itr.checkForComodification(Unknown Source)
at java.util.AbstractList$Itr.next(Unknown Source)
at Duelist.main(Duelist.java:74)

Код

import java.util.Arrays;
import java.util.Random;
import java.util.ArrayList;
import java.util.Collections;

public class Duelist implements Comparable{

private String name;
private int chance;
private boolean alive;


public Duelist(String duelistName, int hitChance){
    alive = true;
    name = duelistName;
    chance = hitChance;

    }
public String getName(){
    return name;
}
public int getChance(){
    return chance;
}
public boolean getLife(){
    return alive;
}
public void kill(){
    alive = false;
}

 public int compareTo(Object anotherDuelist) throws ClassCastException {
        if (!(anotherDuelist instanceof Duelist))
          throw new ClassCastException("A Duelist object expected.");
        int anotherDuelistChance = ((Duelist) anotherDuelist).getChance();  
        return this.chance - anotherDuelistChance;    
      }

public static void main(String[] args){

ArrayList<Duelist> duelers = new ArrayList<Duelist>(5);
//ArrayList<Duelist> personToKill= new ArrayList<Duelist>(5);
ArrayList<Duelist> rank = new ArrayList<Duelist>(5);
Random generator = new Random();




Duelist Antoine = new Duelist("Antoine", 3);
Duelist Francis = new Duelist("Francis", 6);
Duelist Juliette = new Duelist("Juliettee", 1);

duelers.add(Antoine); duelers.add(Francis); duelers.add(Juliette);



//System.out.println(duelers);


//for(Duelist element : duelers){
//  System.out.println(element.getName());
//  System.out.println(element.getChance());
//}
Collections.sort(duelers);
Collections.reverse(duelers);
//for(Duelist element : duelers){
    //System.out.println(element.getName());
    //System.out.println(element.getChance());
//}

while(duelers.size() > 1){


    for(Duelist element : duelers){



            System.out.println(element.getName());
            System.out.println("Chance:" + element.getChance());
            int randomInt = generator.nextInt(element.getChance());
            System.out.println("Random int " + randomInt);



            //Destroy target if randomInt equals 0
            if (randomInt == 0){


                //Check to make sure the best duelist is not killing themselves
                if (element.equals(duelers.get(duelers.size()-1))){
                    System.out.println("LASTDUDE");
                    Duelist bestDueler = duelers.get(duelers.size()-2);
                    bestDueler.kill();
                    rank.add(element);
                    duelers.remove(bestDueler);
                }

                else {
                    System.out.println("Killshot");
                    Duelist bestDueler = duelers.get(duelers.size()-1);
                    bestDueler.kill();
                    rank.add(element);
                    duelers.remove(bestDueler);
                }

            }


    }
}
System.out.println(duelers);


}
}

Answer 1

for(Duelist element : duelers){

Пока вы находитесь внутри этого блока, вы перебираете список дуэлянтов, что означает, что вы не можете изменить список, не вызывая ConcurrentModificationException, если вы не используете * 1005 итератора*remove() метод.

Проблема в том, что у вас нет доступа к итератору.Поэтому вам придется изменить цикл foreach на цикл while с итератором:

Iterator<Duelist> it = duelists.iterator();
while(it.hasNext()){
    Duelist element = it.next();
    // continue as before

(обновить после комментариев) или лучше:

for(Iterator<X> it=x.iterator();it.hasNext();){
    Duelist element = it.next();
    //continue as before

и вместо

duelers.remove(element);

запись

// the problem here is: you can only remove the current element,
// so you'll have to re-think some of your code
it.remove();

Или: вы можете создать копию своего списка для повторения.Это пустая трата памяти (и запах кода), но если ваше приложение маленькое, оно не должно иметь большого значения и потребует наименьшего количества переписывания с вашей стороны.

for(Duelist element : new ArrayList<Duelist>(duelers)){
// continue as above

Answer 2

Вы не можете изменить коллекцию внутри цикла foreach, вместо этого используйте Iterator и когда вы хотите что-то удалить, используйте Iterator.remove ():

Answer 3

Проблема в этой строке duelers.remove(bestDueler);

Вы не должны изменять список во время итерации по нему и не использовать итератор
Вы неявно используете итератор по for(Duelist element : duelers)
Используйте итератор явно, т.е. duelers.listIterator() и используйте remove из iterator вместо