[英]What is a good way to handle an Optional<User> object returned by a Spring Data JPA repository in a situation like this?
我正在开发 Spring 引导应用程序。 进入服务 class 我有这个服务方法:
@Override
public User getUserById(Integer userId) {
Optional<User> retrievedUser = this.userRepository.findById(userId);
return retrievedUser.get();
}
像往常一样,它调用 Spring 数据 JPA findById()方法返回一个可选的 object。
我正在返回用户object。 在这里我有以下疑问:如果Optional为空,则在执行此操作时会引发异常:
retrievedUser.get()
所以现在......当我有一个可选的 object 时,处理这种情况的最常见方法是什么。
前面的服务方法被调用到实现 API 的 controller 方法中,这个:
@ApiOperation(
value = "Retrieve an user by its id",
notes = "",
produces = "application/json")
@GetMapping(value = "/{useridOrEmail}", produces = "application/json")
public ResponseEntity<User> getUserByIdOrEmail(@PathVariable("useridOrEmail") String userIdOrEmail,
@RequestParam(value="isEmail", required = false, defaultValue = "false") boolean isEmail)
throws NotFoundException {
log.info(String.format("****** Retrieve the user having ID or email: %s *******", userIdOrEmail));
User retrievedUser = null;
if(!isEmail)
retrievedUser = this.userService.getUserById(Integer.parseInt(userIdOrEmail));
else
retrievedUser = this.userService.getUserByemail(userIdOrEmail);
if (retrievedUser == null) {
String ErrMsg = String.format("The user having ID or email %s was not found", userIdOrEmail);
log.warning(ErrMsg);
throw new NotFoundException(ErrMsg);
}
return new ResponseEntity<User>(retrievedUser, HttpStatus.OK);
}
正如您在我的 controller 方法中看到的那样,我正在检查先前服务的结果是否为 null,以防抛出异常。 此异常由 class 扩展ResponseEntityExceptionHandler以创建正确响应的方式处理。
所以我的想法是改变我的服务方法,把这个retrievedUser.get()操作放到try catch中。 如果我获得异常,我的getUserById()服务方法将返回null ,因此我的 controller 方法将抛出NotFoundException异常,该异常将由 Z38008DD81C2F4D7981Z.
可能是一个很好的解决方案或存在更好的方法来处理这种情况?
我会从服务中返回 Optionals 然后你可以像这样很好地链接它们:
public ResponseEntity<User> getUserByIdOrEmail(@PathVariable("useridOrEmail") String userIdOrEmail,
@RequestParam(value="isEmail", required = false, defaultValue = "false") boolean isEmail)
throws NotFoundException {
log.info(String.format("****** Retrieve the user having ID or email: %s *******", userIdOrEmail));
return fetchUser(userIdOrEmail, isEmail)
.map((user) -> new ResponseEntity<User>(user, HttpStatus.OK))
.orElseThrow(() -> new NotFoundException(String.format("The user having ID or email %s was not found", userIdOrEmail)));
}
private Optional<User> fetchUser(String idOrEmail, boolean isEmail) {
return isEmail
? this.userService.getUserByemail(idOrEmail)
: this.userService.getUserById(Integer.parseInt(idOrEmail));
}
并且记录警告应该在一个常见的异常处理程序中处理,该处理程序记录所有未找到的类似方式。
一个好的方法是接受数据库返回的任何内容。 然后 controller应该决定做什么:
Optional<User> retrievedUser = isEmail
? this.userService.getUserByemail(userIdOrEmail)
: this.userService.getUserById(Integer.parseInt(userIdOrEmail));
if (!retrievedUser.isPresent()) {
String ErrMsg = String.format("The user having ID or email %s was not found", userIdOrEmail);
log.warning(ErrMsg);
throw new NotFoundException(ErrMsg);
}
return new ResponseEntity<User>(retrievedUser.get(), HttpStatus.OK);
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.