设计 DataLayer 或任何其他分层架构时如何处理异常

Question

我正在创建一个数据访问层，我想在其中处理应该由业务层捕获的异常，并清楚地了解异常的来源。 我正在做这样的事情..

编辑

private void OpenConnection()
        {
                if (ConnectionState.Closed == _connection.State)
                    _connection.Open();
        }

在上面给出的代码中我知道异常的原因。并想把它扔给BL处理以显示消息。

但我在 DL 中仅将这个函数与其他一些代码行一起使用。

protected DataTable GetDataTable(string Query)
 {
     DataTable dt =new DataTable();
     SqlCommand cmd = InitializeCommand(Query);

     SqlDataAdapter adp = new SqlDataAdapter(cmd);
     try
     {
         OpenConnection();
         adp.Fill(dt);
         CloseConnection(true);
         return dt;
     }
     catch (SqlException ex)
     { throw ex; }
     finally
     {
         adp.Dispose();
         cmd.Dispose();
     }
 }

现在，如果在尝试打开连接时 OpenConnection() 中发生异常，那么当我期望它返回异常时，应用程序会在 OpenConnection 本身的 throw ex 行处崩溃。 我该如何处理这个问题。 此外，在第二个函数 GetDataTable 中扩展了我的问题，如果发生异常，应该抛出哪种异常，并具有与此时发生的相同的详细信息以及如何抛出。 我只知道这种方式，但我认为这是一种错误的方式。

throw new ApplicationException(ex.message,ex.innerexception)

编辑

假设与服务器的连接丢失或者我使用了错误的连接字符串。现在我使用 BL 的 GetDataTable 函数。哪里会出现异常，应该在哪里处理？ 我想知道DL中出现的问题。考虑到BL不知道DL的代码

Answer 1

您可以做几件事。

您可以使用不同的 try 块，并根据出现问题的位置抛出新的异常：

 try
 {
     OpenConnection();
     adp.Fill(dt);
 }
 catch (SqlException ex)
 { throw new SqlException("Could not open/populate", ex); }
 finally
 {
     adp.Dispose();
     cmd.Dispose();
 }

 try
 {
     CloseConnection(true);
     return dt;
 }
 catch (SqlException ex)
 { throw new SqlException("Could not close connection", ex); }
 finally
 {
     adp.Dispose();
     cmd.Dispose();
 }

Answer 2

我的建议是删除捕获并重新抛出，因为这会破坏异常的堆栈跟踪。你可以尝试不带任何捕获，因为无论如何你都会抛出异常。通过这种方式，您可以保留堆栈跟踪并仍然使用finally：

protected DataTable GetDataTable(string Query)
 {
     DataTable dt =new DataTable();
     SqlCommand cmd = InitializeCommand(Query);

     SqlDataAdapter adp = new SqlDataAdapter(cmd);
     try
     {
         OpenConnection();
         adp.Fill(dt);
         CloseConnection(true);
         return dt;
     }
     finally
     {
         adp.Dispose();
         cmd.Dispose();
     }
 }