Sammlung wurde geändert; Aufzählungsoperation darf nicht ausgeführt werden

Question

Ich kann diesem Fehler nicht auf den Grund gehen, denn wenn der Debugger angeschlossen ist, scheint er nicht aufzutreten.

Sammlung wurde geändert; Aufzählungsoperation darf nicht ausgeführt werden

Nachstehend finden Sie den Code.

Dies ist ein WCF-Server in einem Windows-Dienst. Die Methode NotifySubscribers() wird vom Dienst immer dann aufgerufen, wenn ein Datenereignis vorliegt (in zufälligen Abständen, aber nicht sehr oft - etwa 800 Mal pro Tag).

Wenn ein Windows Forms-Client ein Abonnement abschließt, wird die Abonnenten-ID zum Abonnentenverzeichnis hinzugefügt, und wenn der Client das Abonnement abbricht, wird sie aus dem Verzeichnis gelöscht. Der Fehler tritt auf, wenn (oder nachdem) ein Client sich abmeldet. Es scheint, dass das nächste Mal, wenn der NotifySubscribers() Methode aufgerufen wird, wird die foreach() Schleife schlägt mit dem Fehler in der Betreffzeile fehl. Die Methode schreibt den Fehler in das Anwendungsprotokoll, wie im folgenden Code gezeigt. Wenn ein Debugger angeschlossen ist und ein Kunde sich abmeldet, wird der Code ordnungsgemäß ausgeführt.

Sehen Sie ein Problem mit diesem Code? Muss ich das Wörterbuch thread-sicher machen?

[ServiceBehavior(InstanceContextMode=InstanceContextMode.Single)]
public class SubscriptionServer : ISubscriptionServer
{
    private static IDictionary<Guid, Subscriber> subscribers;

    public SubscriptionServer()
    {            
        subscribers = new Dictionary<Guid, Subscriber>();
    }

    public void NotifySubscribers(DataRecord sr)
    {
        foreach(Subscriber s in subscribers.Values)
        {
            try
            {
                s.Callback.SignalData(sr);
            }
            catch (Exception e)
            {
                DCS.WriteToApplicationLog(e.Message, 
                  System.Diagnostics.EventLogEntryType.Error);

                UnsubscribeEvent(s.ClientId);
            }
        }
    }

    public Guid SubscribeEvent(string clientDescription)
    {
        Subscriber subscriber = new Subscriber();
        subscriber.Callback = OperationContext.Current.
                GetCallbackChannel<IDCSCallback>();

        subscribers.Add(subscriber.ClientId, subscriber);

        return subscriber.ClientId;
    }

    public void UnsubscribeEvent(Guid clientId)
    {
        try
        {
            subscribers.Remove(clientId);
        }
        catch(Exception e)
        {
            System.Diagnostics.Debug.WriteLine("Unsubscribe Error " + 
                    e.Message);
        }
    }
}

Answer 1

Was wahrscheinlich passiert ist, dass SignalData während der Schleife indirekt das Abonnentenwörterbuch ändert und zu dieser Meldung führt. Sie können dies überprüfen, indem Sie das

foreach(Subscriber s in subscribers.Values)

An

foreach(Subscriber s in subscribers.Values.ToList())

Wenn ich Recht habe, wird das Problem verschwinden.

Aufruf von subscribers.Values.ToList() kopiert die Werte von subscribers.Values in eine separate Liste am Anfang der foreach . Nichts anderes hat Zugriff auf diese Liste (sie hat nicht einmal einen Variablennamen!), so dass sie innerhalb der Schleife nicht verändert werden kann.

Answer 2

Wenn ein Abonnent sich abmeldet, ändern Sie den Inhalt der Abonnentensammlung während der Aufzählung.

Es gibt mehrere Möglichkeiten, dies zu beheben. Eine davon besteht darin, die for-Schleife so zu ändern, dass sie eine explizite .ToList() :

public void NotifySubscribers(DataRecord sr)  
{
    foreach(Subscriber s in subscribers.Values.ToList())
    {
                                              ^^^^^^^^^  
        ...

Answer 3

Effizienter ist es meiner Meinung nach, eine weitere Liste zu erstellen, in die Sie alles eintragen, was "entfernt werden soll". Nachdem Sie Ihre Hauptschleife beendet haben (ohne .ToList()), führen Sie eine weitere Schleife über die "zu entfernende" Liste durch und entfernen jeden Eintrag, sobald er auftritt. In Ihrer Klasse fügen Sie also hinzu:

private List<Guid> toBeRemoved = new List<Guid>();

Dann ändern Sie es in:

public void NotifySubscribers(DataRecord sr)
{
    toBeRemoved.Clear();

    ...your unchanged code skipped...

   foreach ( Guid clientId in toBeRemoved )
   {
        try
        {
            subscribers.Remove(clientId);
        }
        catch(Exception e)
        {
            System.Diagnostics.Debug.WriteLine("Unsubscribe Error " + 
                e.Message);
        }
   }
}

...your unchanged code skipped...

public void UnsubscribeEvent(Guid clientId)
{
    toBeRemoved.Add( clientId );
}

Dies löst nicht nur Ihr Problem, sondern verhindert auch, dass Sie immer wieder eine Liste aus Ihrem Wörterbuch erstellen müssen, was bei einer großen Anzahl von Abonnenten teuer ist. Vorausgesetzt, die Liste der zu entfernenden Abonnenten ist bei jeder Iteration kleiner als die Gesamtzahl der Abonnenten in der Liste, sollte dies schneller gehen. Aber Sie können natürlich gerne ein Profil erstellen, um sicherzugehen, dass dies in Ihrer speziellen Situation der Fall ist.

Answer 4

Warum dieser Fehler?

Im Allgemeinen unterstützen .Net-Sammlungen nicht die gleichzeitige Aufzählung und Änderung. Wenn Sie versuchen, die Sammlungsliste während der Aufzählung zu ändern, löst dies eine Ausnahme aus. Das Problem, das sich hinter diesem Fehler verbirgt, ist also, dass wir die Liste/das Wörterbuch nicht ändern können, während wir eine Schleife durch dieselbe ziehen.

Eine der Lösungen

Wenn wir ein Wörterbuch anhand einer Liste seiner Schlüssel durchlaufen, können wir parallel dazu das Wörterbuchobjekt ändern, da wir durch die Schlüsselsammlung und nicht durch das Wörterbuch iterieren und nicht das Wörterbuch (und dessen Schlüsselsammlung) durchlaufen.

Beispiel

//get key collection from dictionary into a list to loop through
List<int> keys = new List<int>(Dictionary.Keys);

// iterating key collection using a simple for-each loop
foreach (int key in keys)
{
  // Now we can perform any modification with values of the dictionary.
  Dictionary[key] = Dictionary[key] - 1;
}

Hier ist ein Blogbeitrag über diese Lösung.

Und für einen tieferen Einblick in StackOverflow: Warum tritt dieser Fehler auf?

Answer 5

Okay, was mir geholfen hat, war, rückwärts zu iterieren. Ich habe versucht, einen Eintrag aus einer Liste zu entfernen, aber die Iteration nach oben und es vermasselt die Schleife, weil der Eintrag nicht mehr existiert:

for (int x = myList.Count - 1; x > -1; x--)
{
    myList.RemoveAt(x);
}