Discussion :

[TSalm]

Bonjour,

Je parcours un Dictionnary et je veux supprimer des élements dans certains cas.
Mais une Exception "InvalidoperationException" est levée (La collection a été modifiée ; l'opération d'énumération peut ne pas s'exécuter.)

Le 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
 
foreach (KeyValuePair<string, string> pair in dicAttached) {
                // Si séparé par des $ , => c'est une variable multiple
                string[] clefSplit = pair.Key.Split('$');
 
                if (clefSplit.Length > 1) { 
 
                    // Retire la clef de colonne du dictionnaire
                    dicAttached.Remove(pair.Key);
 
                    // Et ajoute la variable si elle n'est pas déjà dedans
 
                    string nomVariable = clefSplit[clefSplit.Length -1]; // prends le dernier élement
 
                    if ( ! dicAttached.ContainsKey(nomVariable) ) dicAttached.Add(nomVariable, null);
 
                }
 
            }

Comment modifier une un dictionnaire tout en le parcourant ?
D'avance merci,
TSalm

[SirJulio]

Salut,

pour repondre simplement, tu ne peux pas, il faut faire le traitement en deux temps. D'abord, tu enregistres la clé de tous les elements correspondants, et ensuite tu pars de cette base pour modifier ton dictionary<>.

[AP]

Tout à fait, il faut faire cela en 2 étapes.
- Tu crées un liste pour stoquer les éléments à supprimer
- Tu parcours ton dictionnaire et tu ajoutes à ta liste les éléments à supprimer
- Tu parcours ta liste et tu supprimes de ton dico les éléments
- tu vides ou supprimes ta liste

[TSalm]

ok merci

[cinemania]

Note : ceci est valable pour toute collection que tu parcours avec un foreach.

Tu peux cependant procéder autrement et récupèrer un tableau avec les clés de ton dictionary et ensuite faire un foreach dans ce tableau obtenu.

Cette solution permet de masqué l'étape en deux temps et permet de diminuer la mémoire locale si tu ne te contente pas de faire des suppressions mais également des ajouts ou modifications, car tu ne met pas ces modifications en mémoire pour traitement dans phase ultérieure.

Attention toutefois à la façon dont tu gere le tableau des clés.

Si tu fait
foreach (TKey key in mydic.Keys)
{
}

tu aboutiera exactement au meme probleme...

(une des raisons et que lorsque tu demande une énumération ils utilisent yield, ce qui oblige a respecter une assertion, qui est que la liste énumérée ne doit pas varier pendant qu'on effectue l'énumération.)

il faut récupérer la collection et la stocker dans une liste annexe.
en contrepartie le traitement de ton dictionary sera un peu plus long mais pourra économiser de la mémoire de traitement.

Il n'y a pas de mystère, quand on veut gagner du temps on perd de la mémoire et quand on veut gagner de la mémoire, on perd du temps. C'est l'un ou l'autre, pas les deux.