如何正确取消/中止带有@Transactional注解的方法？

你的核心问题在于：Controller方法上的@Transactional让实体处于Spring的持久化上下文（托管状态），哪怕你没显式调用repository.save()，只要实体字段被修改，事务提交时Hibernate/JPA会自动将变更flush到数据库。你用detach的方式确实不够优雅，下面是几种干净且正确的解决方案：

方案1：遵循分层架构，将事务移到Service层（推荐） #

Controller的职责应该是处理请求参数、验证和视图跳转，事务管理属于业务逻辑范畴，应该放到Service层实现：

1. 新建Service类，封装保存和审计逻辑并添加事务：

@Service
public class EntryService {
    private final EntryRepository repository;

    public EntryService(EntryRepository repository) {
        this.repository = repository;
    }

    @Transactional
    public void saveEntry(Entry entry) {
        repository.save(entry);
        updateAudit(entry); // 这里实现你的审计逻辑
    }
}

2. 修改Controller方法，移除@Transactional，验证通过后调用Service：

@PostMapping("/save")
public ModelAndView save(@ModelAttribute(SELECTED_ENTRY) @Valid T entry, BindingResult bindingResult, Model model, EntryService entryService) {

    errorCheck(entry, bindingResult);
    if (bindingResult.hasErrors()) {
        // 验证失败直接返回视图，无事务上下文，实体不会被持久化
        return new ModelAndView(getEditView(), model.asMap());
    }

    entryService.saveEntry(entry);
    
    return new ModelAndView(REDIRECT + getBasePath(), model.asMap());
}

这种方式完全符合Spring的分层设计，从根源上避免了Controller层事务带来的问题。

方案2：验证失败时手动标记事务回滚 #

如果因为某些原因必须在Controller保留@Transactional，可以在验证失败时手动将事务标记为回滚状态：

@Transactional
@PostMapping("/save")
public ModelAndView save(@ModelAttribute(SELECTED_ENTRY) @Valid T entry, BindingResult bindingResult, Model model) {

    errorCheck(entry, bindingResult);
    if (bindingResult.hasErrors()) {
        // 标记事务为回滚，方法结束后不会提交任何变更
        TransactionAspectSupport.currentTransactionStatus().setRollbackOnly();
        return new ModelAndView(getEditView(), model.asMap());
    }

    repository.save(entry);
    updateAudit(entry);
    
    return new ModelAndView(REDIRECT + getBasePath(), model.asMap());
}

这样无需detach实体，事务会在方法结束时回滚，所有实体变更都不会写入数据库。

方案3：用DTO接收请求参数，避免直接操作托管实体 #

在Controller层用DTO（数据传输对象）接收前端提交的参数，验证通过后再将DTO映射到实体进行保存：

@PostMapping("/save")
public ModelAndView save(@ModelAttribute @Valid EntryDto entryDto, BindingResult bindingResult, Model model, EntryService entryService) {

    errorCheck(entryDto, bindingResult);
    if (bindingResult.hasErrors()) {
        return new ModelAndView(getEditView(), model.asMap());
    }

    // 将DTO映射为实体，再调用Service保存
    Entry entry = convertDtoToEntity(entryDto);
    entryService.saveEntry(entry);
    
    return new ModelAndView(REDIRECT + getBasePath(), model.asMap());
}

这种方式下，DTO是普通POJO，不属于持久化上下文，即使验证失败也不会触发任何数据库操作，彻底规避了自动持久化的问题。

内容的提问来源于stack exchange，提问作者IVR Avenger