2

Por que é considerado incorreto remover um objeto de uma coleção desta forma? E por que o lançamento da exceção ConcurrentModificationException?

for(String item: list) {
  list.remove(item);
}
2

2 Respostas 2

4

No for nesse modelo de for each existe um iterador, que é um mecanismo de controle dos itens que está na coleção para ir navegando pelos itens. Se você remove um item fica complicado continuar mantendo o iterador, é até possível que alguma implementação consiga se virar bem com isso, mas nem todas são garantidas que a iteração pode continuar sem problemas, pode nem ser possível saber mais quais são os próximos itens, pode ser que não consiga identificar mais quando acabaram os itens, pode até mesmo que acabe os itens prematuramente. Esse tipo de for existe para garantir que o acesso é coerente. Se deseja o risco e flexibilidade, ainda pode usar o for "bruto" e aí é problema seu iterar em cada item, ele deixa você fazer e assumir a responsabilidade por problemas.

E cada linguagem ou implementação pode ter suas próprias dificuldades.

Mas este erro pode não ocorrer. Eu fiz o que postou e não deu o erro:

import java.util.*;

class Ideone {
    public static void main (String[] args) {
        List<String> list = new ArrayList<String>();
        list.add("a");
        list.add("b");
        list.add("c");
        for (String i: list) {
            System.out.println("Antes => " + i);
            if (i == "b") list.remove(i);
            System.out.println("Depois => " + i);
        }
    }
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

Esta forma é mais garantida:

import java.util.*;

class Ideone {
    public static void main (String[] args) {
        List<String> list = new ArrayList<String>();
        list.add("a");
        list.add("b");
        list.add("c");
        Iterator<String> i = list.iterator();
        while (i.hasNext()) {
            String nome = i.next();
            System.out.println("Antes => " + nome);
            i.remove();
            System.out.println("Depois => " + nome);
        }
    }
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

Veja um exemplo onde pode ou não fazer exatamente o que se espera:

import java.util.*;

class Ideone {
    public static void main (String[] args) {
        List<String> list = new ArrayList<String>();
        list.add("a");
        list.add("b");
        list.add("c");
        for (int i = 0; i < list.size(); i++) {
            System.out.println("Passo " + i + " Antes => " + list.get(i));
            list.remove(list.get(i));
            System.out.println("Passo " + i + " Depois => " + list.get(i));
        }
    }
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

Após eliminar b ele já vai direto para c, era isto que queria? Nem sempre é o esperado, afinal você ainda está em um passo (o segundo) do laço que estava tratando de b. Mas é diferente do ocorrido nos casos acima.

Assim dá erro:

import java.util.*;

class Ideone {
    public static void main (String[] args) {
        List<String> list = new ArrayList<String>();
        list.add("a");
        list.add("b");
        list.add("c");
        for (String i: list) {
            System.out.println("Antes => " + i);
            list.remove(i);
            System.out.println("Depois => " + i);
        }
    }
}

É porque o iterador já deu erro. Note que ele não dá o erro preventivamente, ele dá quando efetivamente se perdeu.

4

por que o lançamento da exceção ConcurrentModificationException?

Essa exceção é lançada pois é um mecanismo de fail-fast para os casos onde existe qualquer modificação na coleção que estamos iterando.

Em outras palavras, antes que "de fato" dê problema, caso haja qualquer modificação na coleção, é feito o trigger da Exception.


Para contornar isso, você pode fazer o uso do Iterator.
Na verdade, o for each usa o Iterator por debaixo dos panos, mas para ser menos verboso, ele "esconde" o código completo e você não precisa digitar tudo sempre.

Para resolver o problema você pode fazer algo como:

for (Iterator<String> iterator = list.iterator(); iterator.hasNext();) {
    String item = iterator.next();
    iterator.remove();
}

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .