哪种更好的设计方法 - 返回bool或抛出异常?

时间:2014-04-28 07:42:21

标签: c# java exception handle

我想知道哪种方法更好。我们假设我们有一种方法,例如,发送通知电子邮件。

void SendNotificaitonEmail();

所以,我可以编辑我的SendNotificaitonEmail()方法,现在它可以执行以下操作:

bool SendNotificationEmail(out string errorMessage)
{
    try
    {
        // This is the code that handles the actual sending of the email.
        // ..
    }
    catch(Exception ex)
    {
        errorMessage = ex.Message;
        return false;
    }
}

但就设计而言,这不是错吗?例如,errorMessage变量与SendNotificationEmail()方法的概念无关。此外,我应该为我的所有方法添加两个新变量 - 布尔值,声明方法的结果(true / false),以及包含错误消息的字符串1(如果出现的话)。

另一种方法是创建我的自定义异常并在调用第一个异常的其他方法中处理它们。

public void SendNotificaitonEmail()
{
    try
    {
        // This is the code that handles the actual sending of the email.
        // ..

        if (somethingIsWrong == true)
        {
            throw new MyCustomException();
        }
    }
    catch(Exception ex)
    {
        // Other exception handling code.
        // ..
    }
}

public void OtherMethod()
{
    try
    {
        SendNotificaitonEmail();
    }
    catch(MyCustomException ex)
    {
        // Exception handling code.
        // ..
    }
}

修改 让我们说我想确保在DAL代码中处理的所有操作都能成功执行。

我有UpdateUserDataGetUserByIdChangeUserPicture等方法。

因此,如果我想检查这些操作是否已成功执行,我应该添加一些其他变量,如:

bool UpdateUserData(User userToUpdate, out string errorMessage); 
User GetUserById(int id, out bool isError, out string errorMessage);
bool ChangeUserPicture(Picture picture, int id, out string errorMessage);
// ..

我有一个使用所有这些方法的简单应用程序:

string errorMessage;
bool isUserUpdatedSuccessfully = UpdateUserData(someUserToUpdate, out errorMessage); 

if (isUserUpdatedSuccessfully == true)
{
    // If the DAL operation was executed successfully, do something..
}
else
{
    // Code that informs the user that an error has occurred.
    MyCustomErrorLogger(errorMessage);        
}

1 个答案:

答案 0 :(得分:11)

将异常视为异常。不要将它们用于正常的程序流控制。

返回值适用于您期望的事物。

[在本地处理异常而不是返回错误代码的问题在概念上是正常的,但只有当该代码的所有使用者都检查错误值时,否则会发生错误,然后被忽略。]