Spring Data JPA - 如何实现正确的异常处理？

Question

我正在使用 Spring Data JPA 来处理数据库调用。为此，我创建了：

1. 一个 EmployeeRepository 接口，它扩展了 JpaRepository<Employee, Long>

2. 一个 EmployeeService，它定义了三个方法：

    Employee saveEmployee(Employee employee);
    Optional<Employee> getEmployee(Long id);
    Long deleteEmployee(Long id);
   

3. EmployeeService 的实现：

    @Override
    public Employee saveEmployee(Employee employee) {
        return employeeRepository.save(employee);
    }
   
    @Override
    public Optional<Employee> getEmployee(Long id) {
        return employeeRepository.findEmployeeById(id);
    }
   
    @Override
    public Long deleteEmployee(Long id) {
        employeeRepository.deleteById(id);
        return id;
    } 
   

   问题如下：

   get-methods 工作正常，可以返回一个可选的。另一方面，保存方法不能返回一个可选的。显然 JpaRepository 在调用 save() 时返回已保存对象的实例。我宁愿返回一个可选的，因为在保存员工时可能会出错，在这种情况下，我想抛出一个错误——即，只要可选不存在，我就会抛出一个错误。

   删除操作也是如此：例如，如果我要求删除一个员工并传入一个不存在的 id 怎么办？如果删除操作成功，我想捕获此错误，然后才返回传入的 ID。为此我必须捕获哪个错误？谁可以给我解释一下这个？

   =================================================

   更新：

   • 我通过简单地在调用 deleteById(id) 之前检查给定的 employee-id 是否存在来解决 delete-call 的问题；如果没有，则服务返回 null，如果有，则返回 id。控制器看起来像这样：

       @DeleteMapping("/{id}")
       public ResponseEntity<Long> deleteEmployee(@PathVariable Long id) {
           Long deletedEmployeeId = employeeService.deleteEmployee(id);
           if (deletedEmployeeId != null) {
               return ResponseEntity.ok(deletedEmployeeId);
       } else {
           return ResponseEntity.status(HttpStatus.BAD_REQUEST);
       }
     

   但是，我缺少 DataAccessException。那么，我是否真的必须执行以下操作：

       @DeleteMapping("/{id}")
       public ResponseEntity<Long> deleteEmployee(@PathVariable Long id) {
           try {
               Long deletedEmployeeId = employeeService.deleteEmployee(id);
               if (deletedEmployeeId != null) {
                   return ResponseEntity.ok(deletedEmployeeId);
               } else {
                   return ResponseEntity.status(HttpStatus.BAD_REQUEST);
               }
            } catch (DataAccessException e) {
                return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR)
            }
   

   老实说，这看起来有点矫枉过正。

   • 我仍然不确定如何处理保存调用。在我发布这个问题之前，我的控制器只是在做以下事情：

       @PostMapping
       public ResponseEntity<Employee> saveEmployee(@RequestBody Employee employee) {
           return ResponseEntity.ok(employeeService.saveEmployee(employee));
       }
     

   如果 employeeService.saveEmployee(employee) 抛出 DataAccessException 会发生什么？当我将响应包装在 ResponseEntity.ok() 中时，我是否仍会返回 HTTP-status-code 200？

   如果是这样，我建议执行以下操作：

       @PostMapping
       public ResponseEntity<Employee> saveEmployee(@RequestBody Employee employee) {
           try {
               Employee savedEmployee = employeeService.saveEmployee(employee);
               return ResponseEntity.ok(savedEmployee);
           } catch (DataAccessException e) {
               return ResponseEntity.status(HttpStatus.INTERNAL_SERVER_ERROR);
           }
       }
   

   这是人做的事吗？还是 DataAccessExceptions 通常被忽略，因为它们不是预期的？

Answer 1

“保存”方法总是返回您要保存的相同对象。只有通过检查“id”，你才能看到对象是否被保存。但是，如果数据库中发生错误，则会抛出异常，您可以通过将“employeeRepository.save(employee)”放入 try-catch 块中来捕获它。与 deleteById 相同的方式