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

由于 using 块只是 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

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

如果你的 catch 语句需要在 using 中引用的对象在被释放之前，那么 inside 是你唯一的选择。

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

每当我遇到与此类似的场景时，try-catch 块通常使用不同的方法，在调用堆栈中从 using 开始。一个方法知道如何处理它内部发生的异常是不典型的。

所以我的一般建议是在外面——在外面。

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 4

这取决于。如果您使用的是 Windows Communication Foundation (WCF)，如果 using 语句中的代理处于异常状态，using(...) { try... } 将无法正常工作，即处理此代理将导致另一个异常。

就我个人而言，我相信最少的处理方法，即只处理您在执行时知道的异常。换句话说，如果你知道using 中变量的初始化可能会抛出一个特定的异常，我用try-catch 包装它。同样，如果在using 正文中可能发生某些事情，这与using 中的变量没有直接关系，那么我将它与另一个try 包装起来以处理该特定异常。我很少在catches 中使用Exception。

但我确实喜欢IDisposable 和using，所以我可能有偏见。

Answer 5

两者都是有效的语法。它真的归结为您想要做的事情：如果您想捕获与创建/处置对象相关的错误，请使用第二个。如果没有，请使用第一个。

Answer 6

我要在这里指出一件重要的事情：第一件事将不会捕获因调用 MyClass 构造函数而产生的任何异常。

Answer 7

从C# 8.0 on开始，您可以在某些条件下简化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 8

如果您在 Using() 块中初始化的对象可能会引发任何异常，那么您应该使用第二种语法，否则两者同样有效。

在我的场景中，我必须打开一个文件，并且我在对象的构造函数中传递了 filePath，我在 Using() 块中初始化了该对象，如果 filePath 错误/为空，它可能会引发异常。所以在这种情况下，第二种语法是有意义的。

我的示例代码：-

try
{
    using (var obj= new MyClass("fileName.extension"))
    {

    }
}
catch(Exception ex)
{
     //Take actions according to the exception.
}

Answer 9

从 C# 8.0 开始，我更喜欢像这样使用第二个

public class Person : IDisposable
{
    public Person()
    {
        int a = 0;
        int b = Id / a;
    }
    public int Id { get; set; }

    public void Dispose()
    {
    }
}

然后

static void Main(string[] args)
    {

        try
        {
            using var person = new Person();
        }
        catch (Exception ex) when
        (ex.TargetSite.DeclaringType.Name == nameof(Person) &&
        ex.TargetSite.MemberType == System.Reflection.MemberTypes.Constructor)
        {
            Debug.Write("Error Constructor Person");
        }
        catch (Exception ex) when
       (ex.TargetSite.DeclaringType.Name == nameof(Person) &&
       ex.TargetSite.MemberType != System.Reflection.MemberTypes.Constructor)
        {
            Debug.Write("Error Person");
        }
        catch (Exception ex)
        {
            Debug.Write(ex.Message);
        }
        finally
        {
            Debug.Write("finally");
        }
    }