更好的 Java 方法语法？ 早回来还是晚回来？

Question

重复： 一个函数应该只有一个 return 语句吗？

通常，您可能有一种方法可以检查多个返回 语句条件并返回状态（现在假设为布尔值）。 是定义一个标志，在方法期间设置它，然后在最后返回它更好：

boolean validate(DomainObject o) {
  boolean valid = false;
  if (o.property == x) {
     valid = true;
  } else if (o.property2 == y) {
     valid = true;
  } ...
  return valid; 
}

还是在知道方法的结果后简单地返回更好/更正确？

boolean validate(DomainObject o) {

  if (o.property == x) {
     return true;
  } else if (o.property2 == y) {
     return true;
  } ...
  return false; 
}

现在显然可能有 try/catch 块和所有其他类型的条件，但我认为这个概念很清楚。 意见？

Answer 1

如果您将调用该方法数千次，那么尽早返回可以更好地实现[稍微]提高的性能。

如果没有，那么我更喜欢延迟返回，因为它可以提高可读性。

请记住，程序员通常花在阅读上的时间比编写代码的时间多，因此您可以采取任何措施来提高可读性肯定会受到欢迎。

Answer 2

与大多数编码风格一样，这实际上是一个偏好问题，但许多人认为保护子句成为最佳实践。

Answer 3

我更喜欢早点返回并避免深嵌套。 在方法开始时尤其如此：测试任何简单的东西，如果可以尽早退出（或抛出异常）。

如果它位于方法的中间，那么它更像是一个判断调用。

请注意，我会立即重构您的示例以使用单个 if：

boolean validate(DomainObject o) {    
  if (o.property == x || o.property2 == y) {
     return true;
  } ...
  return false; 
}

我意识到这只是一个玩具示例，但我的观点是，寻找更多方法来简化代码总是值得的:)

Answer 4

我唯一会说你绝对不应该提前返回的是，如果你无法轻松地在一个屏幕中看到每个返回（无论使用相同代码库的人的标准是什么），你至少应该添加注释，表明如果有提前返回，该函数可以提前返回。

我唯一会说你绝对应该早点返回的情况是，如果你的代码看起来像......

boolean valid = true;
if( condition1 ) {
   valid = false;
}
if( valid ) {
   ...
   if( condition2 ) {
      valid = false;
   }
}
if( valid ) {
   ...
   if( condition3 ) {
      valid = false;
   }
}
... (etc)

但是，如果你发现自己处于这两种情况之一......你可能应该重构该函数。

Answer 5

对我来说，这是一种没有正确答案的宗教战争话题。 反对提前返回的论点本质上可以归结为这样一个事实：只有一个函数可以退出的点会减少代码中可能的路径数量，从而至少在理论上减少出现错误的机会。 我个人的风格是，在需要提前返回的情况下这样做，在需要限制为一个返回声明的情况下，我会这样做。

Answer 6

如果不存在例外情况，我更愿意尽可能立即返回。

标志变量很容易被错误地管理，而且我一般反对标志变量。 不返回也可能会让维护者认为可以完成进一步的工作（如果方法很长）。

Answer 7

有两个因素相互影响。

第一个因素是易于调试。 如果您立即返回（如第二个代码片段所示），有时调试大函数会变得困难，因为很难找到这些返回语句，特别是如果它们被错误地放在那里。

第二个因素是易于实施。 如果您在函数开头检查参数的基本正确性，并且在函数完成之前有很长一段代码，则可能必须将整个代码放入条件循环中。 如果不这样做，在某些时候该参数可能会被用于一些长时间的计算，浪费时间，因为无论如何它最终都会被拒绝。

所以，答案可能是这样的：

If the function is small, 
        save the return status in a variable and return at the end. 
else 
        return immediately.

Answer 8

就我个人而言，我更喜欢第二种方法。 这对我来说简单明了，但我确实知道有些人必须在函数中只有一个返回。

Answer 9

老实说我认为这取决于具体情况。 就我个人而言，我同时使用这两种方法，并根据哪一种使代码更清晰、更易于阅读来决定。

如果您有大量嵌套的 if 语句（或任何其他控制结构）并且可能会感到困惑，那么我会在语句内返回

不要太担心这种情况下的“最佳实践”，因为更重要的是代码清晰易懂。 使用适合具体情况的方法。

Answer 10

对于这种情况，我更喜欢：

boolean validate (DomainObject o) {
    if (o.property == x ||
        o.property2 == y ||
        ...) {
          return true;
    } else {
          return false;
}

一般来说，我喜欢使用提前返回来处理错误情况，并在最后返回来返回计算结果。