ConcurrentModificationException для ArrayList [дубликат]

Question

ConcurrentModificationException для ArrayList [дубликат]

этот вопрос уже есть ответ здесь:

итерация по коллекции, избегая ConcurrentModificationException при удалении в цикле

24 ответов

у меня есть следующий фрагмент кода:

private String toString(List<DrugStrength> aDrugStrengthList) {

    StringBuilder str = new StringBuilder();

        for (DrugStrength aDrugStrength : aDrugStrengthList) {

            if (!aDrugStrength.isValidDrugDescription()) {

                aDrugStrengthList.remove(aDrugStrength);

            }

        }

        str.append(aDrugStrengthList);

        if (str.indexOf("]") != -1) {

            str.insert(str.lastIndexOf("]"), "n          " );

        }

    return str.toString();

}

когда я пытаюсь запустить его, я получаю ConcurrentModificationException, может кто-нибудь объяснить, почему это происходит, даже если код работает в том же потоке? И как мне этого избежать?

631 6

java concurrency collections

6 ответов:

Comments

Ничего не найдено.

Konrad Garus · Accepted Answer · 2010-07-06 12:13:46

вы не можете удалить из списка, если вы просматриваете его с помощью "для каждого" цикла. Вы можете использовать Iterator. Заменить:
for (DrugStrength aDrugStrength : aDrugStrengthList) {
    if (!aDrugStrength.isValidDrugDescription()) {
        aDrugStrengthList.remove(aDrugStrength);
    }
}
С:
for (Iterator<DrugStrength> it = aDrugStrengthList.iterator(); it.hasNext(); ) {
    DrugStrength aDrugStrength = it.next();
    if (!aDrugStrength.isValidDrugDescription()) {
        it.remove();
    }
}

scompt.com · Accepted Answer · 2010-07-06 12:11:48

Iterator и удаление элемента там.

Iterator<Item> iter = list.iterator();
while(iter.hasNext()) {
  Item blah = iter.next();
  if(...) {
    iter.remove(); // Removes the 'current' item
  }
}

froman · Accepted Answer · 2012-12-20 02:27:55

Мне нравится обратный порядок для цикла, например:
int size = list.size();
for (int i = size - 1; i >= 0; i--) {
    if(remove){
        list.remove(i);
    }
}
потому что это не требует изучения каких-либо новых структур данных или классов.

idiotgenius · Accepted Answer · 2010-07-06 16:29:34

должна быть параллельная реализация интерфейса списка, поддерживающего такую операцию.

попробуйте java.утиль.параллельный.CopyOnWriteArrayList.класс

bragboy · Accepted Answer · 2016-03-07 16:51:25

при повторении цикла вы пытаетесь изменить значение списка в операции remove (). Это приведет к ConcurrentModificationException.

следуйте приведенному ниже коду, который достигнет того, что вы хотите, и все же не будет бросать никаких исключений
private String toString(List aDrugStrengthList) {
        StringBuilder str = new StringBuilder();
    List removalList = new ArrayList();
    for (DrugStrength aDrugStrength : aDrugStrengthList) {
        if (!aDrugStrength.isValidDrugDescription()) {
            removalList.add(aDrugStrength);
        }
    }
    aDrugStrengthList.removeAll(removalList);
    str.append(aDrugStrengthList);
    if (str.indexOf("]") != -1) {
        str.insert(str.lastIndexOf("]"), "\n          " );
    }
    return str.toString();
}

Suganthan Madhavan Pillai · Accepted Answer · 2013-11-12 06:02:56

мы можем использовать параллельные классы коллекции, чтобы избежать ConcurrentModificationException при итерации по коллекции, например CopyOnWriteArrayList вместо ArrayList.

проверьте этот пост для ConcurrentHashMap

http://www.journaldev.com/122/hashmap-vs-concurrenthashmap-%E2%80%93-example-and-exploring-iterator