线上故障之-redis锁处理幂等性失效和幂等性问题解决方案

redis锁处理幂等性失效

 @Override
    @Transactional(rollbackFor = Exception.class)
    public void addError(User user) {
   
   
        log.info("add user params user:{}", JSON.toJSONString(user));
        Assert.isTrue(StringUtils.isNotBlank(user.getIdCard()), "身份证号不允许null");
        String key = "key";
        RLock lock = redissonClient.getLock(key);
        lock.lock();
        try {
   
   
            LambdaQueryWrapper<User> wrapper = new LambdaQueryWrapper<User>()
                    .eq(User::getIdCard, user.getIdCard());
            long count = userMapper.selectCount(wrapper);
            if (count == 0) {
   
   
                userMapper.insert(user);
            }
        } catch (Exception e) {
   
   
            log.error("add user error", e);
        } finally {
   
   
            lock.unlock();
        }
        System.out.println("并发执行，同时插入了两条");
    }

以上代码问题：
1：对事物的理解使用有问题，幂等设计bug
2：redis锁使用有问题（单独案例讲述）
详细解释：
锁是在事务内，多线程进来是不同的事务，到System.out.println("并发执行，同时插入了两条");事务还未提交，所以查不到数据，但锁却释放了。

优化

    @Override
    @Transactional(rollbackFor = Exception.class)
    public void addOk(User user) {
   
   
        LambdaQueryWrapper<User> wrapper = new LambdaQueryWrapper<User>()
                .eq(User::getIdCard, user.getIdCard());
        long count = userMapper.selectCount(wrapper);
        if (count == 0) {
   
   
            userMapper.insert(user);
        }
    }

   //正例
   @Test
    public void addTest() {
   
   
       User user = new User();
        user.setName("张三");
        user.setIdCard("1000");
        String key = "key";
        RLock lock = redissonClient.getLock(key);
        lock.lock();
        try {
   
   
            userService.addOk(user);
        } catch (Exception e) {
   
   
            e.printStackTrace();
        }finally {
   
   
            lock.unlock();
        }
        }

拆开后，锁的时候把整个事务都锁住。

扩展：
事务在生产实践中经常犯的错误：
事务范围：应该加入事务的代码未加入到事务中
下图是另一个真实生产当中的事故-仅供参考：