Discussion :

[bcodp]

Bonjour à tous,
Je réalise un petit script pour récupérer des informations sur un gitlab.
Je stock les noms des différents projets git dans une liste. Je la parcours ensuite pour trouver tout les projets contenant le mot "wiki" et les supprimer de la liste.
Mon code fonctionne bien mais laisse passer un projet "wiki".

Code :

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

1
2
3
4
5
print(projectsName)
for projet in projectsName:
        if "wiki.git" in projet:
            projectsName.remove(projet)
print(projectsName)

Les output avant apres:

Code :

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

1
2
['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_3.wiki.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_2.wiki.git']

Je débute en python, il est possible qu'il me manque une partie du raisonnement mais je ne comprends pas pourquoi j'obtiens ce résultat.
Merci de vos suggestions

[VinsS]

Salut,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
>>> w = ['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_3.wiki.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
>>> for i in w:
...     print(w)
...     if 'wiki.git' in i:
...             w.remove(i)
... 
['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_3.wiki.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_3.wiki.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_3.wiki.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_3.wiki.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
>>> w
['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_2.wiki.git']

Lorsque le quatrième objet 'RD_3.wiki.git' est supprimé, l'itération pointe sur le cinquième objet et tu peux voir que c'est maintenant 'RD_1.wiki.git'.

Règle générale: ne jamais modifier un itérable en cours d'itération.

Crée d'abord un liste des wiki et ensuite utilise cette liste pour ton remove.

[bistouille]

Salut.

Cela vient du fait que tu supprimes des éléments dans la liste sur laquelle tu es en train d'itérer,

Code :

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

1
2
3
4
5
6
7
>>> l = ['a1', 'b1', 'a2', 'a3']
>>> for n in l :
...     if 'a' in n :
...         l.remove(n)
... 
>>> l
['b1', 'a3']

C'est pour cette raison que tu dois itérer sur une copie de ta liste, ou reconstruire une nouvelle liste.

Grillé par VinsS

[tyrtamos]

Bonjour,

Un manière simple de faire une boucle avec une copie de la liste est:

Code :

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

1
2
3
4
for projet in projectsName[:]:
        if "wiki.git" in projet:
            projectsName.remove(projet)
print(projectsName)

C'est le "[:]" qui fait la copie.

[bcodp]

Merci pour vos réponses j'ai compris mon erreur.
La solution de tyrtamos pour effectuer la copie fonctionne parfaitement.

[fred1599]

L'autre solution, c'est créer une liste avec seulement les éléments souhaités

Code :

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

1
2
3
>>> w = ['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_3.wiki.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
>>> [projet for projet in w if 'wiki.git' not in projet]
['RD_2.git', 'RD_1.git', 'RD_3.git']

[lg_53]

On peut aussi s'en sortir en parcourant la liste à l'envers. Ainsi lorsqu'on supprime un élément, il y a un décalage des éléments qui sont après dans la liste, mais comme on la parcours à l'envers on a déjà traité la fin de la liste. Donc la partie décalé on y retouche pas et le début lui reste inchangé et prêt à traiter.

Code :

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

1
2
3
4
5
6
projectsName=['RD_2.git', 'RD_1.git', 'RD_3.git', 'RD_3.wiki.git', 'RD_2.wiki.git', 'RD_1.wiki.git']
print(projectsName)
for projet in reversed(projectsName):
        if "wiki.git" in projet:
            projectsName.remove(projet)
print(projectsName)