Question

Hola.

El problema es bastante básico pero parece que me he atascado.

Estoy recorriendo un ArrayList lleno de Objetos. Si el objeto cumple una propiedad concreta elimino el objeto de ArrayList. Pero en ese momento, el tamaño del ArrayList a cambiado y no termino de recorrer todo el ArrayList.

Que solución me proponéis?

            for(int i=0; i<instalaciones.size();i++ ){
                Instalacion instalacion = instalaciones.get(i);
                if(instalacion.cumpleDistancia(this.loc, distancia)){
                nombre = instalacion.getNombre();
                idInstalacion = instalacion.getIdInstalacion();

                }
                else{
                    instalaciones.remove(i);

                }
        }

saludos y gracias.

Answer 1

Hola.

Yo creo que lo más fácil que puedes hacer es recorrer la lista con un iterador. Tiene la ventaja de que ya tiene el método remove especificado en la interfaz.

http://docs.oracle.com/javase/7/docs/api/java/util/Iterator.html

Básicamente, un iterador es un objeto que señala a uno de los elementos de la colección. Te proporciona métodos para ver si hay un elemento siguiente y para saltar a dicho elemento siguiente, así como para eliminarlo.

También es más eficiente que las otras soluciones propuestas, ya que el iterador marca la posición a borrar y el programa no necesita volver a buscar dentro de la colección para comparar objetos.

Answer 2

Guardar los índices que quieres borrar, después due recorriste todo, eliminar la posición n - (veces que haz borrado). Por ejemplo, si quieres borrar las posiciones 1, 4 y 8, debes recorrer el array para borrar las posiciones 1, 3, y 6 (posiciones actualizadas).

La otra es que guardes en un array diferente los que no se vayan a borrar, una vez que termines la validación, borras el array original completo, y renombras el array creado en la función.

O puedes usar un bucle while.

Quizá eso sirva.

Answer 3

La mejor manera de hacer esto sin tener inconvenientes al borrar es la siguiente:

for(Instalacion instalacion : instalaciones){
    if(instalacion.cumpleDistancia(this.loc, distancia)){
         nombre = instalacion.getNombre();
         idInstalacion = instalacion.getIdInstalacion();
     } else {
         instalaciones.remove(instalacion);
     }
}

Así te evitas el manejo de los indices. Esta es una buena práctica desde java 1.5