【发布时间】:2018-03-19 23:25:50
【问题描述】:
要求是遍历一系列文件夹以根据文件夹名称模式识别和删除子文件夹(在任何级别)及其内容。我有使用 Java 7 的访问者类的工作代码,但现在正试图使其与 Java 8 流一起使用。
我想出了一些工作代码(见下文),但我的印象是它应该存在一种更好、更简单的方法来做同样的事情,而无需重复自己。
public void delete(List<Path> folders) {
folders
.stream()
.filter(Files::isDirectory)
.forEach(this::evaluateIfTrash);
}
private void evaluateIfTrash(Path folder) {
try (Stream<Path> dir = Files.walk(folder, FileVisitOption.FOLLOW_LINKS)) {
dir.filter(Files::isDirectory)
.filter(TrashPredicate::isTrashFolder)
.forEach(this::recursivelyDelete);
} catch (IOException e) {
log.error("Failed to evaluate {}", folder, e);
}
}
private void recursivelyDelete(Path folder) {
try (Stream<Path> dir = Files.walk(folder, FileVisitOption.FOLLOW_LINKS)) {
dir.sorted(Comparator.reverseOrder())
.map(Path::toFile)
.filter(File::exists)
.forEach(File::delete);
} catch (IOException e) {
log.error("Failed to delete {}", folder, e);
}
}
在这里阅读了一些文档和几个答案后,我尝试使用嵌套流来消除一些重复:
private void delete(List<Path> folders) {
folders
.stream()
.filter(Files::isDirectory)
.flatMap(
f -> {
try {
return Files.walk(f, FileVisitOption.FOLLOW_LINKS);
} catch (IOException e) {
return Stream.empty();
}
})
.filter(TrashPredicate::isTrashFolder)
.forEach(this::recursivelyDelete);
}
private void recursivelyDelete(Path folder) {
try (Stream<Path> dir = Files.walk(folder, FileVisitOption.FOLLOW_LINKS)) {
dir.sorted(Comparator.reverseOrder())
.map(Path::toFile)
.filter(File::exists)
.forEach(File::delete);
} catch (IOException e) {
log.error("Failed to delete {}", folder, e);
}
}
这消除了其中一种方法(即使可读性不太好),但我真正不喜欢的是不关闭 flatMap 操作返回的内部流的事实。当我尝试这样做时(通过使用 try-with-resources 构造并在 try 中移动过滤器和 forEach 操作)我收到一条错误消息,指出流已关闭且不可用于进一步操作。
过去两天我一直在尝试完成这项工作,但没有成功,所以问题是,使用 Java 8 是否有更好(更清洁)的方法?
【问题讨论】:
-
我认为这更适合Code Review。
-
使 FileVisitor 更胜一筹的一点是它不需要您抑制 IOExceptions。另一个是您不必使用 java.io.File 的过时方法,这些方法不会显示失败。
标签: java java-8 functional-programming java-stream