正在重用GDI+反对不好的做法？（或者：如何使用许多嵌套的 using 块而不感到头疼？）

发布于 2024-08-06 00:51:22 字数 360 浏览 2 评论 0原文

我目前正在为用户控件编写一个相当复杂的绘制方法，涉及大量的绘制代码。我知道所有 GDI+ 资源都需要正确处置，因此我将每个资源包装在 using 块中。

但是当我注意到我为三个不同的 SolidBrushes 使用了三个 using 块时，我想知道我是否不能重复使用它们。创建一个 SolidBrush，用它绘图，分配不同的颜色，绘制其他内容等。然后在末尾放置一个 Dispose() 。

这样的事情是可取的还是我在这里想太多了？我也不太喜欢太多相互嵌套的 using 块。虽然这是一个很好的模式，但有时会妨碍可读性。

原文

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

帝王念 2024-08-13 00:51:22

好吧，重用是一种优化，而且可能还为时过早，因此：是的，这是一种不好的做法。我会等待明显的性能问题出现，然后寻找瓶颈。您将用一些（未知的）性能增益来换取管理这些组件状态的快速增加的复杂性。

顺便说一句，一种几乎被接受的嵌套使用块的形式如下所示：

 using (var a = ...)
 using (var b = ...)
 using (var c = ...)
 {

 }

Well, re-using is an optimization and it is probably premature and therefore: Yes it is a bad practice. I would wait for a clear performance problem to surface and then look for the bottleneck. You would be trading some (unknown) performance gains for the rapidly increasing complexity of managing the state of those components.

On a side note, one pretty much accepted form of nesting using-blocks looks like this:

 using (var a = ...)
 using (var b = ...)
 using (var c = ...)
 {

 }

回复收藏 0 原文

狠疯拽 2024-08-13 00:51:22

我认为这种做法并没有什么不好。但我认为你可以重复使用创建的画笔。在我自己的用户控件中，如果我使用画笔，我会将它们存储在用户控件的本地变量中，并在处理用户控件时处理它们。

public class MyUserControl : UserControl{

  private SolidBrush _brush;

  public MyUserControl() {
    _brush = new SolidBrush(Color.Red);
  }

  protected override void OnPaint(PaintEventArgs e){
    // Draw with the brush;
  }

  protected override void Dispose(bool disposing){
    // other Dispose logic
    if (_brush != null) {
      _brush.Dispose();
    }
  }
}

I think there is nothing bad with this practice. But i think you can reuse the created Brushes. In my own UserControls, if i use Brushes i store them in a local variable of the UserControl and dispose them of when i dispose the UserControl.

public class MyUserControl : UserControl{

  private SolidBrush _brush;

  public MyUserControl() {
    _brush = new SolidBrush(Color.Red);
  }

  protected override void OnPaint(PaintEventArgs e){
    // Draw with the brush;
  }

  protected override void Dispose(bool disposing){
    // other Dispose logic
    if (_brush != null) {
      _brush.Dispose();
    }
  }
}

回复收藏 0 原文