为什么Optional不提供peek方法？

Question

我很想知道为什么Java的可选不提供类似于流的方法。

Stream接口的peekmethod javadoc声明：

• @apiNote此方法主要用于支持调试，您希望在元素流经管道中的某个点时看到这些元素

这几乎完全描述了我的用例：

@Override
@Transactional
public User getUserById(long id) {
    return repository.findById(id)
        .peek(u -> logger.debug("Found user = {} by id = {}", u, id))
        .orElseThrow(() -> new UserNotFoundException("id = " + id));
}

（repository.findById返回可选

但是它不会编译，因为 Optional 上没有 peek 方法。

因此，如果没有< code>peek方法，上述所有内容都会转换为:

@Override
@Transactional
public User getUserById(long id) {
  Optional<User> userOptional = repository.findById(id);
  if (userOptional.isPresent()) {
    logger.debug("Found user = {} with id = {}", userOptional.get(), id);
  }
  return userOptional.orElseThrow(() -> new UserNotFoundException("id = " + id));
}

也可以这样做（参见此答案）：

@NoArgsConstructor(access = PRIVATE)
public abstract class OptionalUtils {
    public static <T> UnaryOperator<T> peek(Consumer<T> consumer) {
        return t -> {
            consumer.accept(t);
            return t;
        };
    }
}

并将其与映射方法一起使用：

return repository.findById(id)
    .map(OptionalUtils.peek(u -> logger.debug("Found user = {} with id = {}", u, id)))
    .orElseThrow(() -> new UserNotFoundException("id = " + id));

但我认为这是一个黑客，而不是对可选的干净用法。

从Java 9开始，可以将< code>Optional转换为< code>Stream,但是该流没有< code > orelsthrow 方法(显然它不应该有)。

也可以使用＜code＞ifPresent。（对我来说，＜code＞如果存在＜/code＞不应返回除＜code＞void＜/code＞以外的任何内容。）

我是否误用了< code>Optional？

缺少peek方法是故意的吗？（但同时Vavr的Option确实提供了peek方法。）

还是只是被认为不值得？

Answer 1

已经有可选::ifPresent和可选::isPresent方法来记录结果。但是您可能想要一些符合要求的东西。对此的答案可能是疏忽。

Answer 2

嗯，只有设计师可以回答你“确切”的细节，说明为什么选项没有peek方法。

因此，现在，您只能使用＜code＞isPresent（）

if (userOptional.isPresent()) 
    logger.debug("Found user = {} with id = {}", userOptional.get(), id);

或者，如果您希望将链接页面上的建议答案作为管道的一部分，您也可以考虑。

顺便说一句，考虑到JDK9中新的＜code＞流

return repository.findById(id) // Optional<User>
                 .stream()  // Stream<User>
                 .peek(u -> logger.debug("Found user = {} by id = {}", u, id)) // Stream<User>
                 .findFirst() // Optional<User>
                 .orElseThrow(() -> new UserNotFoundException("id = " + id))

类似的例子见这个回答。

Answer 3

已经有接受消费者的可选::ifPresent方法。

在Java 8中，唯一的方法是使用< code>Optional::map，将实体映射到自身，并将其用作< code>peek方法:

return repository.findById(id)
                 .map(u -> {
                     logger.debug("Found user = {} with id = {}", u, id)
                     return u;
                 })
                 .orElseThrow(() -> new UserNotFoundException("id = " + id));

...这应该简化实现自己的窥视方法：

<T> UnaryOperator<T> peek(Consumer<T> consumer) {
    return t -> {
        consumer.accept(t);
        return t;
    };
}

…并与＜code＞可选＜code＞一起舒适使用：

return repository.findById(id)
                 .map(this.peek(logger.debug("Found user = {} with id = {}", u, id)))
                 .orElseThrow(() -> new UserNotFoundException("id = " + id));