@Transactional回滚正常但后续方法仍执行的问题排查

问题原因 #

你的问题核心在于JPA延迟持久化与Spring声明式事务的执行时机：

1. Spring声明式事务基于AOP代理，事务的提交/回滚操作是在目标方法（method1）执行完毕后，由代理类统一处理的。
2. JPA默认会延迟执行数据库操作：调用repository.saveAll()时，只是将实体状态变更记录到EntityManager，并不会立即发送SQL到数据库，直到事务提交阶段才会一次性flush所有操作到数据库。
3. 因此，saveAll的SQL执行异常会在method1完全执行完（包括Util.staticMethod()）之后才抛出，此时事务触发回滚，但静态方法已经执行完毕。

解决方案 #

方案1：手动flush，让数据库操作异常提前抛出 #

在deleteAllAndSaveAll方法中手动触发flush，强制JPA立即执行SQL，这样异常会在调用deleteAllAndSaveAll时就抛出，阻止后续代码执行：

@Transactional(rollbackFor = { Exception.class })
@Service
public class Service {

    public void method1() {
        stuffToGetListOfObjects();
        deleteAllAndSaveAll(listOfObejcts);
        Util.staticMethod();
    }

    private void deleteAllAndSaveAll(List list) {
        repository.deleteAll();
        repository.saveAll(list);
        // 手动flush，立即执行所有待处理的数据库操作
        repository.flush();
    }
}

方案2：将静态方法移出事务上下文 #

把Util.staticMethod()的调用放到事务方法之外，确保只有当事务成功提交后才执行静态方法：

@Service
public class Service {

    @Transactional(rollbackFor = { Exception.class })
    private void doTransactionalWork() {
        stuffToGetListOfObjects();
        deleteAllAndSaveAll(listOfObejcts);
    }

    public void method1() {
        doTransactionalWork();
        // 仅当事务成功提交后，才执行静态方法
        Util.staticMethod();
    }

    private void deleteAllAndSaveAll(List list) {
        repository.deleteAll();
        repository.saveAll(list);
    }
}

注：这里将事务逻辑拆分为私有方法是可行的，因为事务代理是针对Service类的，只要调用链是从外部进入代理类的公共方法（method1），内部私有方法的事务依然会生效。

方案3：捕获异常并终止后续执行（需结合flush） #

如果需要在method1内部处理逻辑，可以捕获异常并重新抛出（确保事务回滚），同时阻止后续代码执行：

@Transactional(rollbackFor = { Exception.class })
@Service
public class Service {

    public void method1() {
        try {
            stuffToGetListOfObjects();
            deleteAllAndSaveAll(listOfObejcts);
            Util.staticMethod();
        } catch (Exception e) {
            // 重新抛出异常，触发事务回滚
            throw new RuntimeException(e);
        }
    }

    private void deleteAllAndSaveAll(List list) {
        repository.deleteAll();
        repository.saveAll(list);
        repository.flush();
    }
}

总结 #

你的推测是正确的，将静态方法移出事务上下文是最稳妥的方案之一；如果需要保留现有方法结构，手动flush让异常提前抛出也能解决问题。

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