我想知道这两种代码是否有相同的行为

Question

这两个代码的行为都会在值不存在时引发异常。

Optional<FreePosts> byId = Optional.ofNullable(freePostsRepository.findById(id))
                .orElseThrow(NoSuchElementException::new);

FreePosts byId = freePostsRepository.findById(id)
                .orElseThrow(NoSuchElementException::new);

我想知道这两个代码之间的区别。

Answer 1

要正确编写第一个版本，我需要从findById进行空检查。我的理解正确吗？

Answer 2

这两个代码的行为都会在值不存在时引发异常。

事实上，事实并非如此。他们不是都那样的。

这两个版本中的一个是不正确的，但它是哪一个取决于findById的声明返回类型。

>

如果返回类型为FreePosts，则第二个版本不编译，因此不正确。

第一个版本相当于：

 FreePosts byId = freePostsRepository.findById(id);
 if (byId == null) {
     throw new NoSuchElementException();
 }

在我看来，在这种情况下最好写一个明确的抛出，尤其是因为可以包含消息。

如果返回类型为可选

在这种情况下，当且仅当findById返回null时，第一个版本才会引发异常。但是如果findById编写正确，它永远不会返回null。（相反，如果应该返回空的可选

>

如果findById中存在导致其返回null的bug，抛出NoTouchElementException会掩盖该bug。

总之，第一个版本是不正确的。

^{1-我想存储库API的设计者可能已经指定了可选}