我应该如何处理使用块中的空对象？

Question

鉴于这样的情况：

using (var foo = CreateFoo()) {
    if (foo != null) {
        // do stuff
    }
}

我想避免嵌套if。 遗憾的是，显而易见的解决方案是不可能的，因为break不能用于：

using (var foo = CreateFoo()) {
    if (foo == null) {
        break;
    }
    // do stuff
}

是否有一种模式仍然可以避免由if != null引起的额外缩进？

Answer 1

如果你对从CreateFoo()返回的类有足够的控制，你可以只实现一个Null对象并返回它而不是实际的NULL值

Answer 2

我赞成小明确命名的方法：

public void DoWhatEver()
{
   using (var foo = CreateFoo())
   {
     if (foo == null) return;

     //DoWhatEver
   }
}

Answer 3

介绍一个带lambda的辅助方法。 所以你的代码变成：

UsingIfNotNull(CreateFoo(), foo => {
  //do stuff
});

哪个有你想要的缩进。 UsingIfNotNull的定义是：

public static void UsingIfNotNull<T>(T item, Action<T> action) where T : class, IDisposable {
  if(item!=null) {
    using(item) {
      action(item);
    }
  }
}

Answer 4

这只是一个样式问题......代码很好。 你真的担心缩进吗？ 这是另一种失去缩进的方法......

public void DoWhatEver()
{
   using(var foo = CreateFoo())
   {
       DoStuffWithFoo(foo);
   }

}

private void DoStuffWithFoo(Foo foo)
{
    if(foo == null) return;

    //DoWhatEver

}

Answer 5

在那种通用意义上，我相信我会将try...catch包装在try...catch块中并在对象为null时抛出异常，但这是个人偏好。

Answer 6

这是一个丑陋的黑客，但它避免了额外的身份：

do using (var foo = CreateFoo()) {
    if (foo == null) {
        break;
    }
    // do stuff
} while (false);

（不，我不建议这样做。这只是一个概念验证，表明它是可能的。）

如果可能的话，我建议改为重构你的代码：

 using (var foo = CreateFoo()) {
    if (foo != null) {
        doSomethingWith(foo);  // only one line highly indented
    }
}

Answer 7

就个人而言，我可能会在您发布代码时留下代码。

但是，既然你问过（并且有可能让自己暴露于这种经常被诽谤的语言特征），你可以随时使用“goto”：

using (var foo = CreateFoo()) {
    if (foo == null) {
        goto SkipUsingBlock;
    }
    // do stuff
}

SkipUsingBlock:
// the rest of the code...

Answer 8

C＃编译器使用（var foo = CreateFoo（））语句处理：

try
{
var foo = CreateFoo();
}
finally
{
  ((IDisposable) foo).Dispose();
}

如果您的方法CreateFoo返回不是一次性对象 - 请勿使用。 在其他情况下，你可以写：

try
{
var foo = CreateFoo();
//do stuff like foo.SomeMethod (if foo == null exception will be thrown and stuff will not be done)
}
finally
{
  ((IDisposable) foo).Dispose();
}