@Transaction注解失效的八种情况及解决办法

来自黑马程序员java面试题

案例准备:

数据库表数据如下,要实现的目标就是刘备曹操孙权向自己手下员工发工资转账,这是一个事务的典型应用场景。
在这里插入图片描述
数据库操作相关方法

 @Override
    public Account getAccountByName(String name) {
        String querySql="select * from t_account where a_name=?";
        BeanPropertyRowMapper<Account> beanPropertyRowMapper = new BeanPropertyRowMapper<>(Account.class);
        return jdbcTemplate.queryForObject(querySql, beanPropertyRowMapper, name);
    }

    @Override
    public int updateAccount(Account account) {
        String updateSql="update t_account set a_balance=? where a_id=?";
        return  jdbcTemplate.update(updateSql, account.getaBalance(), account.getaId());
    }

    @Override
    public void transfer(String sourceName, String objectName, double cash) {
        // 1.得到转账人的信息
        Account A = getAccountByName(sourceName);
        // 2.得到收账人的信息
        Account B = getAccountByName(objectName);
        // 3.修改转账人的金额
        A.setaBalance(A.getaBalance()-cash);
        // 4.修改收账人的金额
        B.setaBalance(B.getaBalance()+cash);
        // 5.更新转账人的信息
        updateAccount(A);
        // 6.更新收账人的信息
        updateAccount(B);
        System.out.println("转账成功");
    }

一.抛出检查异常导致事务不能正确回滚

@Transactional
    @Override
    public void transfer(String sourceName, String objectName, double cash) throws FileNotFindException{
        // 1.得到转账人的信息
        Account A = getAccountByName(sourceName);
        // 2.得到收账人的信息
        Account B = getAccountByName(objectName);
        // 3.修改转账人的金额
        A.setaBalance(A.getaBalance()-cash);
        // 4.修改收账人的金额
        B.setaBalance(B.getaBalance()+cash);
        // 5.更新转账人的信息
        updateAccount(A);
        new FileInputStream("");
        // 6.更新收账人的信息
        updateAccount(B);
        System.out.println("转账成功");
    }
  • 原因:Spring 默认只会回滚非检查异常

  • 解法:配置 rollbackFor 属性

    • @Transactional(rollbackFor = Exception.class)

二.业务方法内自己 try-catch 异常导致事务不能正确回滚

 @Transactional
    @Override
    public void transfer(String sourceName, String objectName, double cash) {
        // 模拟检查时异常
        try {
            // 1.得到转账人的信息
            Account A = getAccountByName(sourceName);
            // 2.得到收账人的信息
            Account B = getAccountByName(objectName);
            // 3.修改转账人的金额
            A.setaBalance(A.getaBalance()-cash);
            // 4.修改收账人的金额
            B.setaBalance(B.getaBalance()+cash);
            // 5.更新转账人的信息
            updateAccount(A);
            new FileInputStream("");
            // 6.更新收账人的信息
            updateAccount(B);
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        }
        System.out.println("转账成功");
    }
  • 原因:事务通知只有捉到了目标抛出的异常,才能进行后续的回滚处理,如果目标自己处理掉异常,事务通知无法知悉

  • 解法1:异常原样抛出

    • 在 catch 块添加 throw new RuntimeException(e);
  • 解法2:手动设置 TransactionStatus.setRollbackOnly()

    • 在 catch 块添加 TransactionInterceptor.currentTransactionStatus().setRollbackOnly();

三.AOP 切面顺序导致导致事务不能正确回滚

@Aspect
@Component
public class LogAspect {
    @Around(value = "execution(* tx.service.impl.AccountServiceImpl.transfer(..))")
    public Object printLog(ProceedingJoinPoint proceedingJoinPoint){
        System.out.println("记入日志");
        try {
            return proceedingJoinPoint.proceed();
        }catch (Throwable e){
            e.printStackTrace();
        }
    }
}
  • 原因:事务切面优先级最低,但如果自定义的切面优先级和他一样,则还是自定义切面在内层,这时若自定义切面没有正确抛出异常…

  • 解法1、2:同情况2 中的解法:1、2

  • 解法3:调整切面顺序,在 MyAspect 上添加 @Order(Ordered.LOWEST_PRECEDENCE - 1) (不推荐)

四.非 public 方法导致的事务失效

@Transactional
    @Override
    void transfer(String sourceName, String objectName, double cash) throws FileNotFindException{
        // 1.得到转账人的信息
        Account A = getAccountByName(sourceName);
        // 2.得到收账人的信息
        Account B = getAccountByName(objectName);
        // 3.修改转账人的金额
        A.setaBalance(A.getaBalance()-cash);
        // 4.修改收账人的金额
        B.setaBalance(B.getaBalance()+cash);
        // 5.更新转账人的信息
        updateAccount(A);
        new FileInputStream("");
        // 6.更新收账人的信息
        updateAccount(B);
        System.out.println("转账成功");
    }
  • 原因:Spring 为方法创建代理、添加事务通知、前提条件都是该方法是 public 的

  • 解法1:改为 public 方法

  • 解法2:添加 bean 配置如下(不推荐)

@Bean
public TransactionAttributeSource transactionAttributeSource() {
    return new AnnotationTransactionAttributeSource(false);
}

五.父子容器导致的事务失效

package day04.tx.app.service;

// ...

@Service
public class Service5 {

    @Autowired
    private AccountMapper accountMapper;

    @Transactional(rollbackFor = Exception.class)
    public void transfer(int from, int to, int amount) throws FileNotFoundException {
        int fromBalance = accountMapper.findBalanceBy(from);
        if (fromBalance - amount >= 0) {
            accountMapper.update(from, -1 * amount);
            accountMapper.update(to, amount);
        }
    }
}

