在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相关还是一般。
在显示错误消息时,我在演示者中使用了消息服务。
这意味着什么?这可以简化吗?
答案 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);
}