如何编写包含一些一次性对象的 If 语句

Question

我有以下 C# 代码

  iSomeObject.MyPoint(SomeConstants.cpU, 2,
                          myInterface.MySystem.MyCustomType == OtherConstants.cpProjected ? Constants.cpU : Constants.cpT,
                          1, ref t1, ref t2, ref t3);

当我运行我的专有分析工具时，它说 myInterface.MySystem 导致资源泄漏。

 class MyClass:MyInterface,IDisposable
 {}
  MyInterface myInterface = new MyClass();

我通过将实例 myInstance 强制转换回 IDisposable 并在其上调用 Dipose() 方法来显式处理它。

现在在 myInterface 上调用 MySystem 属性的 get 方法，其中（MySystem）是 IExampleInterface 类型，进一步实现如下代码：

 class ExampleClass:IExampleClass,IDisposable
 {}

我不确定在 myInterface 上调用 Dipose() 方法是否也会释放 MySystem 创建的资源，或者我是否也需要显式调用 Dispose() 方法。但是一切都发生在 IF 语句条件中，我不确定如何处理这种情况并让我的代码处理掉所有一次性对象，换句话说，我不确定这种情况下的 C# 语法以及如何处理处理概念在这种情况下。

任何建议将不胜感激。

Answer 1

如果 MySystem 是一个 IDisposable 对象，那么您的类应该在它的 Dispose 实现中处理该对象。

否则，请修复您的静态分析工具。

Answer 2

我会说你应该把处置放在知道你有MyClass的代码层。

using(var iSomeObject = new MyClass(...)) {
    // do something 
    iSomeObject.MyPoint(SomeConstants.cpU, 2,
                      myInterface.MySystem.MyCustomType == OtherConstants.cpProjected ? Constants.cpU : Constants.cpT,
                      1, ref t1, ref t2, ref t3);
}

Answer 3

我不确定你所说的 if 语句是什么意思。如果您在方法/属性/构造函数（无论如何）中使用一次性实例，并且它们不属于类的实例成员，则应在使用后立即将其丢弃。

如果您的类中有实现 IDisposable 的成员，那么您的类也应该实现 IDisposable，并且一旦不再需要您的类的实例（就像实现 IDisposable 的其他任何东西一样），应该立即调用 Dispose。

在与实现 IDisposable 的实例交互时，也总是使用 try/finally 或 using 块。

最后请参阅 Microsoft 的 this write up ，了解使用 IDisposable 的完整详细信息和最佳实践。

public interface IExampleInterface { void DoSomething();}
public class ExampleClass : IExampleInterface, IDisposable {

   private bool _switch = true;

   public void DoSomething() {
       // lets use something disposable
       if(_switch) { // is this what you mean by in an if statement??
          var stream = new System.IO.MemoryStream();
          try {
              // do something with stream
          } finally {
              stream.Dispose(); // call dispose!
          }
       }
   }
   private System.IO.FileStream fs; // class level field
   public void Dispose(){
      // dispose of class fields here that implement IDisposable
      if(fs != null)
         fs.Dispose();
      fs = null;
   }
}