No conviene modificar la coleccion sobre la que iteras.
Generalmente, tratar de borrar elementos de una coleccion mientras se recorre, trae un problema:
Lo curioso, es lo que hace este código:
lista = [1, 2, 3, 3, 4, 5, 6, 7]
for item in lista:
if(item == 3):
lista.remove(item)
print(lista)
Que produce
[1, 2, 3, 4, 5, 6, 7]
Como podrás ver, la iteración no siguió correctamente eliminando los 3 de la lista.
Suposición
El iterador relacionado con la lista, el cual probablemente lleva un contador interno, no se actualiza cuando la lista pierde o adquiere un item. Eso significa que la iteración funciona así
posición 0 -> [1, 2, 3, 3, 4, 5, 6, 7] -> item 1
posición 1 -> [1, 2, 3, 3, 4, 5, 6, 7] -> item 2
posición 2 -> [1, 2, 3, 3, 4, 5, 6, 7] -> item 3 (remueve el 3)
posición 3 -> [1, 2, 3, 4, 5, 6, 7] -> item 4 (se salta el 3 por que el contador no se actualizó)
...
Al parecer, los diccionarios SI avisan que su tamaño cambió produciendo un error:
RuntimeError: dictionary changed size during iteration
¿Podríamos cambiar esto?
Podríamos crear nuestro propio iterador que tenga un contador y que este contador se actualize automaticamente según se eliminen o agregen items de la lista. Pero hace trabajar de más al programador y a la maquina, que tiene que hacer estos chequeos.
Por que no usar .remove para eliminar multiples elementos.
Ten en cuenta que, al usarse remove, Python se pone a buscar elemento por elemento hasta encontrar uno que coincida. Si podemos obtener la posición directamente (con un contador o con matematica) eliminar según indice es más rapido.
Ten en cuenta esto para listas grandes.
La mejor solución
Lo habitual, es que se recurra a crear un nuevo iterador, iterable o collecion e ir agregando o devolviendo los elementos que no queríamos remover. Esto le da menos trabajo al programador y a la maquina.
lista = [1, 2, 3, 3]
nueva_lista = []
for item in lista:
if(item != 3):
nueva_lista.append(item)
O, usando comprensión de listas:
lista = [1, 2, 3, 3]
nueva_lista = [item for item in lista if item != 3]