我应该仅使用事件聚合器来防止内存泄漏吗？

Question

我们考虑使用 Prism 事件聚合器来减少由于以下原因导致的内存泄漏事件参考。

1. 这本身就是使用此模式的正当理由吗？其他好处现在对我们来说并不感兴趣。我们计划在模型组件之间使用它，而不是 UI。

2. 我们的问题是一些开发人员忘记取消注册事件。我看到 Prism 有一种使用弱引用的风格，但它有 限制。另一种风格强制显式取消订阅（），这又可能会被忘记。那么如何更好呢？

Answer 1

我们的问题是一些开发人员忘记取消注册事件

如果这是您的问题，那么切换到 Prism 事件聚合器（或使用任何其他实现）不会改善情况。

在混乱的情况下添加一个新的依赖项和一个新的、重要的使用模式根本不会把它整理好。

您需要是清理代码。

我建议不要使用新的代码库，而是研究静态分析工具，例如 fxCop （又名代码分析）、宪兵 和 NDepend。所有这些都能够检测到一些事件未脱钩或 IDisposable 未正确实现的情况。

使用静态分析，您可以冷静地识别需要清理的代码。使用内存分析器（例如 dotTrace Memory），您将能够找到最差的内存违者并及时清理。

更新

回答下面评论中的问题：

您认为确保事件脱钩的良好模式是什么？

可能很难确保所有事件都被取消订阅 - 但考虑到事件的实现方式（请参阅通过 C# 的 CLR< /a> 了解详细信息），您可以通过确保丢弃所有事件订阅来进行一些作弊。

不要

public event EventHandler<Fu> FuBar;

自己处理事件订阅，如下所示：

public event EventHandler<Fu> FuBar {
    add { mFuBar += value; }
    remove { mFuBar -= value; }
}

private EventHandler<Fu> mFuBar;

然后，在您的类上实现 IDisposable ，并在您的 Dispose() 方法中将 mFuBar 设置为null，放弃订阅。然后，FxCop（和其他工具）可以告诉您是否无法处理该类。