2

我试图通过开发一个通用的弱事件侦听器来节省时间，我可以只传递一个 Action 来采取。在未注册之前，一切似乎都很好。这似乎将它们全部注销。我感到困惑的是为什么，这与将其传递给IWeakEventListener 参数有何不同？

  public class GenericWeakEventListener : IWeakEventListener
  {
    #region EventAction

    /// <summary>
    /// Action to take for the event
    /// </summary>
    private Action<Type, object, EventArgs> _eventAction;

    /// <summary>
    /// Gets or sets the action to take for the event
    /// </summary>
    //[DataMember]
    public Action<Type, object, EventArgs> EventAction
    {
      get
      {
        return _eventAction;
      }

      private set
      {
        if (EventAction != value)
        {
          _eventAction = value;
        }
      }
    }

    #endregion EventAction

    #region Constructors

    public GenericWeakEventListener(Action<Type, object, EventArgs> action)
    {
      EventAction = action;
    }

    #endregion Constructors

    #region Public Methods

    public bool ReceiveWeakEvent(Type managerType, object sender, EventArgs e)
    {
      if (EventAction != null)
      {
        EventAction(managerType, sender, e);
      }

      return true;
    }

    #endregion Public Methods
  }

编辑：

这是监听器代码：

  public class SomeClient
  {
    public int ID { get; set; }

    private Timer timer = null;
    private Timer timer2 = null;

    public SomeClient(int id, SomeService service)
    {
      ID = id;
      //EventHandler<GenericEventArgs<string>> d = (o, s) => Console.WriteLine("Client {0}: {1}", ID, s.Item);
      if (service != null) SomeEventChangedEventManager.AddListener(service, new GenericWeakEventListener((t, s, e) => { Console.WriteLine("SomeEvent: " + ID); }));

      timer = new Timer { AutoReset = true, Interval = 1000 };
      SomeTimerElapsedEventManager.AddListener(timer, new GenericWeakEventListener((t, s, e) => { Console.WriteLine("SomeTimer: " + ID); }));
      timer.Start();
    }
  }

这是发布者的代码：

  public class SomeService
  {
    public event EventHandler<GenericEventArgs<string>> SomeEvent;

    public SomeService()
    {
      System.Timers.Timer timer = new Timer { AutoReset = true, Interval = 1000 };
      timer.Elapsed += (sender, args) => { if (SomeEvent != null) SomeEvent(this, new GenericEventArgs<string>(Guid.NewGuid().ToString())); };
      timer.Start();
    }
  }

这是来自 main 方法的代码：

public static void Main(string[] args)
{
  SomeService service = new SomeService();
  List<SomeClient> clients = new List<SomeClient>();

  // Build clients
  for (int x = 0; x < 5; x++)
  {
    clients.Add(new SomeClient(x + 1, service));
  }

  System.Timers.Timer timer = new Timer { AutoReset = true, Interval = 5000 };
  timer.Elapsed += (s, a) =>
    {
      if (clients.Count == 0)
      {
        return;
      }

      Console.WriteLine("Removing\r\n");
      clients.RemoveAt(0);
      GC.Collect();
    };
  timer.Start();

  Console.ReadLine();
}

这是输出： SomeEvent: 1 SomeEvent: 2 SomeEvent: 3 SomeEvent: 4 SomeEvent: 5 SomeTimer: 2 SomeTimer: 3 SomeTimer: 4 SomeTimer: 1 SomeTimer: 5 SomeEvent: 1 SomeEvent: 2 SomeEvent: 3 SomeEvent: 4 SomeEvent: 5 SomeTimer: 1 SomeTimer: 2 SomeTimer: 3 SomeTimer: 4 SomeTimer: 5 SomeEvent: 1 SomeEvent: 2 SomeEvent: 3 SomeEvent: 4 SomeEvent: 5 SomeTimer: 2 SomeTimer: 3 SomeTimer: 4 SomeTimer: 5 SomeTimer: 1 SomeEvent: 1 SomeEvent: 2 SomeEvent: 3 SomeEvent: 4 SomeEvent: 5 SomeTimer: 1 SomeTimer: 2 SomeTimer: 3 SomeTimer: 5 SomeTimer: 4 SomeEvent: 1 SomeEvent: 2 SomeEvent: 3 SomeEvent: 4 SomeEvent: 5 移除

如果没有通用的弱事件侦听器，那么输出会在没有 1 的情况下继续输出，然后在没有 2 的情况下继续输出，依此类推。

4

1 回答 1

3

从 OP 编辑​​编辑：知道了。这是我认为正在发生的事情：

您正在声明 SomeClient 拥有的 lambda 并包含实例变量 ID 的外部闭包。这使得通过 SomeEventChangedEventManager 作为委托传递给 SomeService 上的事件的 lambda 依赖于该 ID 实例的持续存在。

当您删除该 SomeClient 实例时，该 lambda 所需的 ID 变量将超出范围并被 GCed。但是，我没有看到这段代码的任何部分从 SomeService 的 SomeEvent 的处理程序中删除了这个 lambda。因此，lambda 保留在内存中作为对匿名委托的引用，但它所依赖的其他数据现在已经消失了。这会导致运行时抛出异常，该异常会以某种方式被吞没并且不会导致整个程序崩溃。

但是，该事件基本上按照附加的顺序执行处理程序委托（这是他们通常告诉您忽略的实现细节），因为其中一个处理程序已被抛出，所以该事件已停止执行。这使得删除第一个客户端似乎已经删除了所有这些客户端，而实际上这些客户端的处理程序只是因为第一个处理程序出错而没有执行。

修复是双重的：

  • 定义 lambda 并将其存储为 SomeClient 的实例变量。这允许您保留对它的引用，这很重要，因为在确定相等性时不会对委托进行语义比较，因此以下代码不起作用：

    SomeEvent += (a,b,c) => Foo(a,b,c);
//the following line will not remove the handler added in the previous line,
//because the two lambdas are compiled into differently-named methods
//and so this is a different reference to a different method.
SomeEvent -= (a,b,c) => Foo(a,b,c);

  • 在 SomeClient 上实现 IDisposable 和/或终结器。当从列表中删除客户端时，由 GC 调用的处理程序/终结器应该从 SomeEvent 侦听器中删除此实例的 lambda（可能通过 Manager 上的 RemoveListener() 方法）。因为您保留了对指向确切添加内容的委托的引用，所以处理程序将被删除，因此不会被执行并且不会出错。

于 2012-05-03T16:07:45.883 回答