在Java中对流使用Optional

Question

我试图重构旧代码以使用流，我的第一个方法是：

public void run() throws IOException {
   Files.list(this.source)
        .filter(Images::isImage)
        .map(Image::new)
        .filter(image -> image.isProportional(this.height, this.width))
        .map(image -> image.resize(this.height, this.width))
        .forEach(image -> Images.write(image, this.destination));
}

这不是编译，因为new Image（）和Images.write（）抛出IOExceptions。

用UncheckedIOException包装这些异常不会起作用，因为如果其中一个图像失败，我不想停止处理其他图像。

所以我结束了写2个私人方法：

private Optional<Image> createImage(Path imagePath) {
    try {
        return Optional.of(new Image(imagePath));
    } catch (IOException e) {
        return Optional.empty();
    }
}

private void write(Image image) {
    try {
        Images.write(image, this.destination);
    } catch (IOException e) {
        // log error
    }
}

createImage（）返回一个可选值，因为这看起来很明智。然而在此之后，我的代码变得非常丑陋：

public void run() throws IOException {
    Files.list(source)
         .filter(Images::isImage)
         .map(this::createImage)
         .filter(image -> image.isPresent() && image.get().isProportional(this.height, this.width))
         .map(image -> image.get().resize(this.height, this.width))
         .forEach(this::write);
}

有没有办法避免在该代码上使用get（）和isPresent（）？

谢谢

Answer 1

在Java9中，您可以使用flatMap和Optional：：stream过滤空选项：

public void run() throws IOException {
    Files.list(source)
         .filter(Images::isImage)
         .map(this::createImage)
         .flatMap(Optional::stream)
         .filter(image -> image.isProportional(this.height, this.width))
         .map(image -> image.resize(this.height, this.width))
         .forEach(this::write);
}

Answer 2

Optionals的一个优点是，仅当Optional：：isPresent为true时，才对其应用过滤、映射和平面映射函数，因此：

public void run() throws IOException {
    Files.list(source)
         .filter(Images::isImage)
         .map(this::createImage)
         // turns every non-proportional Optional<Image> into empty optionals
         .map(image -> image.filter(i -> i.isProportional(this.height, this.width)))
         // resizes every proportional Optional<Image>, while doing nothing on the empties
         .map(image -> image.map(i -> i.resize(this.height, this.width)))
         // applies the writing consumer for each non-empty Optional<Image>
         .forEach(image -> image.ifPresent(this::write));
}

另一种方法是在单独的流转换中仅调用Optional：：isPresent和Optional：：get：

public void run() throws IOException {
    Files.list(source)
         .filter(Images::isImage)
         .map(this::createImage)
         // filter out the empty optionals
         .filter(Optional::isPresent)
         // replace every optional with its contained value
         .map(Optional::get)
         .filter(image -> image.isProportional(this.height, this.width))
         .map(image -> image.resize(this.height, this.width))
         .forEach(this::write);
}

还有另一种方法（由于其相对奇怪，我拒绝推荐作为主要解决方案）是将静态图像创建方法更改为Stream生成器，而不是可选生成器，以利用平面图：

private Stream<Image> createImage(Path imagePath) {
    try {
        return Stream.of(new Image(imagePath));
    } catch (IOException e) {
        return Stream.empty();
    }
}

public void run() throws IOException {
    Files.list(source)
         .filter(Images::isImage)
         // inserts into the stream the resulting image (empty streams are handled seamlessly)
         .flatMap(this::createImage)
         .filter(image -> image.isProportional(this.height, this.width))
         .map(image -> image.resize(this.height, this.width))
         .forEach(this::write);
}

再想一想，采用这个解决方案；这似乎更简单，而且由于静态方法无论如何都是私有的，因此最终用户、其他开发人员以及能够访问体面的Java 8反编译器的随机人员的尖叫声会更少(http://www.benf.org/other/cfr/).