winform代码优化篇（一）_动态窗口优化代码-CSDN博客

本文链接：https://blog.csdn.net/nbspzs/article/details/131884173

代码重写了OnPaint方法，定制控件外观，包括边框和背景。指出了空catch块的问题，应记录异常信息；创建的Pen和SolidBrush对象需释放；Alpha值处理不清晰；并建议使用ClientSize获取绘制区域。改进后的代码修复了这些问题。

摘要由CSDN通过智能技术生成

 protected override void OnPaint(System.Windows.Forms.PaintEventArgs e)
        {
            try
            {
                float vlblControlWidth;
                float vlblControlHeight;

                Pen labelBorderPen;
                SolidBrush labelBackColorBrush;

                if (_transparentBG)
                {
                    Color drawColor = Color.FromArgb(this._alpha, this.BackColor);
                    labelBorderPen = new Pen(drawColor, 0);
                    labelBackColorBrush = new SolidBrush(drawColor);
                }
                else
                {
                    labelBorderPen = new Pen(this.BackColor, 0);
                    labelBackColorBrush = new SolidBrush(this.BackColor);
                }
                base.OnPaint(e);
                vlblControlWidth = this.Size.Width;
                vlblControlHeight = this.Size.Height;
                e.Graphics.DrawRectangle(labelBorderPen, 0, 0, vlblControlWidth, vlblControlHeight);
                e.Graphics.FillRectangle(labelBackColorBrush, 0, 0, vlblControlWidth, vlblControlHeight);
            }
            catch { }
        }


检查上述代码的问题

在上述代码中，`OnPaint`方法被重写，并且在其中执行绘制操作来定制控件的外观。然而，代码中存在一些潜在问题：

1. **异常处理：** 使用空的catch块来捕获所有异常是不推荐的做法。这样会隐藏可能发生的错误，并且难以调试和定位问题。至少应该在catch块中记录异常信息，以便在出现问题时能够快速发现错误。

2. **资源释放：** 代码中创建了`Pen`和`SolidBrush`对象，但没有在使用完毕后显式释放这些资源。这些GDI+资源应该在使用完毕后通过调用`Dispose`方法来释放，以避免资源泄漏。

3. **Alpha值处理：** 代码中涉及对控件背景色的Alpha值处理。Alpha值是用于指定颜色的不透明度，但上述代码没有展示这个Alpha值是如何被设置和使用的。这可能导致无法按预期显示控件。

4. **绘制区域：** 在`OnPaint`方法中，绘制区域应该是控件的客户区域，而不是整个控件（包括边框）。应该使用`ClientSize`来获取绘制区域的尺寸。

综上所述，以下是改进后的代码：

```csharp
protected override void OnPaint(PaintEventArgs e)
{
try
{
base.OnPaint(e);

float vlblControlWidth = this.ClientSize.Width;
float vlblControlHeight = this.ClientSize.Height;

Color backColor = this._transparentBG ? Color.FromArgb(this._alpha, this.BackColor) : this.BackColor;
using (Pen labelBorderPen = new Pen(backColor, 0))
using (SolidBrush labelBackColorBrush = new SolidBrush(backColor))
{
e.Graphics.DrawRectangle(labelBorderPen, 0, 0, vlblControlWidth, vlblControlHeight);
e.Graphics.FillRectangle(labelBackColorBrush, 0, 0, vlblControlWidth, vlblControlHeight);
}
}
catch (Exception ex)
{
// 记录异常信息，方便调试
Debug.WriteLine("Exception in OnPaint: " + ex.Message);
}
}
```

改进后的代码使用了`using`语句来确保在绘制完成后及时释放资源，并且在`catch`块中记录了异常信息，以便进行调试。同时，绘制区域使用了`ClientSize`属性来获取控件的客户区域尺寸。

请注意，改进的代码可能仍需要根据您的具体需求进行调整。