故意提出使用catch的例外

对于if...else带有异常处理的典型包装，是否建议使用以下示例来避免代码重复？

try
{
    if (GetDataFromServer())
    {
        return ProcessData();
    }
    else
    {
        throw new Exception();
    }
catch(Exception ex)
{
    return null;
}

代替...

try
{
    if (GetDataFromServer())
    {
        return ProcessData();
    }
    else
    {
        return null;
    }
}
catch(Exception ex)
{
    return null;
}

我知道会有轻微的性能下降，但是我想知道这是否可以接受。我目前使用第二种方法-特别是在需要以不同方式处理特定异常的情况下-但我想知道第一种方法是否适合简单情况。

Microsoft不鼓励将异常处理用于流控制。

并设有关于该主题的圆桌会议。

话虽如此，C＃支持这样做，我想这取决于遇到的条件是否异常是最合适的响应。

如该答案所述，性能下降很可能是可以忽略的。

因此，让我们考虑一下性能不是问题的想法。你扔System.Exception，只是移动执行到catch条款。但是，抛出a BadControlFlowThatShouldBeRewrittenException可能会过大。

让我们分解一下。我们有：

• 方法GetDataFromServer（方法名称在C＃中应为PascalCase），该方法可能会引发异常或返回bool。
• 如果结果是true，请运行ProcessData。
• null否则返回。

看起来像编写此代码的方法，只是在做太多事情。GetDataFromServer返回一个bool看起来像设计缺陷的外观，我希望该方法返回它从服务器获取的数据，其中一些IEnumerable<SomeType>包含0个或多个项目-即快乐路径返回n个项目，其中n> 0，不那么令人满意path返回0项，不满意的路径会爆炸，并带有未处理的异常，不管是什么。

这极大地改变了该方法的外观-再次很难说这是否有意义，因为原始帖子只有一个出口点（因此不会编译，因为并非所有代码路径都返回一个值），所以这只是一个疯狂的猜测：

try
{
    var result = GetDataFromServer();
    return ProcessData(result);
}
catch
{
    return null;
}

在这里，您将ProcessData看到它正在迭代result，null如果中没有任何项目，则返回IEnumerable。

现在为什么返回该方法null？服务器已关闭？查询中是否有错误？连接字符串使用了错误的凭据？每当GetDataFromServer意外发生异常爆炸时，就将其吞下，将其推到地毯下并返回null值。在这种情况下，我建议捕获特定的异常，并记录其他所有内容。这样调试起来会容易得多。

由于catch没有捕获异常的通用子句，很难诊断任何东西。我会最少这样做：

catch(Exception e)
{
    return null;
}

现在，您至少可以中断并检查e是否出错。

TL; DR：不，抛出和捕获流控制异常不是一个好主意。

在您的第一个答案中，有一个不必要的性能影响。

try
{
    if (GetDataFromServer())
    {
        return ProcessData();
    }
    else
    {
        throw new Exception();
    }
catch(Exception ex)
{
    return null;
}

可以说，当您退出if语句以进入Catch语句时，不必进行代码切换方向。

如果要return null; 在else语句中执行此操作，而不要在从else语句抛出该异常后捕获的catch中执行。

可能不适用于您的Real代码，但适用于您提供的通用代码。

标准规定您不应该这样做。

标准规定您应该这样做（再次基于OP中给出的通用代码）

if (GetDataFromServer())
{
    return ProcessData();
}
else
{
    Return null
}

并且由于您没有捕获的任何特定异常，因此您甚至不应该在这里尝试捕获。

您希望在发生异常时看到它们，以便您可以解决造成异常的问题。

为什么不那么简单：

if (!GetDataFromServer()) return null;
ProcessData();

如果将要存在异常处理程序，则该异常处理程序应位于ProcessData（）中