控制器类

package day04.tx.app.controller;

// ...

@Controller
public class AccountController {

    @Autowired
    public Service5 service;

    public void transfer(int from, int to, int amount) throws FileNotFoundException {
        service.transfer(from, to, amount);
    }
}

App 配置类

@Configuration
@ComponentScan("day04.tx.app.service")
@EnableTransactionManagement
// ...
public class AppConfig {
    // ... 有事务相关配置
}

Web 配置类

@Configuration
@ComponentScan("day04.tx.app")
// ...
public class WebConfig {
    // ... 无事务配置
}

现在配置了父子容器,WebConfig 对应子容器,AppConfig 对应父容器,发现事务依然失效

  • 原因:子容器扫描范围过大,把未加事务配置的 service 扫描进来

  • 解法1:各扫描各的,不要图简便

  • 解法2:不要用父子容器,所有 bean 放在同一容器

六.调用本类方法导致传播行为失效

@Service
public class Service6 {

    @Transactional(propagation = Propagation.REQUIRED, rollbackFor = Exception.class)
    public void foo() throws FileNotFoundException {
        LoggerUtils.get().debug("foo");
        bar();
    }

    @Transactional(propagation = Propagation.REQUIRES_NEW, rollbackFor = Exception.class)
    public void bar() throws FileNotFoundException {
        LoggerUtils.get().debug("bar");
    }
}

在这里插入图片描述

  • 原因:本类方法调用不经过代理,因此无法增强(bar()方法是代理对象调用的,foo中方法是对象本身的调用)

  • 解法1:依赖注入自己(代理)来调用

  • 解法2:通过 AopContext 拿到代理对象,来调用

解法1

@Service
public class Service6 {

	@Autowired
	private Service6 proxy; // 本质上是一种循环依赖

    @Transactional(propagation = Propagation.REQUIRED, rollbackFor = Exception.class)
    public void foo() throws FileNotFoundException {
        LoggerUtils.get().debug("foo");
		System.out.println(proxy.getClass());
		proxy.bar();
    }

    @Transactional(propagation = Propagation.REQUIRES_NEW, rollbackFor = Exception.class)
    public void bar() throws FileNotFoundException {
        LoggerUtils.get().debug("bar");
    }
}

解法2,还需要在 AppConfig 上添加 @EnableAspectJAutoProxy(exposeProxy = true)

@Service
public class Service6 {
    
    @Transactional(propagation = Propagation.REQUIRED, rollbackFor = Exception.class)
    public void foo() throws FileNotFoundException {
        LoggerUtils.get().debug("foo");
        ((Service6) AopContext.currentProxy()).bar();
    }

    @Transactional(propagation = Propagation.REQUIRES_NEW, rollbackFor = Exception.class)
    public void bar() throws FileNotFoundException {
        LoggerUtils.get().debug("bar");
    }
}

七.@Transactional 没有保证原子行为

**

@Service
public class Service7 {

    private static final Logger logger = LoggerFactory.getLogger(Service7.class);

    @Autowired
    private AccountMapper accountMapper;

    @Transactional(rollbackFor = Exception.class)
    public void transfer(int from, int to, int amount) {
        int fromBalance = accountMapper.findBalanceBy(from);
        logger.debug("更新前查询余额为: {}", fromBalance);
        if (fromBalance - amount >= 0) {
            accountMapper.update(from, -1 * amount);
            accountMapper.update(to, amount);
        }
    }

    public int findBalance(int accountNo) {
        return accountMapper.findBalanceBy(accountNo);
    }
}

上面的代码实际上是有 bug 的,假设 from 余额为 1000,两个线程都来转账 1000,可能会出现扣减为负数的情况(两个线程都查询出1000然后都去执行转账操作导致出现负数情况)

  • 原因:事务的原子性仅涵盖 insert、update、delete、select … for update 语句,select 方法并不阻塞

在这里插入图片描述

  • 如上图所示,红色线程和蓝色线程的查询都发生在扣减之前,都以为自己有足够的余额做扣减

八.@Transactional 方法导致的 synchronized 失效

针对上面的问题,能否在方法上加 synchronized 锁来解决呢?

@Service
public class Service7 {

    private static final Logger logger = LoggerFactory.getLogger(Service7.class);

    @Autowired
    private AccountMapper accountMapper;

    @Transactional(rollbackFor = Exception.class)
    public synchronized void transfer(int from, int to, int amount) {
        int fromBalance = accountMapper.findBalanceBy(from);
        logger.debug("更新前查询余额为: {}", fromBalance);
        if (fromBalance - amount >= 0) {
            accountMapper.update(from, -1 * amount);
            accountMapper.update(to, amount);
        }
    }

    public int findBalance(int accountNo) {
        return accountMapper.findBalanceBy(accountNo);
    }
}

答案是不行,原因如下:

  • synchronized 保证的仅是目标方法的原子性(因为实际调用过程是代理对象调用方法),环绕目标方法的还有 commit 等操作,它们并未处于 sync 块内
  • 可以参考下图发现,蓝色线程的查询只要在红色线程提交之前执行,那么依然会查询到有 1000 足够余额来转账

在这里插入图片描述

  • 解法1:synchronized 范围应扩大至代理方法调用

  • 解法2:使用 select … for update (数据库的锁)替换 select

  • 13
    点赞
  • 55
    收藏
    觉得还不错? 一键收藏
  • 打赏
    打赏
  • 2
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 2
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

活跃的咸鱼

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值