一尘不染

是否有必要在C#中显式删除事件处理程序

c#

我有一堂课,提供一些活动。该类是全局声明的,但不能在该全局声明上实例化-可以根据需要在需要它的方法中实例化该类。

每次在方法中需要该类时,都会实例化该类并注册事件处理程序。在方法超出范围之前是否有必要显式删除事件处理程序?

当方法超出范围时,类的实例也将超出范围。将事件处理程序注册到超出范围的那个实例中是否留下了内存占用隐患?(我想知道事件处理程序是否使GC不再看到类实例不再被引用。)


阅读 307

收藏
2020-05-19

共1个答案

一尘不染

就您而言,一切都很好。它是 发布 事件的 对象 ,使事件处理程序的 目标 保持活动状态。所以,如果我有:

publisher.SomeEvent += target.DoSomething;

然后publisher引用target但并非相反。

在您的情况下,发布者将有资格进行垃圾回收(假定没有其他引用),因此它与事件处理程序目标的引用无关。

棘手的情况是发布者的寿命很长,但订阅者却不想这样做-在 这种
情况下,您需要取消订阅处理程序。例如,假设您有一些数据传输服务,该服务使您可以订阅有关带宽更改的异步通知,并且该传输服务对象是长期存在的。如果我们这样做:

BandwidthUI ui = new BandwidthUI();
transferService.BandwidthChanged += ui.HandleBandwidthChange;
// Suppose this blocks until the transfer is complete
transferService.Transfer(source, destination);
// We now have to unsusbcribe from the event
transferService.BandwidthChanged -= ui.HandleBandwidthChange;

(您实际上希望使用finally块来确保您不泄漏事件处理程序。)如果我们不取消订阅,则BandwidthUI至少将与传输服务一样长。

就我个人而言,我很少碰到这种情况-例如,如果我订阅某个事件,则该事件的目标的生存时间至少与发布者的生存时间相同-
例如,表单会持续到其上的按钮存在的时间。值得知道这个潜在的问题,但是我认为有些人在不需要时会担心它,因为他们不知道引用的方向。

编辑:
这是为了回答乔纳森·迪金森的评论。首先,查看Delegate.Equals(object)的文档,该文档清楚地给出了相等行为。

其次,这是一个简短但完整的程序,用于显示取消订阅的工作:

using System;

public class Publisher
{
    public event EventHandler Foo;

    public void RaiseFoo()
    {
        Console.WriteLine("Raising Foo");
        EventHandler handler = Foo;
        if (handler != null)
        {
            handler(this, EventArgs.Empty);
        }
        else
        {
            Console.WriteLine("No handlers");
        }
    }
}

public class Subscriber
{
    public void FooHandler(object sender, EventArgs e)
    {
        Console.WriteLine("Subscriber.FooHandler()");
    }
}

public class Test
{
    static void Main()
    {
         Publisher publisher = new Publisher();
         Subscriber subscriber = new Subscriber();
         publisher.Foo += subscriber.FooHandler;
         publisher.RaiseFoo();
         publisher.Foo -= subscriber.FooHandler;
         publisher.RaiseFoo();
    }
}

结果:

Raising Foo
Subscriber.FooHandler()
Raising Foo
No handlers

(在Mono和.NET 3.5SP1上测试。)

进一步编辑:

这是为了证明可以在仍然有订阅者引用的情况下收集事件发布者。

using System;

public class Publisher
{
    ~Publisher()
    {
        Console.WriteLine("~Publisher");
        Console.WriteLine("Foo==null ? {0}", Foo == null);
    }

    public event EventHandler Foo;
}

public class Subscriber
{
    ~Subscriber()
    {
        Console.WriteLine("~Subscriber");
    }

    public void FooHandler(object sender, EventArgs e) {}
}

public class Test
{
    static void Main()
    {
         Publisher publisher = new Publisher();
         Subscriber subscriber = new Subscriber();
         publisher.Foo += subscriber.FooHandler;

         Console.WriteLine("No more refs to publisher, "
             + "but subscriber is alive");
         GC.Collect();
         GC.WaitForPendingFinalizers();

         Console.WriteLine("End of Main method. Subscriber is about to "
             + "become eligible for collection");
         GC.KeepAlive(subscriber);
    }
}

结果(在.NET 3.5SP1中; Mono在这里的表现似乎有些奇怪。将在一段时间内进行调查):

No more refs to publisher, but subscriber is alive
~Publisher
Foo==null ? False
End of Main method. Subscriber is about to become eligible for collection
~Subscriber
2020-05-19