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

Question

我们考虑使用Prism event aggregator 来减少由于事件引用导致的内存泄漏。

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

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

Answer 1

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

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

将具有新的、非平凡的使用模式的新依赖项添加到混乱的情况中根本不会整理它。

你需要是清理你的代码。

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

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

更新

回答下面评论中的问题：

对于确保事件脱钩的良好模式，您有什么建议？

可能很难确保所有事件都已取消订阅 - 但考虑到事件的实现方式（有关详细信息，请参阅 CLR via C#），您可以通过确保丢弃所有事件订阅来作弊。

代替

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（和其他工具）可以告诉您您是否未能 Dispose 类。