@Transactional 注解的错误使用场景-优快云博客

1. 无需事务的业务

在没有事务操作的业务方法上使用 @Transactional 注解，比如：用在仅有查询或者一些 HTTP
请求的方法，虽然加上影响不大，但从编码规范的角度来看还是不够严谨，建议去掉。

@Transactional
public String test() {
    standardBak2Service.getById(1);
    return "test";
}

2. 事务范围过大

有些同学为了省事直接将 @Transactional 注解加在了类上或者抽象类上，这样做导致的问题就是类内的方法或抽象类的实现类中所有方法全部都被事务管理。增加了不必要的性能开销或复杂性，建议按需使用，只在有事务逻辑的方法上加@Transactional。

@Transactional
public abstract class BaseService {
}

@Slf4j
@Service
public class MyService extends BaseService{

    private final TestService testService;

    public String test() {

        testService.test();

        return "ok";
    }
}

3. 方法权限问题

不要把 @Transactional注解加在 private 级别的方法上！
(用 final 、static 修饰方法也不会生效)
我们知道 @Transactional 注解依赖于Spring AOP切面来增强事务行为，这个 AOP 是通过代理来实现的，而 private 方法恰恰不能被代理的，所以 AOP 对 private 方法的增强是无效的，@Transactional也就不会生效。

@Transactional
private String test() {

    testAService.testA();

    testBService.testB();

    return "ok";
}

那如果我在 test() 方法内调用 private 的方法事务会生效吗？

答案：事务会生效

@Transactional
public String test() throws Exception {

    ccc();
    
    return "ok";
}

private void ccc() {
    testAService.testA();

    testBService.testB();
}

4. 同类内部方法调用问题

注意了，这种情况经常发生啊！
同类内部方法间的调用是 @Transactional 注解失效的重灾区，网上你总能看到方法内部调用另一个同类的方法时，这种调用是不会经过代理的，因此事务管理不会生效。但这说法比较片面，要分具体情况。
比如：test() 方法开启事务，调用同类非事务的方法 a() 和 b() ，此时 b() 抛异常，根据事务的传播性 a()、b() 事务均生效。

@Transactional
public String test() {

    a();

    b();

    return "ok";
}

public void a() {
    myBaseService.save(testAService.entity1());
}

public void b() {
    myBaseService.save(testBService.entity2());
    throw new RuntimeException("error");
}

5. 异步线程调用

如果我们在 test() 方法中使用异步线程执行事务操作，通常也是无法成功回滚的，来个具体的例子。
test() 方法在事务中调用了 testA()，testA() 方法中开启了事务。接着，在 test() 方法中，我们通过一个新线程调用了 testB()，testB() 中也开启了事务，并且在 testB() 中抛出了异常。
testA() 不会回滚和 testB() 回滚。
testA() 无法回滚是因为没有捕获到新线程中 testB()抛出的异常；testB()方法正常回滚。
在多线程环境下，Spring 的事务管理器不会跨线程传播事务，事务的状态（如事务是否已开启）是存储在线程本地的 ThreadLocal 来存储和管理事务上下文信息。这意味着每个线程都有一个独立的事务上下文，事务信息在不同线程之间不会共享。

@Transactional
public String test() {

    testAService.testA();

    new Thread(() -> {
        try {
            myTestService.testB();
        } catch (Exception e) {
            throw new RuntimeException();
        }
    }).start();

    return "ok";
}

@Transactional
public String testB() {
    DeepzeroStandardBak2 entity2 = buildEntity2();

    dataImportJob2Service.save(entity2);

    throw new RuntimeException("test2");
}

@Transactional
public String testA() {
    DeepzeroStandardBak entity = buildEntity();
    myTestService.save(entity);
    return "ok";
}