c# - 在 3 层架构中放置 try/catch 的位置

Question

我有一个基于 3 层的 Web 应用程序。我想使用我在业务逻辑层中使用 try catch 块。在业务逻辑中使用 try/catch 块是否正确，或者我需要在我的 UI 层中使用它？

请参阅我的 DAL 代码。

Data Access Layer

#region Insert in to Logbook
public int Insert_LogBook(string Vehicle_Number, DateTime Vehicle_Booking_Date, TimeSpan Time_From, TimeSpan Time_To, int KM_Start, int KM_End, string Vehicle_Used_By, string Cost_Code, string Budget_Line, DateTime Entry_Date)
{
    try
    {
        SqlCommand com = new SqlCommand("Insert_LogBook", con);
        com.Parameters.Add("@Vehicle_Number", SqlDbType.NVarChar, 100).Value = Vehicle_Number;
        com.Parameters.Add("@Vehicle_Booking_Date", SqlDbType.DateTime).Value = Vehicle_Booking_Date;
        com.Parameters.Add("@Time_From", SqlDbType.Time).Value = Time_From;
        com.Parameters.Add("@Time_To", SqlDbType.Time).Value = Time_To;
        com.Parameters.Add("@KM_Start", SqlDbType.Int).Value = KM_Start;
        com.Parameters.Add("@KM_End", SqlDbType.Int).Value = KM_End;
        com.Parameters.Add("@Vehicle_Used_Byr", SqlDbType.VarChar, 100).Value = Vehicle_Used_By;
        com.Parameters.Add("@Cost_Code", SqlDbType.NVarChar, 50).Value = Cost_Code;
        com.Parameters.Add("@Budget_Line", SqlDbType.NVarChar, 50).Value = Budget_Line;
        com.Parameters.Add("@Entry_Date", SqlDbType.DateTime).Value = Entry_Date;
        con.Open();
        int res = com.ExecuteNonQuery();

    }
    catch (Exception ex)
    {
        WebMsgBox.Show(ex.Message);
    }
    finally
    {
        con.Close();
        con.Dispose();

    }
    return 1;
}
#endregion

所以我应该在我的 bal 或 UI 层中使用它还是我的代码没问题。因为如果我不在我的 UI 层中使用 try/catch，它将不会捕获异常（如果有的话）并显示错误页面。

score 6 · Accepted Answer

异常处理和抛出是我一直以来都被我所合作过的大多数开发人员所误解的东西。

异常允许您在代码中找到错误。
他们停止正在运行的程序以防止对业务造成“伤害”。
它们允许您在合理预期的异常（移动应用程序上的网络不可用）和意外的错误异常（如 NullReferenceException）之间进行过滤。
他们直截了当地了解为什么会发生错误。
它们允许每个组件添加一层关于上下文和状态的信息来帮助调试，这就是 catch、wrap 和 throw 模式。

您可以而且应该在几乎任何地方使用 try/catch/(finally)，但是...

仅当您知道哪些异常可能发生并且可以从中恢复时才使用 catch。您应该很少捕获基本异常类型。允许所有其他错误冒泡并被程序员/测试人员/用户发现。

如果要抛出另一个异常并将原始异常附加为 InnerException，则可能需要捕获基本异常类型。

您不应该三思而后行，也不要懒于创建自己的异常类型。例如，您可能想要编写一个 DataAccessException 并抛出该异常，并在附加在 InnerException 的层中捕获异常。这样，您的日志记录将记录一个异常类型，该类型可以更准确地显示错误发生的位置，并且调用代码可以选择仅捕获 DataAccessException 并执行重试或其他操作。

您还可以考虑将 DataAccessException 抽象化并将更具体的异常子类化，例如 SqlDataAccessException 或 SecurityDataAccessException。

如您所知，当您想确保在发生错误时运行某些代码时使用 finally，即使您不捕获和处理异常本身。在必须始终释放资源的情况下，try/finally 成为标准模式。

此外，如果可能，请将 try/catch 放在您能够处理错误的最具体的代码补丁周围，从而允许周围代码中的编码错误导致应用程序崩溃。

你可能会想，“如果我还不知道会抛出哪些异常，我怎么能捕捉到一个特定的异常呢？” 您应该看到它们记录在 ExecuteNonSql 方法中，这就是为什么记录您自己的 API/组件及其引发的异常如此重要的原因。使用 XML 注释来执行此操作，如果提供公共 DLL，请打开 XML 注释文件生成器。

