ConcurrentModificationException per ArrayList

Question

Ho il seguente pezzo di codice:

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(); 
} 

Quando provo a farlo funzionare, ottengo ConcurrentModificationException, qualcuno può spiegare perché succede, anche se il codice è in esecuzione in stesso thread? E come potrei evitarlo?

Answer 1

Non è possibile rimuovere dalla lista se si sta navigando con "per ogni" ciclo. È possibile utilizzare Iterator. Sostituire:

for (DrugStrength aDrugStrength : aDrugStrengthList) { 
    if (!aDrugStrength.isValidDrugDescription()) { 
     aDrugStrengthList.remove(aDrugStrength); 
    } 
} 

Con:

for (Iterator<DrugStrength> it = aDrugStrengthList.iterator(); it.hasNext();) { 
    DrugStrength aDrugStrength = it.next(); 
    if (!aDrugStrength.isValidDrugDescription()) { 
     it.remove(); 
    } 
} 

Answer 2

Durante l'iterazione nel ciclo, si sta tentando di modificare il valore dell'elenco nell'operazione remove(). Ciò comporterà ConcurrentModificationException.

Seguire il codice qui sotto, che vi ottenere quello che vuoi ma non buttare eccezioni

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(); 
} 

Answer 3

Come le altre risposte dicono, non è possibile rimuovere un elemento da una collezione che stai iterare su . È possibile aggirare questo utilizzando esplicitamente un Iterator e rimuovere l'elemento lì.

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

Answer 4

ci dovrebbero ha un fine dell'esecuzione concomitante di interfaccia List supportare tale operazione.

provare java.util.concurrent.CopyOnWriteArrayList.class

Answer 5

mi piace un ordine inverso ciclo for come ad esempio:

int size = list.size(); 
for (int i = size - 1; i >= 0; i--) { 
    if(remove){ 
     list.remove(i); 
    } 
} 

perché non richiede l'apprendimento eventuali nuove strutture di dati o classi.

Answer 6

È possibile utilizzare classi di raccolta simultanee per evitare ConcurrentModificationException durante l'iterazione su una raccolta, ad esempio CopyOnWriteArrayList anziché ArrayList.

controllare questo post per ConcurrentHashMap

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