java.util.ConcurrentModificationException Ошибка исключения

Question

Я пытаюсь сделать приложение Java с графическим интерфейсом. я пишу код, который я хочу позволить пользователю изменить некоторые данные и сохранить эти изменения в текстовом файле. Прежде чем сделать это, я хочу удалить старые данные, которые были изменены, из списка, а затем переписать новые данные с последним изменением. Если я пропустил какой-либо класс, который вы хотели бы увидеть, скажите, пожалуйста, что я выложу его в сеть как можно быстрее

это мой

    public void saveChanges(footBall Player, String name, String level,
        int[] scores, int footSize) {

    try {
        if (CompetitorsList.size() != 0) {
            for (Competitors C : CompetitorsList) {

                if (C instanceof footBall) {
                    String Number = Player.playerNumber + "";
                    if (C.getPlayerNumberAsString().equals(Number)) {
                        System.out.println("c");

 //the error hit me here when i try to remove the object from the list the exception error is       java.util.ConcurrentModificationException 
                        CompetitorsList.remove(C);

                    }

                }
            }

            Name NewName = new Name(name);
            System.out.println("Please get in2");
            footBall NewPlayer = new footBall(Player.playerNumber, scores,
                    level, footSize, NewName);

            CompetitorsList.add(NewPlayer);

            SaveOnFile();
        } else {
            System.out.println("No List");
        }

    } catch (Exception ex) {
        System.out.print("testing4");
        System.out.print("something wrong" + ex);
    }

}

это метод SaveOnFile: public void SaveOnFile () {

    String scoresInString;
    FileWriter fw;
    try {

        fw = new FileWriter("footBall");

        for (Competitors C : CompetitorsList) {
            if (C instanceof footBall) {
                footBall Scores = new footBall();
                scoresInString = Scores.returnScoreAsString(C.scores);
                fw.write(C.playerNumber + ", " + C.name.getFullName()
                        + ", " + C.level + ", " + scoresInString + ","
                        + ((footBall) C).footSize() + "\n");
                fw.write("\r\n");
            }

        }

        fw.close();

    }
    // message and stop if file not found
    catch (FileNotFoundException fnf) {
        System.out.println("File not found ");
        System.exit(0);
    }
    // stack trace here because we don't expect to come here
    catch (IOException ioe) {
        ioe.printStackTrace();
        System.exit(1);
    }
}

Answer 1

В качестве альтернативы, если вы хотите использовать тот же синтаксис «для следующего» и не переходить на синтаксис «Итератор», соберите все объекты, которые нужно удалить, во временную коллекцию.например,

ArrayList<Competitors> removeThese = new ArrayList<Competitors>();
for (Competitors C : CompetitorsList) {
   if (wantToRemove(C))  // your number logic goes here...
      removeThese.add(C);
}
CompetitorsList.removeAll(removeThese);

Answer 2

Вызов remove () для коллекции делает недействительным весь активный итератор. Вместо этого вы должны использовать метод Iterator.remove ():

for(Iterator<Competitors> it = CompetitorsList.iterator(); it.hasNext(); ) {
    Competitors C = it.next();
    if(C instanceof ...) {
        if(C.getPlayerNumberAsString().equals(Number))
            it.remove();
    ...

Таким образом, iterator () знает, как изменяется коллекция, что в противном случае было бы невозможно, так как ArrayList не отслеживает созданные итераторы.