c# - WeakEventManager<TEventSource, TEventArgs> 和 PropertyChangedEventManager 导致内存泄漏

Question

我有一个无法删除事件处理程序的应用程序，因为我不知道最后一个引用何时会被释放。

我的应用程序包含一个 PropertyChanged 事件源，该事件源被放入容器类中，该类也实现了 INotifyPropertyChanged。此层次结构包含超过 6 个级别。一个级别的每个实例都可以放置到多个其他实例中。这就是我无法确定何时释放这些实例的原因。

最低级别的实例将在整个应用程序运行时存在。这导致所有其他实例都不会被释放，我遇到了内存泄漏。

为了避免这种事件驱动的内存泄漏，我尝试使用 WeakEventManager(TEventSource, TEventArgs) .此类仅在 .Net 4.5 中可用，并且由于与现有硬件的兼容性，我必须使用 .Net 4.0。

在 .Net 4.0 中是一个 PropertyChangedEventManager可用，应该对 INotifyPropertyChanged 执行相同的操作。

我的类已正确释放。

但是还是有内存泄漏。

我将我的应用程序简化为以下产生内存泄漏的代码:

// This code will force the memory leak
while (true)
{
    var eventSource = new StateChangedEventSource();
    var eventReceiver = new StateChangedEventReceiver();

    PropertyChangedEventManager.AddListener(eventSource, eventReceiver, string.Empty);
}

public class EventSource : INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;
}

public class  EventReceiver : IWeakEventListener
{
    public bool ReceiveWeakEvent(Type managerType, object sender, EventArgs e)
    {
        return true;
    }
}

是的，我知道没有 RemoveListener 调用。我无法确定一个实例何时从不使用并且可以被释放。如果我知道我可以使用正常的事件注册和注销。在这种情况下，我不必使用 PropertyChangedEventManager。

我的示例代码有什么问题？为什么会产生内存泄漏？

编辑 2014/02/17:

我尝试了 WeakEventManager(TEventSource, TEventArgs)和 .Net 4.5，问题仍然存在。

var eventSource = new EventSource();

var i = 0;
while (true)
{
    var eventReceiver = new EventReceiver();

    // --> Use only one of the following three lines. Each of them will produce a memory leak.
    WeakEventManager<EventSource, PropertyChangedEventArgs>.AddHandler(eventSource, "PropertyChanged", eventReceiver.OnEvent);
    PropertyChangedEventManager.AddListener(eventSource, eventReceiver, string.Empty);
    WeakEventManager<EventSource, EventArgs>.AddHandler(eventSource, "SomeOtherEvent", eventReceiver.OnSomeOtherEvent);
    // <--

    ++i;
    if (i == 1 << 18)
    {
        Thread.Sleep(10);
        GC.Collect(2);
        Thread.Sleep(10);
        i = 0;
    }
}


public class EventSource : INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;

    public event EventHandler<EventArgs> SomeOtherEvent;
}

public class EventReceiver : IWeakEventListener
{
    public void OnSomeOtherEvent(object sender, EventArgs args)
    {
    }

    public void OnEvent(object sender, PropertyChangedEventArgs args)
    {
    }

    public bool ReceiveWeakEvent(Type managerType, object sender, EventArgs e)
    {
        return true;
    }
}

这段使用 .Net 4.5 编译的代码也会耗尽内存。我使用 Thread.Sleep 构造得到提示 here .

Answer 1

我不认为 WeakEventManager<,> 有问题特定于非 WPF，因为我也可以在 WPF 应用程序中重现内存泄漏。

问题出在事件表的管理上。对于每个订阅，WeakEventManager在表中创建一个条目。此条目和表格(必然)是强引用的。

问题是，默认情况下，WeakEventManager不清理记录。你必须调用RemoveHandler .但要小心。它不是线程安全的。如果你从另一个线程调用它，它可能会失败(不会抛出异常，你只会体验到仍然存在内存泄漏)。当从终结器调用时，它也不能可靠地工作。

我还调查了源代码，发现虽然它包含对 AddHandler 进行清理的逻辑和当接收到一个事件时，默认情况下它是禁用的(参见WeakEventManager.cs => WeakEventTable.CurrentWeakEventTable.IsCleanupEnabled)。此外，您无法访问 Cleanup方法，因为这样做所需的方法和属性是 private或 internal .所以你甚至不能创建子类来访问这些方法/修改行为。

WeakEventManager<,>坏了

所以基本上(据我所知) WeakEventManager<,>被设计破坏(它保持对订阅者表条目的 StrongReference)。 它不会修复 MemoryLeak，而是只会减少 MemoryLeak(事件源和监听器可以被垃圾收集，但事件订阅的条目不是 => 新的内存泄漏)。当然是 WeakEventManager<,> 引入的内存泄漏很小。