try/catch + using,正确的语法

Question

哪一个:

using (var myObject = new MyClass())
{
   try
   {
      // something here...
   }
   catch(Exception ex)
   {
      // Handle exception
   }
}

要么

try
{
   using (var myObject = new MyClass())
   {
      // something here...
   }
}
catch(Exception ex)
{
   // Handle exception
}

Answer 1

我更喜欢第二个.也可以捕获与创建对象有关的错误.

Answer 2

因为使用块只是try/finally(MSDN)的语法简化,所以我个人会考虑以下内容,但我怀疑它与你的第二个选项有很大不同:

MyClass myObject = null;
try {
  myObject = new MyClass();
  //important stuff
} catch (Exception ex) {
  //handle exception
} finally {
  if(myObject is IDisposable) myObject.Dispose();
}

Answer 3

这取决于.如果您使用的是Windows Communication Foundation(WCF),using(...) { try... }则如果proxy in usingstatement处于异常状态,则无法正常工作,即Disposing此代理将导致另一个异常.

就个人而言,我相信最小的处理方法,即只处理您在执行时意识到的异常.换句话说,如果你知道变量in的初始化using可能抛出一个特殊的异常,我用它包装try-catch.类似地,如果在usingbody中可能发生某些事情,这与变量in没有直接关系using,那么我将其与另一个try特定异常包装起来.我很少使用Exception我的catches.

但我确实喜欢IDisposable,using尽管如此我可能有偏见.

Answer 4

如果您的catch语句需要访问using语句中声明的变量,那么inside是您唯一的选择.

如果您的catch语句在处理之前需要使用中引用的对象,那么inside是您唯一的选择.

如果您的catch语句采取未知持续时间的操作,例如向用户显示消息,并且您希望在此之前处置您的资源,那么外部是您的最佳选择.

每当我有一个与此类似的scenerio时,try-catch块通常采用与使用中调用堆栈相同的不同方法.对于方法来说,知道如何处理在其中发生的异常是不常见的.

所以我的一般推荐是在外面的外面.

private void saveButton_Click(object sender, EventArgs args)
{
    try
    {
        SaveFile(myFile); // The using statement will appear somewhere in here.
    }
    catch (IOException ex)
    {
        MessageBox.Show(ex.Message);
    }
}

Answer 5

Both are valid syntax. It really comes down to what you want to do: if you want to catch errors relating to creating/disposing the object, use the second. If not, use the first.

Answer 6

从C# 8.0 开始，您可以using在某些条件下简化语句以摆脱嵌套块，然后它仅适用于封闭块。

所以你的两个例子可以简化为：

using var myObject = new MyClass();
try
{
   // something here...
}
catch(Exception ex)
{
   // Handle exception
}

和：

try
{
   using var myObject = new MyClass();
   // something here...
}
catch(Exception ex)
{
   // Handle exception
}

这两点都很清楚；然后，这将两者之间的选择减少到您想要对象的范围是什么、您想要在哪里处理实例化错误以及何时要处理它的问题。

Answer 7

There is one important thing which I'll call out here: The first one will not catch any exception arising out of calling the MyClass constructor.