检查 Optional 的布尔值并在 true 时引发异常的最佳方法

Question

我有一个可为空的变量id，通过它我可以通过getLocationById获取位置（同样为空）。
我想做的是if（id！=null

以下是我能想到的方法（除了先取出布尔值，然后在if语句之后抛出异常）。但是当id为空时，它会引发异常，这不是我想要的。

想知道有没有更好的方法？(我用的是Java17)

Optional.ofNullable(id).map(service::getLocationById)
      .filter(Predicate.not(Location::isDeprecated))
      .orElseThrow(()->new BadRequestException("Location is deprecated"))

编辑1:我想我可以做到以下几点，但如果有更好的主意，那就更好了:

Optional.ofNullable(id).map(service::getLocationById)
      .filter(Location::isDeprecated)
      .ifPresent(l->{ throw new BadRequestException("Location is deprecated");});

编辑2：问了所有这些，我同意@Holger在评论中使用旧的好方法提出的建议：

if(id != null) { 
  var loc = service.getLocationById(id); 
  if(loc != null && loc.isDeprecated()) 
    throw … 
}

因为

1. 也相当清晰可读
2. 我的代码在热路径中，可选方式引入了开销和额外的堆分配。




Answer 1

使用try catch块。当你处理可能的异常时，就像你的情况一样，让事情变得简单。

Answer 2

这取决于您的风格和同事的风格。当位置为 null 时，第二个行为不同，因此它们不相同。要使它的行为类似，你必须像这样调整它：

java prettyprint-override">Optional.ofNullable(id).map(service::getLocationById)
  .or(()-> Optional.of(createLocationWithDeprecatedFlag()))
  .filter(Location::isDeprecated)
  .ifPresent(l->{ throw new BadRequestException("Location is deprecated");});

不太好。所以在你的情况下，我建议采用你的第一种方法。它是可读的，从我的观点来看很好理解。

选项不能代替条件，应该在有意义的时候使用，而不是简单地不惜一切代价避免条件。您的情况也可以是一个或多或少简单的if，但可选性在这里并不太差。