[Java] ConcurrentModificationException... non ne vengo fuori!

[Alex'87]

Allora, ho questo metodo che si occupa di togliere da un ArrayList di Download i download completati o cancellati. Il problema è che mi lancia una ConcurrentModificationException (dove ho messo la freccia) e non capisco perche'! Ogni Download e' un Runnable ed il download avviene quindi in parallelo.

   public void removeCompletedOrCancelledDownloads()
    {
        Iterator<Download> iterator = downloadList.iterator();

        while (iterator.hasNext())
        {
            Download download = iterator.next(); <------- questa riga da problemi

            System.out.println("staus: " + download.getStatus()); // per il debug

            if (download.getStatus() == Download.COMPLETE || download.getStatus() == Download.CANCELLED)
            {
                clearDownload(downloadList.indexOf(download));
            }
        }
    }

 public void clearDownload(int row)
    {        
        downloadList.remove(row);

        // Fire table row deletion notification to table.
        fireTableRowsDeleted(row, row);
    } 

Come posso risolvere? Grazie

edit: ah, l'eccezione salta fuori solo se ci sono almeno due download nell'array list (non importa se completati o meno). Con un solo download il metodo termina correttamente :|

edit2: ho provato anche a sincronizzare l'accesso all'arraylist ma niente

[LeleFT]

Non lo puoi fare così.
Stai cercando di rimuovere un elemento da una lista che stai, contemporaneamente, scorrendo con un Iterator.

La cosa più semplice è cercare di farne una copia e utilizzare quest'ultima per lo scorrimento.

L'alternativa è quella di "tenere traccia" da qualche parte delle modifiche da apportare, quindi attuarle dopo lo scorrimento. Ad esempio, se devi togliere degli elementi da una lista, ne memorizzi l'indice in un array e, alla fine, scorri l'array rimuovendo gli oggetti tramite il loro indice.

Ciao.

[Alex'87]

:-O

Ma quanto sono scemo! Grazie mille, ora ho capito!

Ciao

[andbin]

L'unica operazione di modifica sulla collezione che è fattibile mentre si sta iterando, è il remove() dell'Iterator. Qualunque altra modifica con altri metodi, comporta l'eccezione che hai già sperimentato.