@Transactional方法调用外部方法报错未回滚问题求助

核心问题：异常被"吞"了，Spring事务没感知到错误 #

Spring的@Transactional默认只有在未被捕获的RuntimeException/Error抛出时，才会触发事务回滚。你的代码里有两处把异常吞了：

• 主事务方法transactionMethod里的try-catch直接捕获所有异常，只打日志不抛出，事务管理器根本不知道发生了错误，自然不会回滚。
• serviceA和serviceC的方法里也把save()的异常捕获了，就算save失败，错误也传不到上层事务方法里。

具体修复步骤 #

1. 让事务方法感知到异常 #

要么去掉主方法的try-catch，要么在catch块里重新抛出异常，或者手动触发回滚：

方案一：重新抛出异常

import javax.transaction.Transactional;

@Component
public class ComponentClass {
    @Transactional
    public void transactionMethod() {
        try {
            serviceA.method1();
            serviceB.method2();
        } catch (Exception e) {
            log.error("执行失败", e);
            // 重新抛出RuntimeException，让事务管理器感知错误
            throw new RuntimeException(e);
        }
    }
}

方案二：手动标记回滚

如果不想抛出异常，也可以手动标记事务需要回滚：

import org.springframework.transaction.interceptor.TransactionAspectSupport;

@Component
public class ComponentClass {
    @Transactional
    public void transactionMethod() {
        try {
            serviceA.method1();
            serviceB.method2();
        } catch (Exception e) {
            log.error("执行失败", e);
            // 手动通知事务管理器回滚
            TransactionAspectSupport.currentTransactionStatus().setRollbackOnly();
        }
    }
}

2. 不要在Service层吞异常 #

serviceA和serviceC里的try-catch会把save()的错误吃掉，导致上层无法感知失败。要么去掉这些catch，要么把异常转成RuntimeException抛出：

修改后的ServiceA示例：

@Service
public class ServiceA {
    public void method1() {
        // 去掉不必要的catch，让异常自然向上传递
        save();
        
        // 若必须捕获异常，就重新抛出RuntimeException
        /*
        try {
            save();
        } catch (Exception e) {
            log.error("保存失败", e);
            throw new RuntimeException(e);
        }
        */
    }
}

关于事务传播的补充 #

你理解的默认传播行为（Propagation.REQUIRED）是正确的：当前无事务则新建，有事务则加入。但这个逻辑生效的前提是异常能传递到事务方法，如果中间被吞掉，事务管理器就不会触发回滚逻辑。

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