扩展方法中的 Lambda：可能的内存泄漏？

Question

我刚刚使用扩展方法回答了quite simple question。但写下来后，我记得你不能从事件处理程序中取消订阅 lambda。

到目前为止没有什么大问题。但是这一切在扩展方法中是如何表现的呢？

下面是我的代码再次被剪断。那么，如果您多次调用此扩展方法，是否会导致无数计时器在内存中徘徊？

我会说不，因为计时器的范围被限制在这个函数中。所以离开它之后，没有其他人可以引用这个对象。我只是有点不确定，因为我们在静态类的静态函数中。

public static class LabelExtensions
{
    public static Label BlinkText(this Label label, int duration)
    {
        System.Windows.Forms.Timer timer = new System.Windows.Forms.Timer();

        timer.Interval = duration;
        timer.Tick += (sender, e) =>
            {
                timer.Stop();
                label.Font = new Font(label.Font, label.Font.Style ^ FontStyle.Bold);
            };

        label.Font = new Font(label.Font, label.Font.Style | FontStyle.Bold);
        timer.Start();

        return label;
    }
}

更新

只是为了澄清我使用了System.Windows.Forms.Timer。因此，从您的回答看来，尤其是使用这个计时器类是正确的选择，因为在这种情况下，它会按照我所期望的方式执行任何操作。如果您在这种情况下尝试另一个计时器类，您可能会遇到Matthew found out 的麻烦。 如果我的对象是否还活着，我也可以使用 WeakReferences found a way。

更新 2

经过一点睡眠和更多思考后，我还对我的测试仪（answer below）进行了另一项更改，我只是在最后一行之后添加了一个GC.Collect()，并将持续时间设置为 10000。在启动 BlinkText() 之后有几次我一直按我的 button2 来获取当前状态并强制进行垃圾收集。而且看起来所有的计时器都将在调用Stop() 方法之后被销毁。因此，当我的 BlinkText 已经离开并且计时器正在运行时，垃圾收集也不会导致任何问题。

因此，在您的所有良好响应和更多测试之后，我可以很高兴地说，它只是做了它应该做的事情，而不会将计时器留在内存中，也不会在计时器完成工作之前将其丢弃。

Answer 1

你的假设是正确的。 timer 将在方法被调用时分配，并在最后有资格进行垃圾回收。

lamba 事件处理程序（假设它没有在其他地方引用）也有资格进行垃圾回收。

这是一个静态方法和/或扩展方法这一事实不会改变对象可达性的基本规则。

Answer 2

您的代码是正确的，不会泄漏内存或资源，但只会因为您在事件处理程序中停止了 Timer。如果您注释掉 //timer.Stop();，它会一直闪烁，即使您稍后执行 GC.Collect(); 也是如此。

Timer 分配一个 Window 来监听 WM_ 消息，并以某种方式锚定于此。当计时器停止 (Enabled = false) 时，它会释放该窗口。
static 或 extension method 上下文不起作用。

Answer 3

内存泄漏不是您的问题。垃圾收集器可以释放计时器对象。但是虽然计时器没有停止，但似乎有对计时器的引用，因此不会过早地处理它。要检查这一点，请从 Timer 派生一个 MyTimer 类，重写 Dispose(bool) 并设置断点。在 BlinkText 调用 GC.Collect 和 GC.WaitForPendingFinalizers 之后。第二次调用后，第一个定时器实例被释放。

Answer 4

我刚刚按照@cornerback84 所说的内容做了一个有趣的测试。我创建了一个带有标签和两个按钮的表单。一个按钮绑定了您的扩展方法，另一个绑定了强制GC.Collect()。然后我在您的事件循环中删除了timer.Stop()。多次点击开始按钮真的很有趣。眨眼的间隔非常混乱。

这里看起来确实存在内存/资源泄漏……但话说回来，Timer 是一个永远不会被释放的一次性对象。

编辑：...

让乐趣开始吧...这也可能取决于您使用的Timer。

• System.Windows.Forms.Timer => 这个 不会收集，但会正常工作 系统消息泵很好。如果您致电.Stop()，此对象可能有资格收集。

• System.Timers.Timer => 这不是 使用消息泵。你也多 调用消息泵。这不会 收藏。如果您致电.Stop()，此对象可能有资格收集。

• System.Threading.Timer => 这需要调用消息泵。但这也会在GC.Collect() 上停止（这个版本似乎没有泄漏）

Answer 5

似乎 Timer 会一直存活到程序退出。我尝试处理 Label 并将其引用设置为 null，但直到我退出程序才收集 Timer。

Answer 6

在阅读了你所有的答案后，我找到了一种更好的测试方法。

我通过以下方式更新了我的扩展类：

public static class LabelExtensions
{
    public static List<WeakReference> _References = new List<WeakReference>();

    public static Label BlinkText(this Label label, int duration)
    {
        Timer timer = new Timer();

        _References.Add(new WeakReference(timer));

        timer.Interval = duration;
        timer.Tick += (sender, e) =>
        {
            timer.Stop();
            label.Font = new Font(label.Font, label.Font.Style ^ FontStyle.Bold);
        };

        label.Font = new Font(label.Font, label.Font.Style | FontStyle.Bold);
        timer.Start();

        return label;
    }
}

我还创建了第二个按钮、第二个标签并将以下代码添加到点击事件中：

    private void button2_Click(object sender, EventArgs e)
    {
        StringBuilder sb = new StringBuilder();

        for (int i = 0; i < LabelExtensions._References.Count; i++)
        {
            var wr = LabelExtensions._References[i];
            sb.AppendLine(i + " alive: " + wr.IsAlive);
        }

        label2.Text = sb.ToString();
    }

现在我只是按了第一个按钮几次，让第一个标签闪烁。然后我按下我的第二个按钮并得到一个列表，我可以在其中查看我的计时器是否还活着。在第一次点击时，他们都得到了一个真实的。但是后来我再次点击了我的第一个按钮，当我更新我的状态消息时，我看到第一个项目在 IsAlive 中已经是错误的。所以我可以肯定地说这个函数不会导致任何内存问题。

Answer 7

你应该通过调用它的 Dispose 方法来显式地释放 Timer，或者在使用它之后通过调用 Stop 来隐式地释放它，这样你的实现是正确的并且永远不会导致内存泄漏。