商城系统开发从0到1（解决事务和锁引起的问题）

开发商城系统的过程中遇到了个问题，在这里记录一下。

说到商品，那就有库存的问题，那么如何解决在分布式系统下商品的库存问题呢，那肯定要考虑到分布式锁了，一开始想自己实现Redis分布式锁，后面看到网上有开源的klock 码云地址： https://gitee.com/kekingcn/spring-boot-klock-starter，上手非常的简单。

在这里就不多做介绍了，那接下来问题来了，附上代码。

@Override
    @Klock(keys = {"#productId"})
    @Transactional
    public  ResultMap subStock(String productId,Long count,String userid) {
        Product product=new Product();
        product.setStock(productDetailAdminDto.getStock()-count);
        boolean flag=this.update(product,new EntityWrapper<Product>().eq("ID",productId));
        if (flag){
            return ResultUtil.success();
        }else{
            return ResultUtil.error("更新商品库存失败");
        }

    }

使用了Apache Ab做压测 data.txt 放到bin目录下，存放着请求的参数，json格式的。 -H 放请求头的信息，使用了JWTtoken，-n 请求次数  -c 并发次数

ab -p data.txt -T application/json -H "X-TOKEN:eyJ0eXAiOiJKV1QiLCJhbG" -n 1000 -c 100 http://172.0.0.1:9069/product/subStock

在压测100并发请求下，偶尔会出问题，但是压测500并发请求肯定出现问题，库存会出现超卖的问题。一开始还在怀疑是否分布式锁不生效，后来在单机下测试使用synchronized一样出现问题。接下来我定位到了问题，是事务引起的，果不其然。根据以上的排查过程，已经很清楚的确认了事务与锁之间存在的问题。由于事务范围大于锁代码块范围，在锁代码块执行完成后，此时事务还未提交，导致此时进入锁代码块的其他线程，读到的仍是原有的库存数据。

 

下面给出我的解决方案

@Override
    @Klock(keys = {"#productId"})
    @Transactional(propagation = Propagation.NOT_SUPPORTED)
    public  ResultMap subStock(String productId,Long count,String userid) {
  
        Product product=new Product();
        product.setStock(productDetailAdminDto.getStock()-count);
        boolean flag=this.update(product,new EntityWrapper<Product>().eq("ID",productId));
        if (flag){
            return ResultUtil.success();
        }else{
            return ResultUtil.error("更新商品库存失败");
        }

    }

就是把事务设置成NOT_SUPPORT，这样我们这个方法下的事务就不会生效了，因为这个是需要被提交订单方法调用的，而提交订单是有事务的，所以这里才要这样设置，避免被事务影响到而出现的数据脏读。不过因为我这个方法里面只有一个更新操作，所以可以不用考虑事务的问题，如果是有多表的更新，要手动开启事务和事务回滚。