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

时间:2013-02-21 09:22:44

标签: c# try-catch 3-tier

我有一个基于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 OR IN UI层使用它,或者我的代码没问题。因为如果我在UI层中不使用try / catch,它将不会捕获异常(如果有的话)并显示错误页面。

4 个答案:

答案 0 :(得分:5)

我所看到的异常处理和抛出一直被我与之合作的大多数开发人员误解。

  • 例外允许您在代码中查找错误。
  • 他们停止了正在运行的程序,以防止“伤害”。对业务。
  • 它们允许您在合理预期的异常(移动应用程序上的网络不可用)与NullReferenceException等意外错误异常之间进行过滤。
  • 他们直截了当地说明错误发生的原因。
  • 它们允许每个组件添加一层有关上下文和状态的信息以帮助调试,这是catch,wrap和throw模式。

你可以而且应该在任何地方使用try / catch /(最终),但是......

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

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

你不应该三思而后行或者懒得创建自己的异常类型。例如,您可能希望编写一个DataAccessException,并将其与在InnerException中附加的层中捕获的异常一起抛出。这样,您的日志记录将记录一个异常类型,该类型更准确地显示错误发生的位置,并且调用代码只能选择捕获DataAccessException并执行重试等。

您还可以考虑将DataAccessException抽象化并继承更多特定的异常,例如SqlDataAccessException或SecurityDataAccessException。

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

此外,如果可能的话,将try / catch放在最能够处理错误的特定代码补丁周围,允许周围代码中的编码错误导致应用程序崩溃。

您可能会想,"如果我还不知道会抛出哪些例外,我怎么能抓住一个特定的例外?"您应该看到它们记录在ExecuteNonSql方法上,这就是为什么用它抛出的异常来记录您自己的API /组件是如此重要。使用XML注释执行此操作,如果运送公共DLL,请打开XML注释文件生成器。

这可能看起来很多,但它并没有在实践中。当您投资日志记录和正确的异常处理/投掷时,您将能够在几分钟内解决错误,您将感觉自己像是冠军,并且您很快就会学会对其他人感到不满'糟糕的代码:)

在编程生涯的这个阶段,我强烈建议您阅读Cwalina和Abrams的框架设计指南。它将帮助您快速做出关于所有这些类型问题的正确选择,并且您会发现使用自己的代码与使用Microsoft的API(大多数)一样快乐。

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

"不能{执行某些功能}。发生了{类型的异常}。 {提供补救建议或错误的常见原因}。请参阅{inner exception | Fur​​ther log entries}。"

例如,在app中自动保存状态的组件中:

...
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);
}

答案 1 :(得分:4)

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

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

一般来说,

  1. 只捕获您可以处理的异常(在示例中捕获SqlException,而不是所有异常)

  2. 显示用户友好的消息(在您的示例中,只显示错误消息对用户没有意义)

  3. 记录异常

  4. 处理发生的异常;如果它是DAL相关的异常,则在DAL层处理它,如果它是UI相关的异常,则在UI层处理它。

答案 2 :(得分:2)

您应该编写一个try-catch块来处理异常。没有“永远把试试捕获放在这里或那里”的事情。 我看到你像这样处理异常:

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

这很糟糕,原因如下:

  1. 您捕获通用Exception类型。几天前我看到了一个关于此的问题:https://stackoverflow.com/a/14727026/238682
  2. 您尝试在数据访问层中使用WebMsgBox.Show处理异常,从而打破了图层的边界。
  3. 该示例的另一个问题是一个小问题,但我认为从长远来看(整体代码设计)很重要。 您应该将错误处理逻辑与实际应用程序逻辑分开。因此,当您使用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
        }
    
    }
    

答案 3 :(得分:0)

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

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