Discussion :

[BnouK]

Bonjour,

Débutant en Python, je cherche un moyen propre de faire ça :

Code :

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

1
2
3
4
 
for j in range(len(maliste)):
    if maliste[j].test()==True : 
        maliste.remove(maliste[j])

Propre, parce que dans la mesure où je vais modifier la taille de ma liste en cours de route, il n'est évidemment pas approprié d'utiliser la taille de cette même liste dans ma boucle for.
Est-ce qu'il existe une astuce toute bête en Python ou dois-je passer par une seconde liste ?

[fred1599]

Bonjour,

Je ne comprend pas d'où vient ton attribut test()?

[BnouK]

Rien à voir en fait,
Dans la pratique il s'agit d'une liste d'objets issus d'une classe maison, pour laquelle j'ai une méthode qui s'appelle test qui renvoie un booléen.

[fred1599]

Voir peut-etre du coté de filter qui me semble plus beau

exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
>>> def odd(n):                 1
...     return n % 2
...     
>>> li = [1, 2, 3, 5, 9, 10, 256, -3]
>>> filter(odd, li)             2
[1, 3, 5, 9, -3]
>>> [e for e in li if odd(e)]   3
>>> filteredList = []
>>> for n in li:                4
...     if odd(n):
...         filteredList.append(n)
...     
>>> filteredList
[1, 3, 5, 9, -3]

source : http://python.developpez.com/cours/D...ring_lists.php

[alexdevl]

Bonjour
On peut utiliser pop ou une list comprehension mais il faut itérer sur les objets pas sur l'indice

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
 
class Cl:
    "Classe exemple avec un attribut"
    def __init__(self,param):
        self.at=param
 
def lstcre():
    "Creation de la liste d'objet"
    lst=[]
    for i in range(5):
        lst.append(Cl(i))
    return lst
 
def aff(lst):
    "Affichage de la liste d'objet"
    for cl in lst:
        print cl,cl.at
    print
 
print "a) avec enumerate et pop"
lst=lstcre()
aff(lst)
for i,l in enumerate(lst):
    if l.at in [1,3] : lst.pop(i) 
aff(lst)
 
print "b) avec filter"
lst=lstcre()
aff(lst)
def sup13(el):
       return el.at not in [1,3]
lst=filter(sup13,lst)
aff(lst)
 
print "c) Avec une list comprehension"
lst=lstcre()
aff(lst)
lst=[el for el in lst if el.at not in [1,3]]
aff(lst)

[BnouK]

Merci beaucoup.
L'astuce avec enumerate fonctionne à merveille, c'est tout à fait ce que je cherchais.

[shalyd]

Bonjour,

je me permet de faire la remarque suivante, au lieu de

if maliste[j].test()==True :

il est préférable d'utiliser directement "if maliste[j].test():"

[GnuVince]

Utilise la fonction filter() ou une compréhension de liste:

Code :

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

1
2
3
4
5
6
 
# Filter
print filter(lambda o: not o.test(), maliste)
 
# Compréhension
print [o for o in maliste if not o.test()]

Tu peux réassigner le résultat dans maliste pour avoir une nouvelle liste avec seulement les éléments désirés.

[eyquem]

Le problème résidant dans le fait que, si on élimine des éléments de la liste, l'index va se trouver vers la fin du parcours de range(len(maliste)) hors de la dimension de la liste raccourcie, provoquant ainsi une erreur

IndexError: list index out of range

et non pas dans le fait que range(len(maliste)) verrait la valeur len(maliste)varier en cours d'exécution, ce qui n'a évidemment aucun sens puisque range(len(maliste)) utilise la valeur len(maliste) une seule fois, pour créer la liste stable range(len(maliste)) avant de rentrer dans la boucle,

il suffit de mettre un try except pour traiter le problème:

Code :

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

1
2
3
4
for j in range(len(maliste)):
    try:
        if maliste[j].test():    maliste.remove(maliste[j])
    except IndexError:    break

On peut aussi contourner le problème en parcourant la liste de la fin vers le début:

Code :

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

1
2
3
4
ranj = range(len(maliste))
ranj.reverse()
for j in ranj:
    if maliste[j].test():    maliste.remove(maliste[j])

Python, c'est agile et concis !