首页 文章

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

提问于
浏览 次
119

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

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

现在这段代码编译得很好并且可以正常工作，但是从try块中返回它感觉不对，特别是如果最后有关联的话 .

我的主要问题是如果最终抛出它自己的例外会发生什么？你有一个返回的变量，但也有一个例外来处理...所以我很想知道其他人在try块中返回的想法？

c# try-catch try-catch-finally

6 回答

  • 9

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

    回复于
  • 16

    最终将无论如何执行，所以没关系 .

    回复于
  • 157

    就个人而言，我会避免这种编码，因为我不想在最终陈述之前看到返回语句 .

    我的思维很简单，它可以线性地处理事物 . 因此，当我通过代码进行干运行时，我会倾向于认为一旦我能够达到返回语句，一切跟随无关紧要在这种情况下显然是错误的（不是它会影响返回语句但是副作用可能是什么） .

    因此，我会安排代码，以便return语句始终出现在finally语句之后 .

    回复于
  • 3

    这可以回答你的问题

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

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

    也就是说，不管怎么说重组你的代码可能会更好，这样以后就不会让你感到困惑，或者其他人也可能没有意识到这一点 .

    回复于
  • 4

    功能上没有区别 .

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

    回复于
  • 14

    在你的例子中，无论哪种方式都是等价的，如果编译器生成相同的代码，我甚至不会感到惊讶 . 如果在finally块中发生异常，则无论是将return语句放在块中还是在块外部都存在相同的问题 .

    真正的问题是风格上最好的 . 我喜欢编写我的方法，这样只有一个return语句，这样就更容易看到方法的流出，接下来我也想把return语句放在最后，所以很容易看出它是方法的结束和它返回的内容 .

    我认为返回语句如此巧妙地放在最后一个语句中，其他人不太可能将多个返回语句洒到方法的其他部分 .

    回复于

相关问题