@Transactional注解的一个很容易被忽略的错误用法

@Transactional注解的一个很容易被忽略的错误用法

今日审查代码时发现对@Transactional注解的一个如下用法:

less 复制代码
//StockController.java

@AutoWired
private StockService stockService

//商品出库
@PostMapping("/product/stock/out-stock")
public Boolean productOutStock(@RequestBody StockChangeDTO data) {
  return stockService.outStock(data);
}

//StockService服务
public Boolean outStock(StockChangeDTO data) {
  //此处省略相关数据检查的前置代码
  ..........
  
  //执行出库的逻辑,acOut表示出库操作
  return this.doProductStockChange(data, acOut)
}

//执行出库的逻辑
@Transactional(rollback = Exception.class)
public Boolean doProductStockChange(StockChangeDTO data, StockChangeAction action) {
  //记录出入库记录
  
  //改变商品库存
  
  //省略其他逻辑代码
  .............
}

这段代码的逻辑是在StockService.outStock方法中调用了一个公用方法doProductStockChange完成出库逻辑,在doProductStockChange方法中涉及多笔数据操作,有记录出入库记录,有更改商品库存数据等等,为保证数据的一致性,这里添加了事物注解@Transactional来保证数据的一致性,乍一看好像没什么问题。

但是仔细一思考,这么用真的能保证事物能生效吗?

了解@Transactional注解的原理,当我们在方法上使用 @Transactional 注解时,Spring 将会创建一个代理对象来包装该方法。该代理对象将在该方法执行之前和之后添加一些代码,以启动和提交/回滚事务。如下图所示:

也就是说我们在StockController中注入StockService的Bean时,如果StockService中没有用到@Transactional之类的注解,那么注入的就是StockService服务本身的bean,相反如果有这类的注解时,那么注入的就不再是StockService本身的bean了,而是经过包装了一层的代理对象,这个代码对象在方法调用前会启动事物,在方法调用后提交事物,发生异常时执行事物回滚,这样就能保证事物是生效的。

我们再来看看StockService.java中outStock方法调用this.doProductStockChange方法,虽然doProductStockChange方法添加了事物注解,但是由于这个时候的this对象就是StockService本身而不是代理对象,因此事物就是无效的,这段代码并不能保证出入库时的数据一致性。

解决办法有以下几种:

1、在StockService.java中注入自己,然后调用doProductStockChange方法改为调用注入对象的;

kotlin 复制代码
//StockService.java

@AutoWired
private StockService stockService

public Boolean outStock(StockChangeDTO data) {
  //此处省略相关数据检查的前置代码
  ..........
  
  //执行出库的逻辑,acOut表示出库操作
  return stockService.doProductStockChange(data, acOut)
}

2、将outStock方法中的数据检查代码前置到controller中去检查,然后直接调用stockService.doProductStockChange方法。

第一种方法属于为了临时快速解决问题的偏门方法,对代码的可阅读性造成伤害,个人建议还是第二种方法更好更值得推荐使用。 仔细检查下代码中是否还有其他注解类似的错误,比如@ASync, @Cacheable、@CachePut、@CacheEvict, @Scheduled等等。

相关推荐
9ilk5 分钟前
【同步/异步 日志系统】--- 介绍
后端·中间件
Imnobody14 分钟前
吴恩达 Prompt 工程课精讲③:三次迭代,解决 Prompt 的三大致命问题(JupyterLab 实战)
后端
泉城老铁23 分钟前
springboot 对接发送钉钉消息,消息内容带图片
前端·spring boot·后端
千桐科技31 分钟前
qKnow 知识平台【开源版】安装与部署全指南
人工智能·后端
qq_124987075333 分钟前
基于Spring Boot的高校实习实践管理系统(源码+论文+部署+安装)
java·spring boot·后端·毕业设计
韩宁羽33 分钟前
SpringBoot开发双11商品服务系统[完结19章]
spring boot
作伴35 分钟前
结合Apollo手动实现一个动态线程池jar包
后端·架构
该用户已不存在35 分钟前
7个没听过但绝对好用的工具
前端·后端
Java水解36 分钟前
KingbaseES SQL性能调优方案分享
后端·sql
用户40993225021237 分钟前
PostgreSQL连接的那些弯弯绕:笛卡尔积、外连接和LATERAL你都理明白没?
后端·ai编程·trae