c# GDI+，在循环中创建 LinearGradientBrush（内存泄漏）

Question

今天我遇到了一个两难的境地。我创建了一个使用 GDI+ 在表单上绘图的应用程序。每秒由计时器触发绘图。 draw 方法使用 for 循环来迭代对象集合，如果它们处于某种状态，则绘制它们。

我想使用 LinearGradientBrush 来绘制它们，因为它看起来比简单的画笔好得多。看看下面的内容，

            //minutes
        foreach (Led l in MinuteGrid.Leds)
        {
            LinearGradientBrush b = new LinearGradientBrush
                (l.LedRectangle, Color.GreenYellow, Color.Green, 110);

            if (l.IsLit)
                g.FillRectangle(b, l.LedRectangle);

            b.Dispose();
        }

我正在为循环的每次迭代创建一个新的 LinearGradientBrush（这让我很烦恼），但那是因为我必须这样做。我无法在循环外部创建一个，因为它的构造函数集要求我设置仅在循环内部已知的参数。

我发现在 LinearGradientBrush 对象上使用 dispose 方法并不是那么可靠。如果我运行我的应用程序并在任务管理器中查看它，它会喷出内存。当我添加 b = null 行时，它似乎有很大帮助，如下所示，

            foreach (Led l in MinuteGrid.Leds)
        {
            LinearGradientBrush b = new LinearGradientBrush
                (l.LedRectangle, Color.GreenYellow, Color.Green, 110);

            if (l.IsLit)
                g.FillRectangle(b, l.LedRectangle);

            if (b != null)
            {
                b.Dispose();
                b = null;
            }
        }

我只是想知道是否有更好的方法来使用 LinearGradientBrushes ？或者有更好的解决方案可以使用吗？

非常感谢

Answer 1

我建议使用“using”语句：

foreach (Led l in MinuteGrid.Leds)
{
     if (l.IsLit)
     {
         using(LinearGradientBrush b = new LinearGradientBrush(l.LedRectangle, Color.GreenYellow, Color.Green, 110))
         {
            g.FillRectangle(b, l.LedRectangle);
         }
     }
}

但是，请记住，Dispose() 不会释放（托管）内存。它只是释放非托管资源（这很重要，并且可能包括非托管内存）。在 GC 运行之前内存不会释放，这在循环期间可能不会发生。

但是，如果内存压力变得太高，垃圾收集器应该在循环内运行，并且您会看到它下降。这就是 .NET 的设计方式 - 只要接受它，不用担心。 GC 最终会回收这些内存，所以不用担心。

Answer 2

为每个 LED 添加渐变画笔。
如果您无法将其添加到该类中，那么您可以使用 Dictionary将画笔存放起来，以便轻松取用。
这样，每个 Led 只需要一个画笔，而不是每个循环迭代一个画笔

（此外，在示例代码中，如果 !l.IsLit 则没有必要创建画笔）

Answer 3

Dispose 与释放托管内存无关。这完全由 GC 处理，它“在需要时”运行。但是，由于刷子很可能带有手柄，因此您应该将其丢弃。我建议您在 using 块中执行此操作，而不是手动调用 Dispose，因为这将确保即使存在以下情况也会调用 Dispose一个例外。

Answer 4

如果排列数量有限，您可能只需预先创建所有画笔一次：

LinearGradientBrush rectGreenBrush = new LinearGradientBrush(l.LedRect........);
LinearGradientBrush rectRedBrush = new LinearGradientBrush(l.LedRect........);

foreach (Led l in MinuteGrid.Leds)
{
  LinearGradientBrush b = null;
  if (xxx)
    b = rectGreenBrush;
  else if (yyyy)
    b = rectRedBrush;
  else.....


  do painting
}

cleanup brushes

第二个选项类似，但根据需要创建画笔；

List<LinearGradientBrush> createdBrushes = new List<LinearGradientBrush>();

foreach (Led l in MinuteGrid.Leds)
{
  LinearGradientBrush b = null;

  b = FindOrCreateBrushBasedOnLed(l, createdBrushes); 
  // if not already created, creates the brush and adds it to the list

  do painting
}

foreach (LinearGradientBrush b in createdBrushes)
{
  cleanup brushes
}

其他答案是正确的，只要不损害任何东西，.NET 可能会允许托管内存使用量激增。但是，如果有许多 Led 对象需要循环，这应该有助于减少大量创建/删除操作。