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

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。计时器。因此，从您的答案看来，特别是使用这个计时器类是正确的选择，因为它按照我在这种情况下所期望的方式执行任何操作。如果您在这种情况下尝试另一个计时器类，您可能会遇到麻烦 马修发现了。 我还通过使用 找到了一种方法 ="http://msdn.microsoft.com/en-us/library/system.weakreference.aspx" rel="nofollow noreferrer">WeakReference 判断我的对象是否存活。

更新2

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

因此，在您做出了良好的反应并进行了更多测试之后，我可以高兴地说，它只是做了它应该做的事情，不会将计时器留在内存中，也不会在计时器完成工作之前丢弃它们。

Answer 1

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

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

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

Answer 2

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

定时器分配一个窗口来监听 WM_ 消息，并以某种方式锚定它。当计时器停止时 (Enabled = false)，它会释放该窗口。
静态或扩展方法上下文不起作用。

Answer 3

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

Answer 4

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

看起来这里确实存在内存/资源泄漏......但话又说回来，计时器是一个永远不会被处置的一次性对象。

编辑：...

让乐趣开始...这也可能取决于您使用的计时器。

• System.Windows.Forms.Timer =>;这
  不会收集但会工作
  系统消息泵很好。如果您调用.Stop()，此对象可能会符合收集条件。

• System.Timers.Timer =>;这不
  使用消息泵。青春也多
  调用消息泵。这不会
  收集。如果您调用.Stop()，则该对象可能符合收集条件。

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

Answer 5

看起来计时器将一直保持活动状态直到程序退出。我尝试处理标签并将其引用设置为空，但直到退出程序才收集计时器。

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 隐式地处置 Timer，这样您的实现是正确的并且永远不会导致内存泄漏。