Spring Boot开发图书馆系统时findById触发Optional无值异常的排查

报错触发的直接原因 #

你遇到的NoSuchElementException，本质是服务层查询方法拿到了空的Optional容器，却直接调用了get()取值。也就是当前传入的查询id在数据库中完全匹配不到对应记录，并非数据库中真的没有对应图书。

核心问题拆解 #

• 控制器参数绑定失效
  你在editLib方法中使用@ModelAttribute Book book接收表单提交值，这种写法要求前端表单的字段名必须严格按照对象.属性格式（也就是book.isbn），才能把选中的图书id绑定到Book对象的主键上。
  对比你之前验证过能正常运行的多对多关联逻辑：关联作者、主题时你都是用单独的@RequestParam直接接收id值，并没有绑定整个实体对象。在当前的图书馆详情页表单中，如果下拉选择框的字段名没有按照book.isbn的规则命名，Spring就无法把选中的图书id绑定到Book对象上，最终拿到的Book对象的isbn是int类型的默认值0，查询id为0的图书自然会返回空结果，触发报错。
• 关联保存逻辑不符合JPA规则
  你之前能正常运行的多对多关联，是靠中间表维护关联关系的，保存Book对象时JPA会自动更新中间表的关联数据。但当前的一对多业务规则下，外键是存储在Book表上的，你当前的editLib方法只调用了saveLibrary，根本不会触发Book表的字段更新；哪怕参数绑定正确，也没有做任何关联赋值操作，完全无法持久化关联关系。
• 实体映射配置不规范
  按照业务规则，图书馆和图书是双向一对多关系：一本图书仅归属一个图书馆，一个图书馆可持有多本图书。当前你在Book实体中已经通过@ManyToOne标注了外键关联，但Library实体上的@OneToMany没有指定mappedBy = "library"，这会让JPA默认额外创建一张中间表维护关联，和Book表上的外键重复，后续会出现数据存储异常。

分步骤修复操作 #

1. 修正editLib方法的参数接收逻辑，和你之前验证过能正常运行的写法保持统一，不要绑定整个Book对象，直接单独接收前端传的图书id：

   @PostMapping(path = "/editLib")
   public RedirectView editLib(@RequestParam Integer bookId, @RequestParam Integer libId){
       Library library = libraryService.findLibById(libId);
       Book book = bookService.getBookById(bookId);
       // 补全双向关联的赋值操作
       book.setLibrary(library); // 维护外键端，这是JPA实际存储关联的核心
       library.getBook().add(book); // 维护一方的关联缓存，方便后续查询
       // 外键在Book表上，必须保存Book才能把关联关系持久化到数据库
       bookService.saveBook(book);
       return new RedirectView("http://localhost:8080/api/library/details/" + libId);
   }
   

2. 修正Library实体的一对多映射配置，去掉多余的中间表：

   // 原配置：@OneToMany
   // 修改为指定外键维护端在Book实体侧
   @OneToMany(mappedBy = "library")
   private List<Book> book = new ArrayList<>();
   

3. 流程跑通后，可后续优化服务层的空值判断，避免直接调用Optional.get()：比如在getBookById、findLibById方法中添加记录不存在时的异常抛出逻辑，比直接抛空值异常的提示更友好。

按上述步骤修改后，即可实现先创建图书馆、再为图书馆绑定图书的核心流程，后续你提到的DTO引入、代码结构梳理等优化可以在流程跑通后再调整。

内容的提问来源于stack exchange，提问作者Leonardo Tadeu Mantovani