不好的做法？ c# using 语句的非规范用法

Question

C# 有 using 语句，专门用于 IDisposable 对象。据推测，using 语句中指定的任何对象都将持有某种应该被确定性释放的资源。

然而，在我看来，编程中有许多设计都有一个单一的、明确的开始和结束，但缺乏内在的语言支持。 using 构造提供了使用代码编辑器的内置功能的机会，至少可以清晰自然地突出此类设计或操作的范围。

我想到的是那种经常以 BeginXXX() 和 EndXXX() 方法开始的操作，尽管有很多不同的风格，例如涉及“开始”和“加入”。

以这个幼稚的例子为例。

webDataOperation.Start();
GetContentFromHardDrive();
webDataOperation.Join();
// Perform operation that requires data from both sources

如果 Start 方法返回一个对象，该对象的 IDisposable.Dispose 方法执行连接操作。

using(webDataOperation.Start()) {
    GetContentFromHardDrive();
}
// Perform operation that requires data from both sources

或者，更好的是，我特别想到的是：我有一个执行高度专业化的图形位图传输的对象，并有一个 Begin() 和 End() 方法（DirectX 和 XNA 中也存在这种设计）。而是……

using(blitter.BlitOperation()) {
    // Do work
}
// Use result

它似乎更自然、更易读，但它是否不可取，因为它使用IDisposable 接口和using 语句用于非预期目的？换句话说，这是否等同于以非直观方式加载运算符？

Answer 1

我建议不要这样做；我的信念是代码是为了与代码的维护者而不是编译器进行有效的沟通，并且应该在编写时考虑到维护者的理解。我尝试仅使用“使用”来处理资源，通常是非托管资源。

我是少数。大多数人似乎使用“使用”作为通用目的“即使抛出异常，我也希望运行一些清理代码”机制。

我不喜欢这个，因为（1）我们已经有一个机制，称为“try-finally”，（2）它使用一个特性来达到它不打算用于的目的，以及（3）如果调用清理代码很重要，那为什么在调用它的时候看不到呢？如果它很重要，那么我希望能够看到它。

Answer 2

仅仅因为你可以（或者因为 Phil Haack 说没关系），并不意味着你应该这样做。

基本经验法则：如果我可以阅读您的代码并理解它在做什么以及您的意图是什么，那么它是可以接受的。另一方面，如果你需要解释你做了什么，或者你为什么这样做，这可能会让维护代码的初级开发人员绊倒。

还有许多其他模式可以通过更好的封装来实现这一点。

底线：这种“技术”不会给你带来任何好处，只会让其他开发者感到困惑。

Answer 3

这是一种常见的模式，但就个人而言，我相信没有任何借口可以滥用 IDisposable，因为您可以使用匿名委托和/或 lambda 以更明显的方式实现相同的效果；即：

blitter.BlitOperation(delegate
{
   // your code
});

Answer 4

我会说这是可以接受的——事实上，我已经在一些项目中使用过它，我希望在特定代码块的末尾触发一个动作。

Wes Deyer 在他的 LINQ to ASCII Art 程序中使用了它，他称之为一次性操作（Wes 在 C# 编译器团队工作 - 我相信他的判断 :D）：

http://blogs.msdn.com/wesdyer/archive/2007/02/23/linq-to-ascii-art.aspx

class ActionDisposable: IDisposable
{
    Action action;

    public ActionDisposable(Action action)
    {
        this.action = action;
    }

    #region IDisposable Members

    public void Dispose()
    {
        this.action();
    }

    #endregion
}

现在您可以从函数中返回它，然后执行以下操作：

using(ExtendedConsoleWriter.Indent())
{
     ExtendedConsoleWriter.Write("This is more indented");
}

ExtendedConsoleWriter.Write("This is less indented");

Answer 5

这是一种完全可以接受的做法。这些被称为因子类型，Framework Design Guidelines 建议这样做。

基本上，如果该类型包装了一个具有特定生命周期的操作，那么使用 IDisposable 和 using 语句就成为一个合适的考虑因素。

我实际上也写过关于 this specific topic here 的博客。

Answer 6

我认为您应该将 IDisposable 用于它的用途，而不是其他。也就是说，如果可维护性对您很重要。