处理静电刷

Question

我正在编写一个生物节律应用程序。 为了测试它，我有一个带有按钮和图片框的表单。 当我单击按钮时，我

myPictureBox.Image = GetBiorhythm2();

第一次运行正常，但第二次单击时，它会导致以下异常：

System.ArgumentException: Parameter is not valid.
   at System.Drawing.Graphics.CheckErrorStatus
   at System.Drawing.Graphics.FillEllipse
   at Larifari.Biorhythm.Biorhythm.GetBiorhythm2 in c:\delo\Horoskop\Biorhythm.cs:line 157
   at Larifari.test.Button1Click in c:\delo\Horoskop\test.Designer.cs:line 169
   at System.Windows.Forms.Control.OnClick
   at System.Windows.Forms.Button.OnClick
   at System.Windows.Forms.Button.OnMouseUp
   at System.Windows.Forms.Control.WmMouseUp
   at System.Windows.Forms.Control.WndProc
   at System.Windows.Forms.ButtonBase.WndProc
   at System.Windows.Forms.Button.WndProc
   at ControlNativeWindow.OnMessage
   at ControlNativeWindow.WndProc
   at System.Windows.Forms.NativeWindow.DebuggableCallback
   at ComponentManager.System.Windows.Forms.UnsafeNativeMethods.IMsoComponentManager.FPushMessageLoop
   at ThreadContext.RunMessageLoopInner
   at ThreadContext.RunMessageLoop
   at System.Windows.Forms.Application.Run
   at Larifari.test.Main in c:\delo\Horoskop\test.cs:line 20

导致错误的削减函数是：

public static Image GetBiorhythm2() {
        Bitmap bmp = new Bitmap(600, 300);
        Image img = bmp;
        Graphics g = Graphics.FromImage(img);

        Brush brush = Brushes.Black;
        g.FillEllipse(brush, 3, 3, 2, 2); //Here the exception is thrown on the second call to the function

        brush.Dispose(); //If i comment this out, it works ok.

        return img;
 }

如果我注释掉刷子处理工作正常，但我对此不满意并希望找到替代解决方案。 你能帮我吗 ？

Answer 1

Bruhes.Black 是一种系统资源，不适合您处置。 运行时管理 Brushes 类中的画笔、Pen 和其他此类对象。 它根据需要创建和处置这些对象，使常用的项目保持活动状态，这样就不必不断创建和销毁它们。

Brushes 类的文档说：

Brushes 类包含静态
返回 a 的只读属性
颜色由 指示的画笔对象
属性名称。 你通常会这样做
不必明确处置
由属性返回的刷子
类，除非它用于构造
一把新画笔。

简而言之，不要对系统提供的对象调用 Dispose。

Answer 2

看起来您正在尝试处理静态，这会在下次使用时导致一些问题：

    Brush brush = Brushes.Black;
    g.FillEllipse(brush, 3, 3, 2, 2); //Here the exception is thrown on the second call to the function

    brush.Dispose(); //If i comment this out, it works ok.

当您设置画笔 = Brushes.Black 时，您实际上是将画笔设置为静态 Brushes.Black 的引用（或指针） 。 通过处理它，您可以有效地编写：

    Brushes.Black.dispose();

当您回来再次使用黑色画笔时，运行时会说您不能这样做，因为它已经被处理了，并且不是 g.FillEllipse() 的有效

参数更好的写法可能很简单：

    g.FillEllipse(Brushes.Black, 3, 3, 2, 2);

或者，如果你想变得非常复杂：

    Brush brush = Brushes.Black.Clone();
    g.FillEllipse( brush, 3, 3, 2, 2 );
    brush.Dispose();

或者如果你不关心看起来错误的事情，只需注释掉 Brush.Dispose(); 原始代码中的行。

Answer 3

我认为您不需要在静态画笔上调用 .Dispose，除非您创建新的画笔。 虽然，就我个人而言，我会使用 using 语法.. 即：

using (Brush brush = new SolidBrush(...))
{
    g.FillEllipse(brush, 3, 3, 2, 2);
}

您可能应该对您创建的图形对象执行相同的操作。