Question

天才！

我正在练习Java 8.

所以，如果我做这样的事情：

Files.walk(Paths.get(corpusPathStr))
        .filter(path -> path.toFile().isFile())
        .forEach(path -> {
            try {
                Files.lines(path)
                        .forEach(...);
            } catch (IOException e) {
                e.printStackTrace();
            }
        });

我收到了FileSystemException错误。

如果我在forEach下打开一个文件，可能会打开太多文件？

或者是否有其他原因导致FileSystemException（打开的文件过多）？

提前感谢您的帮助！

Answer 1

Files::line以懒惰方式打开并读取文件，即Stream<String>。由于您未关闭任何已打开的Stream，因此您会收到此类错误。

因此，当您完成阅读文件后，您应该关闭它的句柄。由于返回的Stream为AutoCloseable，您可以而且应该使用try-with-resource块。

try (Stream<Path> walk = Files.walk(Paths.get(""))) {
  walk.filter(Files::isRegularFile).forEach(path -> {
    try (Stream<String> lines = Files.lines(path)) {
      lines.forEach(System.out::println);
    } catch (IOException e) {
      e.printStackTrace();
    }
  });
} catch (IOException e) {
  e.printStackTrace();
}

Answer 2

使用

try(Stream<Path> stream = Files.walk(Paths.get(corpusPathStr))) {
    stream.filter(path -> Files.isRegularFile(path) && Files.isReadable(path))
          .flatMap(path -> {
                       try { return Files.lines(path); }
                       catch (IOException e) { throw new UncheckedIOException(e); }
                   })
          .forEach(...);
}
catch(UncheckedIOException ex) {
    throw ex.getCause();
}

必须正确关闭Files.walk和Files.lines返回的流以释放资源，您可以使用try(…)构造或者在{的映射函数中返回它们。 {1}}操作。

不要使用嵌套的flatMap。

forEach可能不仅会被我们的映射函数抛出，还会被流实现抛出。将其翻译回UncheckedIOException可以平等对待它们。

Java 8 Path Stream和FileSystemException（打开的文件太多）

2 个答案: