Springboot服务类-异常处理

Question

我有一个新创建的Springboot服务类，可以使用JPA派生的分类来处理CRUD操作。

在Internet上有很多示例，有些包括已有检查删除，更新（和GET）的检查 - 有些没有。

这就是我的样子：

public class BirdServiceImpl implements BirdService {

    private final BirdRepository birdRepository;

    @Override
    public Bird create(Bird bird) {
        log.info("Saving new bird: {}", bird.getName() );
        return birdRepository.save(bird);
    }

    @Override
    public Collection<Bird> list() {
        log.info("Finding all birds");
        return birdRepository.findAll();
    }

    @Override
    public Bird get(Long id) {
        return birdRepository.findById(id).get();
    }

    @Override
    public Bird update(Bird bird) {
        return null;
    }

    @Override
    public void delete(Long id) {
        log.info("About to delete bird : {}", id);
        Bird bird = new Bird();
        if (birdRepository.existsById(id)) {
             birdRepository.deleteById(id);
        }
    }
}

问题：这些检查是否应该存在在那里，还是应该有一些例外处理，如果是在哪里？

如何确定删除成功，因为它没有返回任何内容？

Answer 1

据我所知，deleteById在以下情况

1. 下，如果给定ID为null illegalArgumentException，则会抛出两个例外。
2. 如果给定ID在持久存储中不存在eyaterResultDataAccessexception。

无需使用已有检查，因为它将启动不必要的查询到持久性商店（数据库）。

我的建议将遵循：

1. 如果没有什么特别的话，除了仅在API响应中返回特定的http status，请不要在<<<<代码>服务层。这里可能有两种情况

   1.1您想为每个服务处理所有现有的ID案例。在这种情况下，您想在控制器建议中处理此操作。返回预定义的HTTP状态和适当的错误消息，用于服务层抛出的异常。

   1.2您可能要处理删除因非存在而失败，在这种情况下，请声明runtime exception延长的专用异常类。在服务中捕获emptyResultDataAccessexception，并准备新创建的异常的异常对象，然后将其投掷。然后在控制器建议中处理该异常特别是。

以下是如何处理控制器建议中的异常，

@RestControllerAdvice()
public class GlobalExceptionHandler extends ResponseEntityExceptionHandler {
    @ExceptionHandler(EmptyResultDataAccessException.class)
    @ResponseStatus(value = HttpStatus.NOT_FOUND)
    public ResponseEntity error(EmptyResultDataAccessException ex) {
        return ...; // Build your response body here.
    }
}

2. 您可能想执行一些特殊的事情，例如我的删除呼叫失败的方式。在这种情况下，请捕获Service内的异常，然后执行您想做的事情（即，您可以在该ID删除失败的活动日志中记录输入）。完成业务任务后，提出适当的异常（基于1.1或1.2，无论哪种适合您的案例），该均应在Controller建议。

Answer 2

您可以使用已有的ByistbyId，如果它返回true，则可以删除特定的记录，否则您可以抛出它不存在的例外。

           if (birdRepository.existsById(id)) {
                     birdRepository.deleteById(id);
                }else {
                     throw new EmptyResultDataAccessException(
                     String.format("No %s entity with id %s exists!", entityInformation.getJavaType(), id), 1)
    );
  }

Answer 3

另一个选项可以使用可选容器。

    Bird bird = birdRepository.findById(id)
                  .orElseThrow(() -> new  EmptyResultDataAccessException(
                         String.format("No %s entity with id %s exists!", entityInformation.getJavaType(), id), 1)))

   birdRepository.delete(bird);

• 取决于您是否要处理运行时间异常并序列化自定义响应并将其发送回客户端。如果是，您可以使用上述方法并在控制器建议中处理此方法。

• 另一个选项是，您可以直接处理控制器建议中的异常，而无需明确地从服务层抛出任何内容。像操作 inlegalargument 例外或 keneryResultDataAccessexception 在控制器建议中。