这似乎需要接受很多，但实际上并非如此。当您投资于日志记录和适当的异常处理/抛出时，您将能够在几分钟内解决错误，您会感觉自己是冠军，并且您很快就会学会对其他人的糟糕代码感到不满 :)

在你编程生涯的这个阶段，我强烈推荐阅读Cwalina 和 Abrams 的框架设计指南。它将帮助您针对所有这些类型的问题快速做出正确的选择，并且您会发现使用自己的代码与使用 Microsoft 的 API（大部分情况下）一样令人愉快。

卢克

添加一些关于消息的内容。我在错误消息中使用了这种东西。

“无法{执行某些功能}。发生了{异常类型}。{提供补救建议或错误的常见原因}。请参阅{内部异常|更多日志条目}。”

例如，在应用程序中自动保存状态的组件中：

...
catch(FileNotFoundException fnfe)
{
    string m = String.Format("Cannot save changes. A FileNotFoundException occurred. Check the path '{0}' is valid, that your network is up, and any removable media is available. Please see inner exception.", path);

    _log.Error(m, fnfe);

    throw new StorageLifecycleException(m, fnfe);
}

score 4 · Accepted Answer

这取决于，您可以try catch在两层中使用块。

但问题不在于您在哪里使用异常处理代码；问题是你如何使用。在您提供的示例中，您正在捕获一个泛型exception，您不知道它是 SqlException 还是任何其他异常。

一般来说，

仅捕获您可以处理的异常（在示例中 catch SqlException，不是所有异常）
显示用户友好的消息（在您的示例中，仅显示错误消息对用户没有意义）
记录异常
处理发生的异常；如果是 DAL 相关异常，则在 DAL 层处理，如果是 UI 相关异常，则在 UI 层处理。

score 2 · Accepted Answer

您应该编写一个 try-catch 块来处理异常。没有“总是把try-catch放在这里或那里”这样的事情。我看到你处理这样的异常：

catch (Exception ex)
{
    WebMsgBox.Show(ex.Message);
}

这很糟糕，有几个原因：

你抓住了泛型Exception类型。前几天看到一个关于这个的问题：https ://stackoverflow.com/a/14727026/238682
您尝试WebMsgBox.Show在数据访问层内处理异常，这打破了层的边界。

该示例的另一个问题是一个小问题，但我认为从长远来看它很重要（整体代码设计）。您应该将错误处理逻辑与实际应用程序逻辑分开。因此，当您使用 try-catch 块时，请尽量减少其中的逻辑，从而使您的代码变得更具可读性。

public int Insert_LogBook(string Vehicle_Number, DateTime Vehicle_Booking_Date, TimeSpan Time_From, TimeSpan Time_To, int KM_Start, int KM_End, string Vehicle_Used_By, string Cost_Code, string Budget_Line, DateTime Entry_Date)
{
    using(SqlCommand com = new SqlCommand("Insert_LogBook", con))
    {
        com.Parameters.Add("@Vehicle_Number", SqlDbType.NVarChar, 100).Value = Vehicle_Number;
        com.Parameters.Add("@Vehicle_Booking_Date", SqlDbType.DateTime).Value = Vehicle_Booking_Date;
        com.Parameters.Add("@Time_From", SqlDbType.Time).Value = Time_From;
        com.Parameters.Add("@Time_To", SqlDbType.Time).Value = Time_To;
        com.Parameters.Add("@KM_Start", SqlDbType.Int).Value = KM_Start;
        com.Parameters.Add("@KM_End", SqlDbType.Int).Value = KM_End;
        com.Parameters.Add("@Vehicle_Used_Byr", SqlDbType.VarChar, 100).Value = Vehicle_Used_By;
        com.Parameters.Add("@Cost_Code", SqlDbType.NVarChar, 50).Value = Cost_Code;
        com.Parameters.Add("@Budget_Line", SqlDbType.NVarChar, 50).Value = Budget_Line;
        com.Parameters.Add("@Entry_Date", SqlDbType.DateTime).Value = Entry_Date;
        con.Open();
        int res = com.ExecuteNonQuery();

        return 1;
    }

}

public void SomeMethodWhichUsesThatInsert()
{
    try
    {
        //call Insert_LogBook
    }
    catch(SomeException e)
    {
        //handle
    }

}

score 0 · Accepted Answer

您应该将数据访问和业务逻辑放入类库中。您不应该从 BLL 或 DAL 调用前端组件。在您的 BLL 和 DAL 日志中创建日志记录，使用类似 log4net 的内容将错误数据放在这里。

如果要通知用户错误状态，则向上抛出异常。

c# - 在 3 层架构中放置 try/catch 的位置

4 回答 4

Related

Reference