.Net：你对《Catch》的信心是什么？ try-catch 块的部分，开发 CRUD 操作时？

Question

我想知道在 .Net 中开发 CRUD 操作时（特别是当您使用数据库作为数据源时）是否有任何可靠的方法可用于 try-catch 块的 catch 部分？

那么，您对以下几行有何看法？

public int Insert(string name, Int32 employeeID, string createDate)
    {
        SqlConnection connection = new SqlConnection();
        connection.ConnectionString = this._ConnectionString;
        try
        {
            SqlCommand command = connection.CreateCommand();
            command.CommandType = CommandType.StoredProcedure;
            command.CommandText = "UnitInsert";
            if (connection.State != ConnectionState.Open)
                connection.Open();
            SqlCommandBuilder.DeriveParameters(command);
            command.Parameters["@Name"].Value = name;
            command.Parameters["@EmployeeID"].Value = employeeID;
            command.Parameters["@CreateDate"].Value = createDate;
            int i = command.ExecuteNonQuery();
            command.Dispose();
            return i;
        }
        catch
        {
            **// how do you "catch" any possible error here?**
            return 0;
            //
        }
        finally
        {
            connection.Close();
            connection.Dispose();
            connection = null;
        }
    }

Answer 1

首先我会使用 using 语句。

我不会返回 0 作为失败。您无法成功更新任何记录，因此 0 将是有效的成功响应代码。使用 -1 清楚地表明出了问题。就我个人而言，我宁愿在发生意外情况时抛出异常。

try
    { 
       using (SqlConnection connection = new SqlConnection())
        {

            connection.Open();         

            using(SqlCommand command = connection.CreateCommand())
            {
                    command.CommandType = CommandType.StoredProcedure;
                    command.CommandText = "UnitInsert";

                    SqlCommandBuilder.DeriveParameters(command);
                    command.Parameters["@Name"].Value = name;
                    command.Parameters["@EmployeeID"].Value = employeeID;
                    command.Parameters["@CreateDate"].Value = createDate;

                    return command.ExecuteNonQuery();
               }

        }
        }
        catch(Exception ex)
        {
          LogException(ex);
           return either -1 or re throw exception.
        }

Answer 2

在我看来，这是完全错误的地方来捕捉任何你无法立即处理的东西。让异常冒泡并让调用应用程序实现它自己的错误处理。如果您在这里捕获并吞下异常，那么对已安装应用程序的调试将是一场噩梦。

只抓住你能处理的东西，然后扔掉其他的东西......

Answer 3

我会做一个

catch(Exception ex){
     // log exception here and set return value
}

Answer 4

我喜欢通过将所有异常收缩到选定的几个异常并将实际异常嵌套为 InnerException 来形式化我的 DAL 的 API。

例如，所有不是调用者错误的数据库异常都会抛出一种类型的异常，所有属于调用者错误的数据库异常（例如没有选择行、无效的 PK、无效数据）将抛出另一种类型的异常（或者可能是甚至在异常类型之间有更细粒度的区别），最后一个异常类型与数据库无关（健全性检查、NullRef 等），除了我无法处理的异常（例如 OutOfMemoryException ）。

这样就可以很容易地捕获我在 DAL 中抛出的异常，并且所有具体的细节仍然可以在 InnerException 中找到。