来自 Eclipse 的 Java try-with 资源警告

Question

为什么对 Eclipse 来说，这要通过 try-with arm 来管理？

BufferedReader reader = null;
try {
  if (condition) {
    try {
      reader = method1();
    } catch (Exception e) {
      ...
    }
  }
  if (reader == null) {
    reader = method2();
  }

  do things ...
} catch(Exception e) {
  ...
} finally {
  if (reader != null) {
    reader.close();
  }
}

有更好的方法来处理这种情况吗？还是只是来自 Eclipse 的垃圾警告？

这种情况无效：

try (BufferedReader reader = null) {
  if (condition) {
    reader = method1();
  } else {
    reader = method2();
  }

  do things ...
}

Answer 1

试试：

try (BufferedReader reader = createBufferedReader(condition)) {
  do things ...
}

private BufferedReader createBufferedReader(boolean condition){
  if (condition) {
    return method1();
  } else {
    return method2();
  }
}

Answer 2

正如jls-14.20.3 中的 Java 语言规范所述

如果在 ResourceSpecification 中声明的资源未显式声明为 final，则该资源被隐式声明为 final（第 4.12.4 节）。

所以你不能在你的 try 块中改变它。如果您希望能够更改它，请使用标准 try-catch-finally 块。其他选项是在将资源与 try-with-resources 一起使用之前决定正确的资源。

Answer 3

您不一定需要 Seby 的回答中的 Callable 或 lambda 表达式。

鉴于问题很简单，您可以简单地使用三元运算符，它适用于所有 java 版本。

final String s = "abc";

try (BufferedReader reader = (condition) ? method1() : method2();) {

  do things ...

} catch (Exception e) {
  ...
}

Answer 4

您可以使用返回类型为 BufferedReader 的函数，您可以在其中使用条件调用，仅当满足您正在寻找的某些特定条件时，才能从 main 调用此函数。

private BufferedReader createReader(!XYZ){
if(a>b)
return abc();
else
return def();
}

public static void main(String[] args){
createReader(!XYZ);
}

Answer 5

恕我直言，简洁明了：

final String s = "abc";

try (BufferedReader reader = new Callable<BufferedReader>() {
  @Override
  public BufferedReader call() throws Exception {
    if (condition) {
      return method1();
    } 
    return method2();

    // different scope, s needs final keyword
  }
}.call()) {

  do things ...

} catch (Exception e) {
  ...
}

在 java 8 中也更容易：

String s = "abc";

try (BufferedReader reader = () -> {
  if (condition) {
    return method1();
  } 
  return method2();

  // same scope, s does not need final keyword
}) {

  do things ...

} catch (Exception e) {
  ...
}