Problema eliminando elemento de un ArrayList

[tunombre]

Bien el caso es el siguiente:

Tengo un declarado: private ArrayList <Coche> listadoCoches = new ArrayList();

Y este método que elimina un coche del ArrayList siempre que coincida la matricula

Código:

23       public void sacaCoche(String matricula)
24	{
25		Coche cocheABorrar = null;
26
27		for(Coche coche:listadoCoches)
28		{
29			if(coche.getMatricula().equalsIgnoreCase(matricula))
30			{
31				cocheABorrar = coche;
32				break;
33			}
34		}
35		if( cocheABorrar == null )
36			System.out.println("Coche no encontrado");
37		else this.listadoCoches.remove(cocheABorrar);		
38	}

Cuando lo ejecuto pasandole una matrícula existente en el Arraylist me devuelve esto:

Código:

Exception in thread "main" java.util.ConcurrentModificationException
        at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:372)
        at java.util.AbstractList$Itr.next(AbstractList.java:343)
        at boletin2.ejercicio7.Concesionario.sacaCoche(Concesionario.java:26)
        at boletin2.ejercicio7.Prueba.main(Prueba.java:40)
Java Result: 1

Según entiendo tiene que ver con el Iterator implicito en el bucle pero no entiendo bien cuál es el problema.


Gracias

[tunombre]

He cambiado el bucle por si podía ser problema del Iterator pero sigue lanzandome el mismo error:

Código:

  public void sacaCoche(String matricula)
	{
		Coche cocheABorrar = null;

		for( int i=0; i<this.listadoCoches.size(); i++)
		{
			Coche coche = this.listadoCoches.get(i);
			if(coche.getMatricula().equalsIgnoreCase(matricula))
			{
				cocheABorrar = coche;
				break;
			}
		}
		if( cocheABorrar == null )
			System.out.println("Coche no encontrado");
		else this.listadoCoches.remove(cocheABorrar);		
	}

Nadie ve dónde puede estar el error?

[GreenEyed]

ConcurrentModificationException

No puedes borrar elementos de una coleccion mientras la estas recorriendo. Lo dice el API. Para borrarlo tienes que usar el metodo propio del Iterator que uses para recorrerlo. Así que lo habitual es recorrer el Iterator y cuando te encuentres el elemento a borrar, ejecutar miIterador.remove();

En el segundo caso que pones no se exactamente por que da el error a no ser que tengas un Iterador por ahi, si no te deberia dar otra excepción, pero como no esta el stacktrace no puedo detallar más.

S!

[josimapi]

El problema lo tienes en la última línea, si te fijas el método "remove" tiene que recibir como parámetro un int y no un objecto. Vamos es lo que creo que está mal, ya nos dirás...

[GreenEyed]

Iniciado por josimapi

El problema lo tienes en la última línea, si te fijas el método "remove" tiene que recibir como parámetro un int y no un objecto. Vamos es lo que creo que está mal, ya nos dirás...

En realidad no. El metodo admite tanto hacerlo por indice como por elemento:
http://java.sun.com/j2se/1.5.0/docs/...va.lang.Object)

S!

PD De todas formas, si la llamada del metodo estuviera mal, el error sería de compilación o le daría un ClassCastException o similar, no el error que le esta dando.

[javix3]

Proba esto y contame =)

Código:

 public void sacaCoche(String matricula)
	{
		Coche cocheABorrar = null;
                Iterator itera = this.listadoCoches.iterator();

		while(itera.hasNext())
		{
			Coche coche = itera.next();
			if(coche.getMatricula().equalsIgnoreCase(matricula))
			{
				cocheABorrar = coche;
				break;
			}
		}
		if( cocheABorrar == null )
			System.out.println("Coche no encontrado");
		else this.listadoCoches.remove(cocheABorrar);		
	}

[javiervillalobos]

Iniciado por tunombre

He cambiado el bucle por si podía ser problema del Iterator pero sigue lanzandome el mismo error:

Código:

  public void sacaCoche(String matricula)
	{
		Coche cocheABorrar = null;

		for( int i=0; i<this.listadoCoches.size(); i++)
		{
			Coche coche = this.listadoCoches.get(i);
			if(coche.getMatricula().equalsIgnoreCase(matricula))
			{
				cocheABorrar = coche;
				break;
			}
		}
		if( cocheABorrar == null )
			System.out.println("Coche no encontrado");
		else this.listadoCoches.remove(cocheABorrar);		
	}

Nadie ve dónde puede estar el error?

Tal vez el problema se de cuando llamas al método "sacaCoche(...)" desde varios lugares en forma concurrente. Ya que this.listadoCoches es una propiedad compartida, pueden haber múltiples threads (o request en una pág. web) accediendo, a través de "sacaCoche(...)" a este listado. De esta forma, cuando hay uno de esos threads (o request) recorriendo el listado para detectar qué coche sacar, puede haber otro thread haciendo el "remove" y ahí la cosa queda "indeterminable". Como precaución a esto, JVM arroja la "exception" que se indica.
Prueba usar bloques "synchronized", utilizando mapas o listas sincronizadas (ver java.util.Collections).

[GreenEyed]

El mensaje es de enero de 2008. RIP.

[xlugo2002]

import java.util.ArrayList;
import java.util.Iterator;

public class Metodos {

private static ArrayList<String> modifica(ArrayList<String> arreglo){
arreglo.add("seis");
int u=1;

Iterator it = arreglo.iterator();

while(it.hasNext()){
String cad = (String) it.next();
if(cad.equals("uno"))
it.remove();
}


return arreglo;
}


public static void main(String[] args){

int a = 10;
ArrayList<String> arreglo = new ArrayList<String>();

arreglo.add("uno");
arreglo.add("dos");
arreglo.add("tres");
arreglo.add("cuatro");
arreglo.add("cinco");

ArrayList<String> arreglol = modifica(arreglo);

for(String iten : arreglol){
System.out.println(iten + "<br>");
}



}




}