Die Auflistung wurde geändert, der Enumerationsvorgang kann nicht ausgeführt werden

Habe ich multithreads Anwendung, und ich bekomme diese Fehlermeldung

************** Exception Text **************
System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
   at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource)
   at System.Collections.Generic.List`1.Enumerator.MoveNextRare()
   at System.Collections.Generic.List`1.Enumerator.MoveNext()
   ...

Wahrscheinlich habe ich problem mit meiner Sammlung, weil in einem thread Las ich, meine Sammlung und meine in einem anderen thread, die ich ändern Sammlung.

public readonly ObservableCollectionThreadSafe<GMapMarker> Markers = new ObservableCollectionThreadSafe<GMapMarker>();


public void problem()
{
  foreach (GMapMarker m in Markers)
  {
    ...
  }
}

Ich versuche zu sperren, die Sammlung mit diesem code, funktioniert aber nicht.

public void problem()
    {
       lock(Markers)
       {
         foreach (GMapMarker m in Markers)
         {
           ...
         }
       }
    }

Irgendwelche Ideen um dieses problem zu beheben?

  • Problem ist mit dem code in der foreach, poste es bitte.
  • Sie nicht ändern Sammlung, während die Schleife mit foreach
InformationsquelleAutor PATO7 | 2012-03-29
  1. 11

    Dies ist ein ziemlich häufiger Fehler - ändern einer collection während Iteration es mit foreach im Auge behalten, dass foreach verwendet readonly IEnumerator Instanz.

    Versuchen, um eine Schleife durch die Sammlung mit for() mit einem zusätzlichen index überprüfen, so dass, wenn der index außerhalb der Grenzen, die Sie würde in der Lage sein, um zusätzliche Logik zu behandeln. Sie können auch die LINQ ist Count() als weitere Bedingung zum beenden der Schleife durch die Auswertung der Count Wert jedes mal, wenn die zugrunde liegende enumeration implementiert nicht ICollection:

    Wenn Markers implementiert IColletion - lock auf SyncRoot:

    lock (Markers.SyncRoot)

    Verwenden for():

    for (int index = 0; index < Markers.Count(); index++)
{
    if (Markers>= Markers.Count())
    {
       //TODO: handle this case to avoid run time exception
    }
}

    Vielleicht finden Sie diesen Beitrag hilfreich: Wie foreach-Schleifen arbeiten, die in C#?

    • aber wenn der Umbau war durch das löschen eines Elements aus der Sammlung,, die werfen ein IndexOutOfRange Ausnahme
    • Ich erwähnte extra-index überprüfen, um dieses Problem zu vermeiden, fügen Beispiel, danke für den Hinweis zu dieser
    • ich denken, zu ersetzen foreach mit, aber dann dachte ich, wenn der besser wäre, zu sperren, die Sammlung, aber nicht arbeiten :/
    • bitte teilen Sie einen code, der modifiziert Sammlung
    • int i = mapMarkers.Marker.IndexOf(oldMarker); if (i != -1) { mapMarkers.Marker[i] = newMarker; }
    • Da ObservableCollectionThreadSafe ist eine eigene Sammlung Klasse, bitte zeigen Sie code zum Hinzufügen/Entfernen, BTW haben Sie eine Sperre this.SyncRoot innen-Add/Remove-Methoden?
    • Nein, ich habe nicht alles sperren
    • ausprobieren lock(this.SyncRoot) { this.Add(item); } innerhalb einer ObservableCollectionThreadSafe Klasse, das gleiche für Remove()
    • GMap.NET.ObjectModel.ObservableCollectionThreadSafe<T> enthält keine definition für 'SynRoot...
    • Hat es geerbt von ObservableCollection? Wenn ja, beheben syncRoot zum lokalen Feld this.syncRoot = (this as ICollection).SyncRoot seit ICollection explizit umgesetzt werden
    • Yep "zurücksetzen", um eine gewöhnliche for-Schleife hat den trick für mich zu.

    InformationsquelleAutor sll
  2. 4

    Gesperrt werden müssen, sowohl auf das Lesen und das schreiben-Seite. Ansonsten einer der threads, die nicht wissen, über das Schloss und versuchen zu Lesen/ändern der Sammlung, während der andere zu verändern/Lesen (bzw.) mit der Sperre

    InformationsquelleAutor Attila
  3. 4

    Versuchen zu Lesen, einen Klon Ihrer Sammlung

    foreach (GMapMarker m in Markers.Copy())
{
   ...
}

    dadurch wird eine neue Kopie von Ihrer Sammlung, die nicht beeinflusst werden durch einen anderen thread, aber kann dazu führen, ein performance-Problem bei der riesigen Sammlung.

    Also ich denke es wird besser sein, wenn Sie Verriegelte die Sammlung beim Lesen und schreiben Prozesse.

    • ...und ändern der ursprünglichen Sammlung.
    • Sie haben Recht, ich denke zu verwenden .Copy aber kann es zu einem performance-Problem.
    InformationsquelleAutor Amir Ismail
  4. 0

    Können Sie verwenden Sie eine foreach-aber Sie haben zu werfen, die Sammlung auf eine Liste und verwenden Sie den Punkt-operator zum Zugriff auf das Verhalten von Methoden.

    Beispiel: Marker.Tolist().ForEach(ich => ich.DeleteObject())

    Nicht ganz sicher, was du tust mit deiner Sammlung. Mein Beispiel ist vorausgesetzt, Sie wollte nur, um zu löschen Sie alle Elemente aus der Auflistung, aber es kann angewendet werden, um jedes Verhalten, das Sie versuchen zu tun, mit Ihrer Sammlung.

    InformationsquelleAutor Tim James
  5. 0

    Ich löste dieses problem, indem

    var data = getData();
lock(data)
{
    return getData().Select(x => new DisplayValueModel(x));
}

    statt

    return getData().Select(x => new DisplayValueModel(x));
    InformationsquelleAutor erionpc
Schreibe einen Kommentar