当涉及到 finally 块的返回值时，在 try 块中使用 return 语句是否是一种好习惯

Question

我想知道，从try 屏蔽return 是一种好习惯吗？

package debug;

/**
 *
 * @author Owner
 */
public class Main {

  public static void main(String[] args) {
    System.out.println(fun());
  }

  static boolean cleanup() {
    // Fail to cleanup.
    return false;
  }

  static boolean fun() {
    boolean everything_is_fine = true;
    try {
      System.out.println("open file stream");
      return everything_is_fine;
    } finally {
      everything_is_fine = cleanup();
    }
  }
}

---

我首先想到false 会被打印出来。但是，这里是输出：

open file stream
true

如您所见，如果我在 try 块中有 return 语句，我将错过 finally 清理期间的失败状态。

我应该把代码写成：

  static boolean fun() {
    boolean everything_is_fine = true;
    try {
      System.out.println("open file stream");      
    } finally {
      everything_is_fine = cleanup();
    }
    return everything_is_fine;
  }

只要是finally块的返回值，是否应该避免try的return？

Answer 1

您建议的代码（在问题的末尾）很好。您可以从 finally 块返回，但不应该 - 例如 eclipse 显示警告“finally 块未正常完成”。

事实上，try/finally 与return 没有直接关系。这里看起来是这样，因为它是方法中唯一的构造，但之后可以有其他代码（例如 - 事件通知），然后返回。

至于您的问题 - 如果 returned variable 在 finally 块中已返回，则您无法更改它的值。所以不要从try返回。

Answer 2

return 语句指示返回的值，在执行return 语句时是true。 finally 确实改变了变量 everything_is_fine 的值，但这不会改变已经执行的 return 语句返回的内容。

您可以在finally 中添加另一个返回值，这将覆盖try 中的return：

static boolean fun() {
    boolean everything_is_fine = true;

    try {
      System.out.println("open file stream");
      return everything_is_fine;
    } finally {
      everything_is_fine = cleanup();
      return everything_is_fine;
    }
  }

但是，使用finally 来修改控制流并不是一种好的做法。这是当然是可能的。在您的情况下，这样做的更好方法是：

static boolean fun() {
    boolean everything_is_fine = true;

    try {
      System.out.println("open file stream");
    } finally {
      everything_is_fine = cleanup();
    }

    return everything_is_fine;
  }

顺便说一句，变量名应根据现行的Java naming conventions 更改为everythingIsFine ;-)

Answer 3

为什么返回“true”的答案：

如果变量是从try返回的，虽然返回的变量值在finally块中改变了，但之前设置的值（在这种情况下是在try块中设置的值）将被返回。 （当然finally中没有return语句）

回答您希望达到的目标：

如果您希望更改要在 finally 块中返回的值，请遵循您的第二种方法。即：

 static boolean fun() {
    boolean everything_is_fine = true;
    try {
      System.out.println("open file stream");      
    } finally {
      everything_is_fine = cleanup();
    }
    return everything_is_fine;
  }

Answer 4

虽然这被认为是不好的做法，但您可以返回finally。这样做也胜过您在try 和catch 块中可能拥有的任何其他returns。证明：

class Main
{
    public static String test() {
        try {
            except();
            return "return from try";
        } catch (Exception e) {
            return "return from catch";
        } finally {
            return "return from finally";
        }
    }

    public static void except() throws Exception {
        throw new Exception();
    }

    public static void main(String[] args) {
        System.out.println(test());
    }
}

将打印“从最终返回”。在ideone 上查看。

finally 块总是被执行（禁止调用System.exit() 或拔掉电源插头）

编辑：请注意，在您的第一个代码示例中，finally 块中没有返回任何内容，但如果您在那里有一个 return false，您的方法将始终返回 false。

Answer 5

finally 块中对everything_is_fine 的赋值不会影响返回的内容。对我来说，这似乎是一种糟糕的做法。目的是什么？

Answer 6

如果你需要返回依赖于 finally 块中运行的代码的东西，那么你需要把你的 return 放在 try 块之外；正如其他人所说，最好只有一个 return 声明。

话虽如此，我从来没有遇到过我的返回值依赖于 finally 块中的计算的情况，我通常会将返回值放在 try 中。