¿Por qué ocurre esto?
El problema es que con remove()
, se está modificando la lista en cuestión. Por lo tanto los valores no van a ser los mismos en cada ciclo. La variable temporal que del ciclo va a tomar los valores de cada elemento en el iterable, significa que la iteración va a seguir por el siguiente elemento y va a dejar ciertos valores atrás.
for i in lista:
print(i)
if i % 2 != 0:
lista.remove(i)
Con este ciclo se puede visualizar que valores toma i, por cada iteración. En pantalla se obtiene
29
23
16
12
5
-12
Como se puede, nunca se evalúan los valores 24, 27 y -5
, se quedan en la lista sin importar si se cumple la condición o no.
¿Cómo soluciono esto?
Lo más práctico es clonar la lista a una lista auxiliar, e iterar sobre esta lista auxiliar. Para no afectar los valores que toma la variable temporal:
lista = [29, 24, 23, 17, 16, 12, 5, -5, -12]
lista_aux = lista[:]
for elemento in lista_aux:
if elemento % 2 != 0:
lista.remove(elemento)
Aquí lo que pasa es que a lista_aux
, se asignan todos los valores de lista. El slice [:]
se refiere a todos los elementos de principio a fin del iterable.
Este pedazo de código mostraría con print(lista)
:
[24, 16, 12, -12]
Si te molesta el hecho de haber creado otra lista entera solo para esto, después del final del ciclo se puede borrar a lista_aux
:
del lista_aux
¿Por qué se debe de copiar un a lista con un slice de lista?
Veamos que pasa cuando se realiza una sentencia como
lista = [8,3,45,6]
lista_copia = lista
Se le está asignando el valor de lista
a lista_copia
. Lo primero que nos puede alarmar, es a la hora de verificar si son el mismo elemento:
print(lista is lista_copia)
muestra
True
Si intentamos modificar los valores de una, se modificará también la otra. Debido a que son el mismo objeto
lista.pop()
print(lista_copia)
muestra
[8, 3, 45]
Aquí se puede ver que se elimina el último elemento de lista
, y esto también a afecta a lista_copia
.
Y... ¿Esto por qué pasa?
El truco está en el operador =
, este es un operador de asignación. Se le asigna el valor de la derecha a la variable de la izquierda. Por lo tanto no hay que asignarle el valor de lista
a lista_copia
, hay que asignarle a lista_copia
cada uno de los elemento de lista
lista = [8,3,45,6]
lista_copia = lista[:]
Esta vez print(lista is lista_copia)
devolverá False
.
Espero te haya servido de algo.