funcion remove ArrayList

[ElAthlit]

Hola a todos,
tengo un problema que es muy raro:

Resumen rápido: Al eliminar filas de una lista si coincide con un literal, me lo elimina "uno si y otro no, así hasta el final"


Tengo un ArrayList<Objeto1> con varias filas ya.

Pero al hacer lo siguiente:

Código Java:

for (int i=0;i<list.size();i++){ //inicio el bucle
Objeto1 obj = (Objeto1)list.get(i); //Extraigo los objetos
if (obj.getName().equals("Pepe"){ //Los comparo
list.remove(i); //Si coincide, elimino la fila
}
}
 

Suponiendo que todas las filas tengan como nombre "Pepe" me elimina una si y otra no hasta el final.

¿Como puedo eliminar todas las filas que tengan ese nombre?

Gracias a todos y un saludo.

[farfamorA]

Pues el error que cometes está claro.
Te lo explico. Supongamos que tienes la lista:

Código Java:

List<String> listaNombres = new ArrayList<String>();
listaNombres.add("Pepe");
listaNombres.add("Pepe");
listaNombres.add("Juan");
listaNombres.add("Pepe");
listaNombres.add("Pepe");
listaNombres.add("Roberto");
 

Para la primera iteración de tu bucle, i vale 0. Se cumple la condicional, y procedes a remover al Pepe que está en el índice 0 de la lista. Como ya no está el Pepe del índice cero, entonces el Pepe que estaba en el índice 1 pasó a estar en el índice 0. De la misma manera, Juan estaba en el índice 2, pero ahora estará en el índice 1, y así sucesivamente con los demás integrantes de la lista.
Ahora para la segunda iteración de tu bucle, i vale 1. ¡Hey, pero hay un Pepe en la posición cero! Para tu mala suerte, el bucle ya pasó el índice cero. ¿Qué hacer? Pues sencillo, cada vez que elimines un elemento de tu lista haz que tu algoritmo mantenga el valor de i.

Código Java:

for (int i=0;i<listaNombres.size();i++){
    if (listaNombres.get(i).equals("Pepe")){
        listaNombres.remove(i);
        i--;
    }
}
 

[zero0097]

Ho puedes dejar el aumento solo en el bloque asi:

Código PHP:

  for (int i=0; i<listaNombres.size(); ){ 
    if (listaNombres.get(i).equals("Pepe")){ 
        listaNombres.remove(i); 
    } else i++; 
}  
   

en teoría también funciona...

[ElAthlit]

Hola farfamorA,
tu explicación más clara imposible. Muchas gracias por aclararme este tema!

Gracias a ambos!!

Un saludo!

[farfamorA]

No hay de qué, estamos para ayudarnos. Saludos, pueden cerrar el tema.