迭代非列表的最短方法？

Question

假设我有3个要关闭的Scanner实例。

我可以做

sc.close()

对于每个扫描仪。

或者我可以做类似的事情

for (Scanner sc: new Scanner[]{sc1,sc2,sc3}) {
    sc.close();
}

使用Java8有没有更简单的方法？

类似的东西？

{sc1,sc2,sc3}.forEach((sc) -> sc.close());

Answer 1

虽然有时代码的长度与其质量之间存在相关性，但这并不是选择代码的好标准。

我可能会使用varargs并执行以下操作：

private void closeScanner(Scanner... scanners) {
    // Faff around with Java 8 in here if you like.
    for (Scanner s : scanners) {
        s.close();
    }
}

    // Some code.
    closeScanner(s, t, u);

Answer 2

如果实例化与处置分离，则使用Guava的com.google.常见的木卫一。更接近。

Answer 3

由于Java7，您应该使用try与资源

try(Scanner sc1 = new Scanner(""); 
    Scanner sc2 = new Scanner(""); 
    Scanner sc3 = new Scanner("")){

}
//all scanners get closed implicitly

所以你根本不需要任何代码。

all for each或stream构造的问题是，从理论上讲，如果第一个close（）在调用底层源代码的close（）方法时出现异常而失败，那么以下扫描程序将无法关闭。扫描仪。close（）实现捕获任何IOException，但不会捕获可能发生的其他异常。

try-with-resources构造处理这个问题，而循环不处理。

编辑：虽然您的问题针对的是更一般的方法，但上述解决方案是对您的特定问题的响应：处理自动关闭的资源，在任何情况下都应与try-with-resources结构一起使用，完全不需要对封闭方法进行特殊处理（=特定问题的最短解决方案）。

关于处理任意项（没有资源）的更一般问题，Java至少有两个选项：

从数组/Varargs创建List并迭代它

for(YourItemType item : Arrays.asList(your,items,here)) {
  //do something
}

从数组/Varargs创建流并将函数应用到它

Stream.of(your,items,here).forEach(item -> { doSomething});

当然，“doSomething”可以替换为方法参考

Stream.of(your,items,here).forEach(this::myMethod);
...
void myMethod(YourItemType item){
  //doSomething
} 

这种方法的问题是，必须在lambda表达式中显式处理选中的异常。让我们以上面的示例为例，让myMethod抛出一个选中的异常

void myMethod(YourItemType item) throws Exception

在这种情况下，您的stream语句必须如下所示

Stream.of(your,items,here).forEach(item -> {
  try {
    myMethod(item);
  } catch (Exception e){
    //omit or throw new RuntimeException(e);
  };

这看起来不太好。但是我们可以用另一种方法来处理lambda的尸体

void myMethodQuietly(YourItemType item) {
  try {
    myMethod(item);
  }catch(Exception e){
    //omit or throw new RuntimeException(e);
  }
}

Stream.of(your,items,here).forEach(this::myMethodQuietly);

这种方法可能对您的特定资源问题感兴趣。我们可以把所有这些都放入一个CompositeAutoCloseable中，它获取类外创建的资源，所有这些资源都应该在调用Cloch（）时安全关闭

public class CompositeAutoCloseable implements AutoCloseable {

  private List<Closeable> resources;

  public CompositeAutoCloseable(Closeable... resources) {
    this.resources = Arrays.asList(resources);
    //you could use a stream here too
  }

  @Override
  public void close() {
      this.resources.stream().forEach(this::closeQuietly);
  }

  void closeQuietly(Closeable res) {
    if(res == null)  {
        return;
    }
    try {
        res.close();
    }catch(Exception e){
        //omit
    }
  } 
}

一旦您有了这样一个帮助类，您就可以通过使用资源尝试再次使用它。

try(CompositeAutoCloseable cac = new CompositeAutoCloseable(sc1,sc2,sc3)) {
  //do something
}

我把它留给你来决定，如果这与初始解决方案相比是有意义的；）