Erro com Collections: java.util.ConcurrentModificationException

Entusiasta Java Membro desde: 23/11/2004 08:54:15 Mensagens: 17 Offline

Olá Pessoal!

Estou com um problema em trabalhar com collections em um trecho do meu aplicativo. Segue o código

for(Despesa d : despesas) {
            if(!d.isVitalicia()) {
                if (!pagamentoDespesa.isPaga(d)) {
                    List<ParcelaDespesa> parcelas = d.getParcelasDespesa();

if(parcelas != null) {
                        Iterator<ParcelaDespesa> it = parcelas.iterator();
                        
                        while (it.hasNext()) {                                                        
                            //ConcurrentModificationException é lancado na segunda iteração.
                            ParcelaDespesa p = it.next();

if (!pagamentoDespesa.isPaga(p))
                                 //se eu tirar esse código funciona, 
                                 //porém não e o comportamento que eu desejo.
                                 d.addParcelaDespesa(p);
                        }
                    }
                    retornar.add(d);
                }
            }
            else {
                //tratar despesas vitalícias
            }            
        }

Segue abaixo o código que tem em Despesa.addParcelaDespesa(ParcelaDespesa);

Já tentei implementar em ParcelaDespesa a interface Cloneable e sobreescrever o clone. Não funcionou também

Segue stacktrace lancado;

Exception in thread "AWT-EventQueue-0" java.util.ConcurrentModificationException
        at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:449)
        at java.util.AbstractList$Itr.next(AbstractList.java:420)
        at gfi.modelo.Despesa.obterDespesasPagar(Despesa.java:185)
        at gfi.forms.PagarDespesasMestre.btnPesquisar2ActionPerformed(PagarDespesasMestre.java:165)
        at gfi.forms.PagarDespesasMestre.access$000(PagarDespesasMestre.java:19)
        at gfi.forms.PagarDespesasMestre$1.actionPerformed(PagarDespesasMestre.java:56)
        at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1849)
        at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2169)
        at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:420)
        at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:258)
        at javax.swing.plaf.basic.BasicButtonListener$Actions.actionPerformed(BasicButtonListener.java:285)
        at javax.swing.SwingUtilities.notifyAction(SwingUtilities.java:1571)
        at javax.swing.JComponent.processKeyBinding(JComponent.java:2763)
        at javax.swing.JComponent.processKeyBindings(JComponent.java:2798)
        at javax.swing.JComponent.processKeyEvent(JComponent.java:2726)
        at java.awt.Component.processEvent(Component.java:5265)
        at java.awt.Container.processEvent(Container.java:1966)
        at java.awt.Component.dispatchEventImpl(Component.java:3955)
        at java.awt.Container.dispatchEventImpl(Container.java:2024)
        at java.awt.Component.dispatchEvent(Component.java:3803)
        at java.awt.KeyboardFocusManager.redispatchEvent(KeyboardFocusManager.java:1810)
        at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(DefaultKeyboardFocusManager.java:672)
        at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(DefaultKeyboardFocusManager.java:920)
        at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(DefaultKeyboardFocusManager.java:798)
        at java.awt.DefaultKeyboardFocusManager.dispatchEvent(DefaultKeyboardFocusManager.java:636)
        at java.awt.Component.dispatchEventImpl(Component.java:3841)
        at java.awt.Container.dispatchEventImpl(Container.java:2024)
        at java.awt.Window.dispatchEventImpl(Window.java:1774)
        at java.awt.Component.dispatchEvent(Component.java:3803)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:463)
        at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

Qualquer ajuda será bem-vinda!

Abraços

Ramon

Esse erro é gerado qdo vc tenta modificar o conteudo de uma coleção no loop do iterator.

Entusiasta Java Membro desde: 23/11/2004 08:54:15 Mensagens: 17 Offline

Fabrício Cozer Martins wrote:Esse erro é gerado qdo vc tenta modificar o conteudo de uma coleção no loop do iterator.

Obrigado pela resposta!

Isso eu já sei. O que eu quero saber é onde está sendo alterado o item da collection? E qual outra estratégia de implementação que eu posso usar para sanar esse problema.

Alguma sugestão?

Abs

Ramon

vc pode usar a estratégia 'separate collections', onde você trabalha com duas coleções onde uma vc vai adicionando elementos da outra que vc deve tratar.

Entusiasta Java Membro desde: 23/11/2004 08:54:15 Mensagens: 17 Offline

Fabrício Cozer Martins wrote:vc pode usar a estratégia 'separate collections', onde você trabalha com duas coleções onde uma vc vai adicionando elementos da outra que vc deve tratar.

Obrigado! Eu usei a estratégia que voce me indicou e funcionou. Mas não gostei de ter que usar duas collections para uma operação tão simples. Será que isso não é um problema de thread? Já usei o Collections.synchronizedList(minhalista) e deu o mesmo erro.
Não sei oque pode ser.

Abraços!

Ramon

a moral da historia eh: nao modifique a collection durante a iteracao, pq pode dar problemas - mesmo com synchronized, como vc esta modificando a lista durante a iteracao no mesmo thread, a excecao eh lancada...
a sugestao dada eh a mais simples mesmo

ConcurrentModificationException não diz respeito nenhum a modificação entre threads.

Essa exception é lançada sempre que você fizer alguma modificação estrutural em uma collection sem ser pelo iterator. A solução é você realizar as alterações pelo iterator ou usar colections separadas.

pois eh, mas muitas pessoas jah erraram (eu, inclusive) achando que a collection estava sendo modificada por outra thread ou coisa parecida...
pelo menos para mim a excecao eh um pouco sugestiva nesse sentido, alem do javadoc mencionar:

For example, it is not generally permissible for one thread to modify a Collection while another thread is iterating over it

Eita... eu estou com o mesmo problema que ele (ou ao menos parecido) e não sei como resolver. Se puderem me explicar passo-a-passo por favor, eu ficaria muito agradecido!

Segue abaixo parte do meu código:

Sendo que lá em cima, no início do código, eu criei:

O erro é o mesmo ( java.util.ConcurrentModificationException ) e está marcado nessa linha : p2 [i] = (Produto) busca.next();

Dêem-me uma mãozinha, pessoal. Tou precisando disso pra ontem!! =/

JavaTeenager Membro desde: 14/12/2007 16:20:02 Mensagens: 160 Offline

Outro Collection? É muito Collection! Eu faço assim:

Thread.start() Membro desde: 14/12/2006 09:32:41 Mensagens: 27 Offline

Mais uma opção...

Será que o pessoal de 2006 já não tinha percebido essas duas soluções?

Estava passando pelo mesmo problema,pirando achando que era problemas de thread, to usando sockets e thread, mas o --i; resolveu meu problema!

valeu pessoal!

Thread.start() Membro desde: 14/09/2009 15:37:52 Mensagens: 31 Offline

--i me salvou tb... tks!

Smalltalk Membro desde: 21/03/2010 19:03:06 Mensagens: 1 Offline

Tive o mesmo problema quando eu tentava assim:

Resolvi da seguinte forma:

Notícias, artigos e o maior fórum brasileiro sobre Java