System.ArgumentException：参数无效。 GraphicsPath.AddString

Question

所以我一直有这个问题，当调试，我得到一个访问冲突，但是当我没有调试器运行，我得到一个错误，告诉我，一个参数是无效的。它将我引向path.AddString（...）;任何理由为什么？老实说，所有的参数都是正确的，否则编译器会抓住它。这让我很生气。

 protected override void OnPaint(PaintEventArgs e) 
     { 
      Graphics g = e.Graphics; 
      if (!extended) 
      { 
       setColor (); 
       g.FillRectangle (new SolidBrush (currColor), this.ClientRectangle); 
      } 
      g.SmoothingMode = SmoothingMode.AntiAlias; 
      g.InterpolationMode = InterpolationMode.HighQualityBicubic; 

      string szbuf = Program.AppName; 
      SolidBrush brushWhite = new SolidBrush (Color.White); 
      g.FillRectangle (brushWhite, 0, 0, 
      this.ClientSize.Width, this.ClientSize.Height); 

      FontFamily fontFamily = this.Font.FontFamily; 
      StringFormat strformat = StringFormat.GenericDefault; 
      SolidBrush brush = new SolidBrush (Color.FromArgb (255, 255, 255)); 

      SizeF sz = g.MeasureString(szbuf, this.Font); 
      int w = ((this.Width/2) - ((int) sz.Width/2)); 
      int h = 10; 
      GraphicsPath path = new GraphicsPath (); 
      float emSize = g.DpiY * this.Font.Size/72; 
      path.AddString (szbuf, fontFamily, 0, 48f, new Point (w, h), strformat); 

      for (int i = 1; i < 8; ++i) 
      { 
       Pen pen = new Pen (getColor (), i); //Color.FromArgb (32, 0, 128, 192), i); 
       pen.LineJoin = LineJoin.Round; 
       g.DrawPath (pen, path); 
       pen.Dispose (); 
      } 

      g.FillPath (brush, path); 

      fontFamily.Dispose (); 
      path.Dispose (); 
      brush.Dispose (); 
      g.Dispose (); 
     } 

Answer 1

这一行：

fontFamily.Dispose(); 

你处置this.Font.FontFamily对象。 Control将处于无效状态，下一次拨打Paint将失败。你也在处理Graphics对象，不要这样做，因为它可能在你的函数之后被使用。

一般来说，你必须只处理你创建的对象，没有别的也没有少（创建者拥有处置该对象的责任）。编译器无法捕获这种错误（除非运行静态代码分析），因为这是由程序执行路径引起的运行时错误。如果你幸运的话，你会有一个例外（ArgumentException，因为你传递了一个无效的参数：一种处理过的字体）。

此外，您不需要明确地调用Dispose()，使用using语句更安全（它也适用于异常情况）。让我重构点点代码：

protected override void OnPaint(PaintEventArgs e) 
{ 
    Graphics g = e.Graphics; 
    if (!extended) 
    { 
     setColor(); 
     using (var backgroundBrush = new SolidBrush(currColor)) 
     { 
      g.FillRectangle(backgroundBrush, this.ClientRectangle); 
     } 
    } 

    g.SmoothingMode = SmoothingMode.AntiAlias; 
    g.InterpolationMode = InterpolationMode.HighQualityBicubic; 

    string szbuf = Program.AppName; 

    g.FillRectangle(Brushes.White, 0, 0, 
     this.ClientSize.Width, this.ClientSize.Height); 

    StringFormat strformat = StringFormat.GenericDefault; 

    SizeF sz = g.MeasureString(szbuf, this.Font); 
    int w = ((this.Width/2) - ((int)sz.Width/2)); 
    int h = 10; 

    using (var path = new GraphicsPath()) 
    { 
     float emSize = g.DpiY * this.Font.Size/72; 
     path.AddString(szbuf, Font.FontFamily, 0, 48f, new Point(w, h), strformat); 

     for (int i = 1; i < 8; ++i) 
     { 
      using (var pen = new Pen(getColor(), i)) 
      { 
       pen.LineJoin = LineJoin.Round; 
       g.DrawPath(pen, path); 
      } 
     } 

     g.FillPath(Brushes.White, path); 
    } 
} 

请注意，创建资源对于每个油漆是效率不高（允许的，但很慢）。你应该尽可能地重用它们（例如使用字典）。此外，对于不变的颜色，最好使用预定义的画笔（例如Brushes.White，不要将它们处理）。让我告诉一个很天真的实现（易于扩展到同时缓存Pen和SolidBrush）：

private Dictionary<Color, SolidBrush> _solidBrushes; 
private SolidBrush GetSolidBrush(Color color) 
{ 
    if (_solidBrushes == null) 
     _solidBrushes = new Dictionary<Color, SolidBrush>(); 

    if (!_solidBrushes.ContainsKey(color)) 
     _solidBrushes.Add(color, new SolidBrush(color)); 

    return _solidBrushes[color]; 
} 

然后，它会像这样使用：

if (!extended) 
{ 
    setColor(); 
    g.FillRectangle(GetSolidBrush(currColor), this.ClientRectangle); 
}