Suppression d'un élément de la liste - pendant l'itération - quel est le problème avec cet idiome?

Quelques réponses pour expliquer pourquoi cela se produit et certaines expliquer ce que vous devriez vous avez fait. Je vais sans vergogne mettre les morceaux ensemble.

---

Quelle est la raison?

Parce que le langage Python est conçu pour traiter ce cas d'utilisation différente. La documentation indique clairement:

Il n'est pas sûr de modifier la séquence itérée dans la boucle (cela peut arriver pour mutable types de séquence, comme les listes). Si vous avez besoin de modifier la liste, vous êtes à parcourir (par exemple, pour dupliquer les éléments sélectionnés), vous devez effectuer une itération sur une copie.

_{Accent de la mine. Voir la page liée, la documentation est protégée par copyright et tous les droits sont réservés.}

Que vous pouvez facilement comprendre pourquoi vous l'avez obtenu ce que vous avez, mais c'est essentiellement un comportement indéfini qui permettent de changer facilement sans l'avertissement de construire de construire. Il suffit de ne pas le faire.

C'est comme vous vous demandez pourquoi i += i++ + ++i est-ce que l'enfer c'est que la ligne ne sur votre architecture sur votre build spécifique de votre compilateur pour votre langue -- y compris mais non limité à bousiller votre ordinateur et prise de démons voler hors de votre nez 🙂

---

Comment cela pourrait-il être ré-écrit pour supprimer chaque élément?

• del letters[:] (si vous avez besoin de changer toutes les références à cet objet)
• letters[:] = [] (si vous avez besoin de changer toutes les références à cet objet)
• letters = [] (si vous voulez juste travailler avec un nouvel objet)

Peut-être vous souhaitez supprimer certains éléments en fonction d'une condition? Dans ce cas, vous devez effectuer une itération sur une copie de la liste. Le moyen le plus facile de faire une copie est à rendre une tranche contenant l'ensemble de la liste avec la [:] syntaxe, comme suit:

#remove unsafe commands
commands = ["ls", "cd", "rm -rf /"]
for cmd in commands[:]:
  if "rm " in cmd:
    commands.remove(cmd)

Si votre chèque n'est pas particulièrement compliqué, vous pouvez (et devriez) filtre à la place:

commands = [cmd for cmd in commands if not is_malicious(cmd)]

Vous ne pouvez pas modifier la liste que vous itération, sinon vous obtenez cette étrange type de résultat. Pour ce faire, vous devez effectuer une itération sur une copie de la liste:

for i in letters[:]:
  letters.remove(i)

ce que vous voulez faire est:

letters[:] = []

ou

del letters[:]

Ce sera de préserver objet d'origine letters a été pointant vers. D'autres options comme, letters = []serait de créer un nouvel objet et le point letters: ancien objet serait généralement être récupérées après un certain temps.

La raison de ne pas toutes les valeurs ont été supprimés, c'est que vous êtes à la modification de la liste lors de l'itération sur elle.

ETA: si vous souhaitez filtrer des valeurs d'une liste vous pouvez utiliser interprétations de la liste comme ceci:

>>> letters=['a','b','c','d','e','f','g','h','i','j','k','l']
>>> [l for l in letters if ord(l) % 2]
['a', 'c', 'e', 'g', 'i', 'k']

    #!/usr/bin/env python
    import random
    a=range(10)

    while len(a):
        print a
        for i in a[:]:
            if random.random() > 0.5:
                print "removing: %d" % i
                a.remove(i)
            else:
                print "keeping: %d"  % i           

    print "done!"
    a=range(10)

    while len(a):
        print a
        for i in a:
            if random.random() > 0.5:
                print "removing: %d" % i
                a.remove(i)
            else:
                print "keeping: %d"  % i           

    print "done!"

Je pense que c'est ce qui explique le problème un peu mieux, le bloc de code fonctionne, alors que celui du bas ne marche pas.

Les éléments qui sont "gardés" en bas de la liste de ne jamais obtenir de l'imprimé, parce que vous êtes modifiying la liste, vous êtes à parcourir, ce qui est une recette pour un désastre.