处理 findAll() 上的空结果：列表是否有 orElseThrow() ？

Question

我正在创建一个小型 Spring REST 服务。我有一个findById()电话：

@GetMapping("/items/{id}")

MyItem one(@PathVariable String id) {

    return repository.findById(id).orElseThrow(() -> new MyItemNotFoundException(id));

}

如果没有给定的 MyItem 对象id，我将使用该方法抛出异常Optional<T>.orElseThrow()。这非常有用而且非常简单。

现在我添加了findAll()来自PagingAndSorting<T, ID>存储库的调用：

@GetMapping("/items")

List<MyItem> all() {            

    return repository.findAll();

}

是否有一种简单的方法来处理空列表输出，其方式与处理单个项目类似？或者我需要创建类似的东西：

@GetMapping("/items")

List<MyItem> all() {    

    List<MyItem> items = repository.findAll();

    if  (items.isEmpty())

        throw new MyItemNotFoundException();        

    return items;

}

（真实用例处理一些请求参数来过滤整个列表）

Answer 1

Optionalin背后的原因findById是它避免返回null.另一方面，空集合可以安全地迭代和处理，因此没有.throwIfEmpty()内置的特殊机制。空集合本质上本身就是一个Optional。它不为 null，并且可能包含也可能不包含元素。如果在您的业务逻辑中没有结果意味着错误，那么就由您来处理它。

Answer 2

您可以流式传输列表，获取OptionalwithfindAny并映射回列表（如果结果非空）：items.stream().findAny().map((e)&nbsp;->&nbsp;items).orElseThrow(NotFoundException::new);但您应该考虑这是否真的需要导致异常。作为搜索功能的消费者，我的期望将是一个空结果，没有任何元素符合我的标准。

Answer 3

解决方案可能是封装调用的实用函数findAll。public class MyItemNotFoundException ... {&nbsp; &nbsp; public static <T> List<T> requireNotEmpty(List<T> items) throws MyItemNotFoundException {&nbsp; &nbsp; &nbsp; &nbsp; if (items.isEmpty()) {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; throw new MyItemNotFoundException();&nbsp; &nbsp; &nbsp; &nbsp;&nbsp;&nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; &nbsp; return items;&nbsp; &nbsp; }}@GetMapping("/items")List<MyItem> all() {&nbsp; &nbsp;&nbsp;&nbsp; &nbsp; return MyItemNotFoundException.requireNotEmpty(repository.findAll());}将函数放在 MyItemNotFoundException 中可读性不太好。更好的名字仍然感觉不自然：&nbsp; &nbsp; return MyItemNotFoundException.whenEmpty(repository.findAll());但是您会找到一个位置，也许在某个基类/接口中。（在某系统中，存储库 findAll 可能返回 null（非常难看），并且使用这样的包装函数也可以处理。）

Answer 4

通过 REST，您应该返回以下内容：找不到单个元素：抛出异常 - 映射 404 响应代码空列表：返回带有 - 200 状态码的空列表