集合已被修改；枚举操作可能不会执行。

Question

集合已被修改；枚举操作可能不会执行。

我无法找到这个错误的底部，因为当附加调试器时，它似乎不会发生。下面是密码。

这是Windows服务中的WCF服务器。每当有数据事件时，服务都会调用NotifySubscriber方法(随机间隔，但不经常-每天大约800次)。

当Windows窗体客户端订阅时，订阅者ID将添加到订阅者字典中，而当客户端取消订阅时，它将从字典中删除。当客户端取消订阅时(或之后)发生错误。似乎下一次调用NotifySubscriber()方法时，foreach()循环会失败，主题行中的错误会导致失败。该方法将错误写入应用程序日志，如下代码所示。当附加调试器和客户端取消订阅时，代码执行良好。

你看到这个代码有问题了吗？我需要让字典线程安全吗？

[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

可能发生的是SignalData在循环期间间接地更改订阅者字典并导致消息。您可以通过更改foreach(Subscriber s in subscribers.Values)到foreach(Subscriber s in subscribers.Values.ToList())如果我是对的，问题就会消失调用订阅者，值。ToList()将订阅者的值复制到Foreach开头的单独列表中。其他任何东西都无法访问这个列表(它甚至没有变量名！)，所以在循环中没有任何东西可以修改它。

Answer 2

当订阅服务器取消订阅时，您将在枚举期间更改订阅服务器集合的内容。有几种方法可以解决这一问题，其中一种方法是更改for循环以使用显式.ToList():public void NotifySubscribers(DataRecord sr)  {     foreach(Subscriber s in subscribers.Values.ToList())     {                                               ^^^^^^^^^           ...

Answer 3

您还可以锁定订阅者字典，以防止当它被循环时被修改： lock (subscribers)  {          foreach (var subscriber in subscribers)          {                //do something          }  }