Modificar un diccionario que estoy iteración a través de

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); 
} 

generado una excepción:

Collection fue modificado; la operación de enumeración no se puede ejecutar.

Por alguna razón, ya no es así.

Sé que no puede eliminar algo, mientras lo itera de esta manera. Mi pregunta es: ¿cómo lo resuelvo?

Answer 1

No puede modificar una colección sobre la que está iterando. En este caso, una solución mucho mejor sería la creación de una lista de entradas de eliminar mediante la iteración en el diccionario, y luego pasar sobre ese lista, la eliminación de las entradas del diccionario:

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); 
} 

Tenga en cuenta que si que está utilizando .NET 3.5, se puede utilizar una consulta LINQ para expresar la primera 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

No puede modificar la colección al iterar sobre ella con foreach. En cambio, itere sobre él con un bucle for.

Answer 3

puede cambiar foreach(BruteforceEntry be in Entries.Values) a foreach(BruteforceEntry be in new List<BruteforceEntry>(Entries.Values))

de esa manera usted no modifique su colección, pero ra hay una copia de eso.

Answer 4

ebullición abajo de su problema ...

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

} 

Como los otros han dicho que va a modificar el iterador mientras que la iteración.

Puede, como dijo @David, utilizar un bucle for en su lugar, pero asegúrese de comenzar al final (iteración inversa).

Answer 5

En pocas palabras: no se puede eliminar una entrada de una colección mientras se itera sobre ella.

Una posible solución es crear una copia superficial de la colección (por ejemplo, utilizando ToList) y pasar sobre ese:

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

Answer 6

Parece que tu pregunta es acerca de por qué una excepción es no ser lanzado en solía ser. Como dicen las otras respuestas, generalmente no puede cambiar una colección a través de la cual está iterando, pero está iterando sobre la colección Values, que, en mi opinión, es una copia de los valores en el diccionario, en lugar de referirse a la colección principal del diccionario sí mismo. Entonces ya no tiene el problema de iterar y modificar lo mismo.

Answer 7

Este (en mi opinión) es la forma más fácil:

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); 
} 

En su 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); 
}