正确使用try/catch块中的finally子句

Question

我试图找出在我的sql数据访问层方法中执行(关闭)语句的"最佳"方式.

我想知道这两种方式中的哪一种更正确(谢谢):

选项1

public void dbOperation( ... )
{
    try
    {
        _cmd.Open();        
        _cmd.Execute();
    }
    catch (Exception ex)
    {
        throw ex;       
    }
    finally
    {
        _cmd.Close()
    }   
}

选项#2

public void dbOperation( ... )
{
    try
    {
        _cmd.Open();        
        _cmd.Execute();
    }
    catch (Exception ex)
    {
        _cmd.Close()    
        throw ex;   
    }

    _cmd.Close();
}

Answer 1

两者都不正确.你不应该只有一个catch子句来重新抛出异常,清除它的堆栈跟踪,并且什么都不做,这是你的第一个选择.

你应该只被关在最后:

try
{
    _cmd.Open();        
    _cmd.Execute();
}
finally
{
    _cmd.Close()
} 

你的第二个片段也有同样的问题,因为你不正确地重新抛出异常.

最好的选择是使用a using,这只是try/finally的语法糖而没有catch:

using(var command = ...)
{
    command.Open();        
    command.Execute();
}

这还有一个额外的好处,即确保命令的范围与使用它的有效时完全相同.try/finally块要求命令在被处理后成为有效的标识符.