DAL中的异常处理

时间:2014-06-13 12:26:05

标签: c# .net exception-handling mvp

在MVP winforms应用程序中,我在DAL中按如下方式处理异常。

由于用户消息传递不是DAL的责任,我想将其移至我的Presentation类。

你能告诉我一个标准的方法吗?

    public bool InsertAccount(IBankAccount ba)
    {
        string selectStatement = @"IF NOT EXISTS (SELECT ac_no FROM BankAccount WHERE ac_no=@ac_no) BEGIN INSERT INTO BankAccount ...";

        using (SqlConnection sqlConnection = new SqlConnection(db.ConnectionString))
        {
            using (SqlCommand sqlCommand = new SqlCommand(selectStatement, sqlConnection))
            {
                try
                {
                    sqlConnection.Open();
                    sqlCommand.Parameters.Add("@ac_no", SqlDbType.Char).Value = ba.AccountNumber;
                    //
                    //

                    sqlCommand.ExecuteNonQuery();
                    return true;
                }
                catch (Exception e) { MessageBox.Show(("Error: " + e.Message)); }
                if (sqlConnection.State == System.Data.ConnectionState.Open) sqlConnection.Close();
                return false;
            }

        }
    }

EDIT2:

所以基于我编辑帖子的答案,现在我的异常处理代码看起来像这样......

DAL

public bool InsertAccount(IBankAccount ba)
{
    try
    {
        sqlConnection.Open();
        //   
    }
    catch (SqlException)
    {
        throw new Exception("DataBase related Exception occurred");
    }
    catch (Exception)
    {
        throw new Exception("Exception occurred");
    }
}

BankAccountPresenter

    private void SaveBankAccount()
    {
        try
        {
           _DataService.InsertAccount(_model);
        }
        catch (Exception e) { MessagingService.ShowErrorMessage(e.Message); }
    }

为什么我在DAL中发现了例外情况,即使目前我没有记录错误,我将来也可能不得不这样做。

同样,我可以区分DAL中的错误按摩,无论是sql相关还是一般。

在显示错误消息时,我在演示者中使用了消息服务。

这意味着什么?这可以简化吗?

1 个答案:

答案 0 :(得分:5)

您返回false表示存在异常。不建议这样做。

反思它

catch(Exception e) {
  //blah, maybe add some useful text to e
   throw;
}
finally { //close up shop Although, look up what using does first incidentally }

然后在更高级别处理它(catch (Exception e) { MessageBox.Show(("Error: " + e.Message)); })。

回复EDIT2:

没关系。但是,您当前的实施会引发实际的'异常及其堆栈中的堆栈跟踪有利于您的有用消息。您需要将其添加为内部异常,如下所示:

catch(Exception e){    
     throw new Exception("some helpful message", e);
}