Discussion :

[jojo971]

j'ai une liste

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
li=[3,5]
 
for liste in li:  
                nb=liste
                if (nb)>'7':
                        print"    + "+nb+" ||  OK"  
                else:
                        print"    + "+nb+" || FAILED"
                        li.remove(nb)

Je veux juste qu' à chaque fois qu'il rencontre une valeur supérieur à 7 il ecrit OK par contre si inférieur il met FAILED et si inérieur retire de la liste
Problème il passe une seul fois dans la boucle il retire 3 mais pas 5 ,j'ai pas encore compris pourqoi
Avez une piste la solution est surement simple.....il doit probablement sortir de la boucle à cause du remove

[shaiHulud]

Bonjour

Si ton but est de comparer des entiers ou réels, enlève les guillemets.
Sinon il va comparer des caractères (et j'imagine que le caractère '7' ne vaut pas 7)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
if (nb)>7:

Ensuite, tu aura effectivement un bug en faisant un remove sur une liste pendant que tu iteres dessus. A la place, tu stockes les éléments que tu veux retirer dans un autre liste, et à la fin de ta boucle, tu retires tous ces éléments d'un coup.

Par ailleurs, liste est un tres mauvais nom de variable, puisqu'elle représente les éléménts. Fais plutot directement

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for nb in li:

[PauseKawa]

Bonjour,

C'est bien le remove lors de l'itération (for) qui vous pose problème.
Vous avez le premier élément de votre liste (index 0) qui est 3 et le second (index 1) 5.
Si vous enlevez 3, 5 prend l'index 0 et vous etes en fin de liste (sortie de l'itération/for)
Exemple:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
>>> li = [3, 8, 5, 6]
>>> for liste in li:
...     nb = liste
...     if nb > 7:
...         print" + %d || OK" % (nb)
...     else:
...         print" + %d || FAILED" % (nb)
...     li.remove(nb)
...
 + 3 || FAILED
 + 5 || FAILED

Vous remarquerez le décallage de l'index: Avec le remove de 3 8 prend l'index 0 est n'est pas traité. Lorsque 5 est enlevé on arrive a la fin de la liste

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
>>> print li
[8, 6]

Vous ne pouvez pas (ou plutot ne devez pas) modifier une liste lors de son itération.

@+

[PauseKawa]

Petit coup de main au passage

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
>>> li = [3, 8, 5, 6]
>>> li[:] = [val for val in li if val > 7]
>>> li
[8]

@++

[josmiley]

j'aurai plutôt fait comme ça, puisqu'il faut de toute façon vider la liste ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
>>> li = [3, 8, 5, 6]
>>> while li:
...     nb = li.pop(0)
...     print" + %d || %s" %(nb,'OK' if nb > 7 else 'FAILED')
... 
 + 3 || FAILED
 + 8 || OK
 + 5 || FAILED
 + 6 || FAILED

[PauseKawa]

puisqu'il faut de toute façon vider la liste ...

Bonjour josmiley,

On ne parlais pas de vider la liste, non ?
C'est plutot dans le style

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
>>> li = [3, 8, 5, 3, 6, 8]
>>> map(li.remove, filter(lambda x: x < 7, li))
[None, None, None, None]
>>> li
[8, 8]

[josmiley]

Bonjour josmiley,

On ne parlais pas de vider la liste, non ?

arf, je me suis basé sur ton code ici sans lire que c' était un exemple.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
>>> li = [3, 8, 5, 6]
>>> for nb in li[:]:
...     print" + %d || %s" %(nb,(nb <= 7)and(li.remove(nb)or'FAILED')or'OK')
...     
... 
 + 3 || FAILED
 + 8 || OK
 + 5 || FAILED
 + 6 || FAILED
>>> li
[8]

[jojo971]

Bonjour,

C'est bien le remove lors de l'itération (for) qui vous pose problème.
Vous avez le premier élément de votre liste (index 0) qui est 3 et le second (index 1) 5.
Si vous enlevez 3, 5 prend l'index 0 et vous etes en fin de liste (sortie de l'itération/for)
Exemple:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
>>> li = [3, 8, 5, 6]
>>> for liste in li:
...     nb = liste
...     if nb > 7:
...         print" + %d || OK" % (nb)
...     else:
...         print" + %d || FAILED" % (nb)
...     li.remove(nb)
...
 + 3 || FAILED
 + 5 || FAILED

Vous remarquerez le décallage de l'index: Avec le remove de 3 8 prend l'index 0 est n'est pas traité. Lorsque 5 est enlevé on arrive a la fin de la liste

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
>>> print li
[8, 6]

Vous ne pouvez pas (ou plutot ne devez pas) modifier une liste lors de son itération.

@+

Merci c'est l'explication que je cherchais c'est plutot logique je comprend mieux je voyais d'ou cela pourrais venir