你可以/应该在 ac# switch 语句中抛出异常吗？

Question

我有一个返回 int 的插入查询。基于该 int 我可能希望抛出一个异常。这适合在 switch 语句中执行吗？

 switch (result)
        {

            case D_USER_NOT_FOUND:
                throw new ClientException(string.Format("D User Name: {0} , was not found.", dTbx.Text));
            case C_USER_NOT_FOUND:
                throw new ClientException(string.Format("C User Name: {0} , was not found.", cTbx.Text));
            case D_USER_ALREADY_MAPPED:
                throw new ClientException(string.Format("D User Name: {0} , is already mapped.", dTbx.Text));
            case C_USER_ALREADY_MAPPED:
                throw new ClientException(string.Format("C User Name: {0} , is already mapped.", cTbx.Text));
            default:

                break;
        }

我通常在开关中添加break语句，但它们不会被击中。这是一个糟糕的设计吗？请与我分享任何意见/建议。

谢谢， ~ck 在圣地亚哥

Answer 1

为什么不呢？

来自 Anders Hejlsberg 等人的《C# 编程语言，第三版》，第 362 页：

switch 部分的语句列表通常以 break、goto case 或 goto default 语句结尾，但任何呈现的构造允许语句列表的结束点无法到达。 [...] 同样，throw 或 return 语句总是将控制权转移到其他地方，并且永远不会到达其终点。因此以下示例是有效的：

开关(i) {
案例0：
    while(true) F();
案例1：
    抛出新的 ArgumentException();
案例2：
    返回;
}

Answer 2

没问题...为什么这是糟糕的设计？

或者，由于所有情况下的异常类型都相同，因此您可以为错误消息构建一个查找表。这将为您节省一些代码重复。例如：

static private Dictionary<int, string> errorMessages;
static
{
    // create and fill the Dictionary
}

// meanwhile, elsewhere in the code...
if (result is not ok) {
    throw new ClientException(string.Format(errorMessages[result], cTbx.Text, dTbx.Text));
}

在消息本身中，您可以使用 {0}、{1} 等选择适当的参数。

Answer 3

我不太同意你的变量命名约定;)，但我不明白为什么你所做的事情是不合适的。这似乎是一种将错误从一种媒介转换为另一种媒介的相当优雅的方式。

我假设您的“插入查询”是某种形式的存储过程。

Answer 4

如果可能的话，如果您在设置失败结果的地方抛出，可能会更好，否则您最终必须同时进行结果检查和抛出。但当然可能不可能。

另外，我会将您的错误字符串放入带有占位符的资源或常量中，这样如果您想更改措辞，就不必更改多个位置。

Answer 5

我认为这还可以。看起来您正在使用 switch 语句将返回代码映射到异常。只要病例不是太多，这不是问题。

Answer 6

我认为在您的情况下使用开关没有任何问题。

更强烈的考虑应该是例外本身是否适当。一般来说，只有当出现超出预期行为范围的情况时才应使用异常。异常不应用作程序流逻辑。在您的情况下，您可能可以根据我看到的代码使用它们。

Answer 7

您所采取的方法没有任何问题。 Switch 语句比 if/then 语句更容易阅读（并且可能也更快）。您可以做的另一件事是将可能的异常加载到 a 中

Dictionary<Result_Type, Exception>

并从那里提取异常。如果您有很多 switch 语句，这将使代码更加紧凑（如果需要，您可以在运行时添加和删除它们）。

Answer 8

也许我请求与这里的所有答案不同。

我宁愿将 result 传递给 ClientException 类，而不必切换代码。让它决定需要显示什么字符串，而不是到处都有一个丑陋的开关来创建各种消息

我的代码看起来像：

throw new ClientException(result, cTbx.Text);

所以，即使你可以在 switch...case 中抛出错误，你可以避免这一切都是我的看法