Discussion :

[lecknaat]

Bonjour,

J'ai une liste de nom de fichier. Je souhaite parcourir cette liste pour renommer certains des fichiers en fonction de règles que j'ai défini. Lorsque je boucle sur ma liste et que je renomme un des éléments, ma boucle se coupe...

Voici mon code :

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
 
Public Structure structNommage
        Dim tabOld() As String
        Dim newNom As String
End Structure
Dim ListFichierACopier As New List(Of String)
...
...
...
 
Public Sub Maj_tabFichACopier()
        Dim i As Integer = 0
        For Each nom As String In ListFichierACopier
            For Each nommage As structNommage In TabNommage 
                For Each old As String In nommage.tabOld
                    If nom = old Then
                        nom = nommage.newNom
                        ListFichierACopier(i) = nom
                        GoTo trouvé2
                    End If
                Next
            Next
trouvé2:
            i = i + 1
        Next
    End Sub

Pouvez-vous m'indiquer pourquoi ma boucle ne fonctionne pas ? J'ai également essayé de passer par une liste temporaire (que j'initialise avec le contenue de la liste à traiter) qui contiendrais mes noms de fichiers modifiés afin de ne pas altérer la liste sur laquelle je boucle mais cela ne change rien ! Le pire, c'est qu'en faisant évoluer ma liste temporaire, la liste principale évolue aussi...

Je ne dois absolument pas me servir des list correctement !

Merci d'avance !

[DonQuiche]

Bonjour.

Une variable itérée ("nom") par une boucle foreach ne doit pas être modifiée par le code car cela lève une exception. La solution est d'utiliser des indices, comme tu l'avais deviné. Il faut donc assigner la nouvelle valeur directement à ListFichierACopier(i), sans modifier "nom" au passage. S'il est important pour la suite de la boucle que "nom" ait aussi la nouvelle valeur, alors il faudra ajouter une nouvelle variable nomModifié ou quelque chose dans ce goût-ci.

Enfin je ne suis pas familier du VB mais n'y aurait-il pas un problème avec cette ligne : If nom = old Then ? S'agit-il bien d'une comparaison ?

[lecknaat]

Ha oui merci ! Effectivement j'avais bien compris que je ne devais pas modifier le contenu de ma liste mais j'vais pas fait gaffe à la variable "nom" ! En passant par la liste temporaire et en utilisant simplement la ligne :

Code :

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

ListFichierACopier(i) = nommage.newNom

cela fonctionne !

En ce qui concerne la ligne

Code :

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

if nom = old

, c'est bien une comparaison mais je ne vois pas ou est le problème ?

[DonQuiche]

En ce qui concerne la ligne

Code :

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

if nom = old

, c'est bien une comparaison mais je ne vois pas ou est le problème ?

Comme je le disais je ne suis pas familier de VB et je n'étais donc pas certain que ce soit une comparaison plutôt qu'une assignation. Puisque tu confirmes que c'en est une, aucun problème.

[lecknaat]

ok merci !