Modificare un dizionario che sto scorrendo

Question

foreach(BruteforceEntry be in Entries.Values) 
{ 
    if (be.AddedTimeRemove <= now) 
     Entries.Remove(be.IPAddress); 
    else if (be.Unbantime <= now && be.Unbantime.Day == DateTime.Now.Day) 
     Entries.Remove(be.IPAddress); 
} 

È stata generata un'eccezione:

Collection è stato modificato; l'operazione di enumerazione potrebbe non essere eseguita.

Per qualche motivo, non lo è più.

So che non è possibile rimuovere qualcosa, mentre iterando attraverso di essa in questo modo. La mia domanda è: come risolverlo?

Answer 1

Non è possibile modificare una raccolta su cui si sta iterando. In questo caso, una soluzione molto migliore sarebbe quella di creare un elenco di voci da rimuovere con l'iterazione di dizionario, e poi iterare che lista, rimuovere le voci dal dizionario:

List<string> removals = new List<string>();      
DateTime now = DateTime.Now; 
foreach(BruteforceEntry be in Entries.Values) 
{ 
    if (be.AddedTimeRemove <= now || 
     (be.Unbantime <= now && be.Unbantime.Day == DateTime.Now.Day)) 
    { 
     removals.Add(be.IPAddress); 
    } 
} 
foreach (string address in removals) 
{ 
    Entries.Remove(address); 
} 

Nota che se si sta utilizzando .NET 3.5, è possibile utilizzare una query LINQ per esprimere la prima parte:

List<string> removals = (from be in Entries.Values 
         where be.AddedTimeRemove <= now || 
           (be.Unbantime <= now && 
           be.Unbantime.Day == DateTime.Now.Day) 
         select be.IPAddress).ToList(); 

Answer 2

Non è possibile modificare la raccolta durante l'iterazione con foreach. Invece, scorrere su di esso con un ciclo for.

Answer 3

È possibile modificare foreach(BruteforceEntry be in Entries.Values)-foreach(BruteforceEntry be in new List<BruteforceEntry>(Entries.Values))

In questo modo non si modifica la vostra collezione, ma ra ne ne copia una copia.

Answer 4

ebollizione il vostro problema ...

foreach(... ... in Entries.Values) 
{ 
     Entries.Remove(...); 

} 

Come gli altri hanno detto si sta modificando l'iteratore, mentre l'iterazione.

È possibile, come dice @David, utilizzare un ciclo for, ma assicurarsi di iniziare alla fine (iterazione inversa).

Answer 5

In poche parole: non è possibile rimuovere una voce da una raccolta mentre si sta iterando su di essa.

Una possibile soluzione consiste nel creare una copia della collezione (ad esempio utilizzando ToList) e iterare che:

foreach(BruteforceEntry be in Entries.Values.ToList()) 
{ 
    // modify the original collection 
} 

Answer 6

Sembra che la tua domanda riguarda il motivo per cui l'eccezione è non essere gettato in cui lo era. Come le altre risposte dichiarano, generalmente non è possibile modificare una collezione attraverso cui si sta iterando, ma si sta iterando sulla raccolta Values, che, credo, è una copia dei valori nel dizionario, piuttosto che fare riferimento alla raccolta di dizionari principale si. Quindi non ha più il problema di iterare e modificare la stessa cosa.

Answer 7

questo (a mio parere) è il modo più semplice:

Dictionary<String, String> A = new Dictionary<string, string>(); //Example Dictionary 
A.Add("A", "A"); //Example Values 
A.Add("B", "B"); 
A.Add("C", "C"); 

for (int i = A.Count - 1; i >= 0; i--) //Loop backwards so you can remove elements. 
{ 
    KeyValuePair<String, String> KeyValue = A.ElementAt(i); //Get current Element. 
    if (KeyValue.Value == "B") A.Remove(KeyValue.Key); 
} 

Nel tuo caso:

for (int i = Entries.Count - 1; i >= 0; i--) 
{ 
    KeyValuePair<String, BruteforceEntry> KeyValue = Entries.ElementAt(i); 
    if (KeyValue.Value.AddedTimeRemove <= now) 
     Entries.Remove(KeyValue.Key); 
    else if (KeyValue.Value.Unbantime <= now && KeyValue.Value.Unbantime.Day == DateTime.Now.Day) 
     Entries.Remove(KeyValue.Key); 
}