ChatGPT解决这个技术问题 Extra ChatGPT

从 try catch finally 块中返回是不好的做法吗?

所以今天早上我遇到了一些看起来像这样的代码:

try
{
    x = SomeThingDangerous();
    return x;
}
catch (Exception ex)
{
    throw new DangerousException(ex);
}
finally
{
    CleanUpDangerousStuff();
}

现在这段代码编译得很好并且可以正常工作,但是从 try 块中返回感觉不正确,特别是如果有关联的 finally。

我的主要问题是如果 finally 抛出它自己的异常会发生什么?您有一个返回的变量,但也有一个要处理的异常......所以我很想知道其他人对从 try 块中返回的看法?

这种样式的一个好处是您不必在 try 块之外声明 x。你可以保持它的声明接近它的使用。

m
mmx

不,这不是一个坏习惯。将 return 放在有意义的地方可以提高可读性和可维护性,并使您的代码更易于理解。您不必关心,因为如果遇到 return 语句,将执行 finally 块。


N
Nicolas R

finally 无论如何都会被执行,所以没关系。


不,事实上,这不仅仅是拔掉,还有一些异常称为异步异常,例如 StackOverflowException、ThreadAbortException 和 OutOfMemoryException,它们可能会导致 finally 块不被执行。阅读有关处理这些场景的受约束的执行区域。
C
Conrad

就个人而言,我会避免这种编码,因为我不想在 finally 语句之前看到 return 语句。

我的头脑很简单,它处理事情的方式相当线性。因此,当我通过代码进行空运行时,我会倾向于认为一旦我可以到达 return 语句,接下来的一切都无关紧要,这在这种情况下显然是非常错误的(不是它会影响 return 语句,而是副作用可能是什么)。

因此,我会安排代码,使 return 语句始终出现在 finally 语句之后。


C
Community

这可能会回答你的问题

What really happens in a try { return x; } finally { x = null; } statement?

通过阅读该问题,如果您认为它可能会引发异常,听起来您可以在 finally 语句中使用另一个 try catch 结构。编译器将确定何时返回该值。

也就是说,无论如何重组代码可能会更好,这样以后就不会混淆您或其他可能不知道这一点的人。


嗯很有趣。所以这意味着它是安全的,但这是否意味着应该避免它?
就我个人而言,我认为它使您的代码的可读性有点困难,仅此一项就足以让我想出一些其他方式来构建代码。但实际上这只是个人喜好。
我倾向于同意你的个人喜好:)
我认为回报很好。该方法将返回该值,除非抛出异常;最终清理代码中是否出现异常并不重要。
请注意,Java 中不存在从 finally 返回的限制(但我认为该限制是一个很好的限制——对 C# 表示敬意)。
I
Ifeanyi Echeruo

功能上没有区别。

然而,不这样做是有原因的。具有多个出口点的较长方法通常更难以阅读和分析。但这种反对更多地与 return 语句有关,而不是 catch 和 finally 块。


我会挑战这一点,因为你最终可能会得到更多的嵌套,这也使得阅读更高的圈复杂度变得更加困难
B
BeWarned

在您的示例中,任何一种方式都是等效的,如果编译器生成相同的代码,我什至不会感到惊讶。如果在 finally 块中发生异常,无论将 return 语句放在块中还是在块之外,都会遇到相同的问题。

真正的问题是风格上哪个是最好的。我喜欢把我的方法写成只有一个return语句,这样更容易看到方法的流出,所以我也喜欢把return语句放在最后,这样很容易看出它是方法的结尾以及它返回的内容。

我认为由于 return 语句与最后一个语句一样整齐地放置,其他人不太可能来将多个 return 语句分散到方法的其他部分。