抛出异常还是只返回一个字符串？

Question

假设我有这个检查用户详细信息的功能

//check valid user's details.
private bool checkUserDatails(String nickname, String groupId)
{
    //check nickname
    if ((nickname == null || nickname.Trim().Equals("")))
    {
        return false;
    }

    //check groupId
    if (groupId == null)
        return false;

    //convert to int
    int id;
    try
    {
        id = int.Parse(groupId);
    }
    catch (Exception) { return false; }

    //check id
    if (id < 0 | id > 100)
        return false;

    return true;
}

它工作得很好，但调用者函数不知道为什么函数返回 false。我最初的想法是创建自己的异常，例如IncorrectUsernameException 等并抛出它们。我也可以制作返回值字符串并返回“不正确的用户名”。

正确的 C# 方法是什么？

Answer 1

遵循 Microsoft 约定（至少在某些地方），这应该如下所示：

private void EnsureValidDetails(string nickname, string groupId)
{
    if (string.IsNullOrWhiteSpace(nickname))
    {
        throw new ArgumentNullException(nameof(nickname));
    }
    else if (string.IsNullOrEmpty(groupId))
    {
        throw new ArgumentNullException(nameof(groupId));
    }

    int parsedGroupId;

    if (!int.TryParse(groupId, out parsedGroupId))
    {
        // or some better wording
        throw new ArgumentException("GroupId is not a valid number."); 
    }

    if (parsedGroupId < 0 || parsedGroupId > 100)
    {
        throw new ArgumentOutOfRangeException("GroupId must be between 0 and 100.");
    }
}

但请注意，这种方法做得比它应该做的要多。 groupId 应该是 int 参数，并且该方法应该只检查值是否在有效范围内：

private void EnsureValidDetails(string nickname, int groupId)
{
    if (string.IsNullOrWhiteSpace(nickname))
    {
        throw new ArgumentNullException(nameof(nickname));
    }
    else if (groupId < 0 || groupId > 100)
    {
        throw new ArgumentOutOfRangeException("GroupId must be between 0 and 100.");
    }
}

然而，这可能不是真正验证这类东西的最佳方式。它当然可以作为一种通用的、独立于框架的方式，但一些 .NET 框架（WPF、WCF、WinForms、ASP.NET）提供了内置方式来执行此操作。

Answer 2

您不应该抛出异常，因为返回值false 可能会经常发生。这将导致相当大的性能下降。相反，您可以使用 enum，例如使用 correct、invalid_nickname、invalid_id 等值。

enum CheckUserDetailsResult { correct, invalid_nickname, invalid_id }

或者，您可以像这样通过 out 引用提供消息：

private bool checkUserDetails (String nickname, String groupId, out String message)
{
    if (String.IsNullOrEmpty (nickname))
    {
        message = "Invalid nickname!";
        return false;
    }
    //and so on
}