关于synchronized失效wenti

  国庆前公司的商城做了一波活动，导致平时流水五六千的商城在9月30日的一天流水暴涨到近40W，随之而来的就是一波大坑。

简单介绍一下业务场景，一个商城系统，一个发码系统。商城是卖电子券的，用户在商城购买卡券，商城系统就会调用发码系统给客户进行发码（短信形式），然后把发送的码返回给商城系统存入卡包。我们有部分商城的券码是购买第三方平台的券码（如星巴克，肯德基等），购买后把对应的码导入到发码平台。当用户兑换某个券码时，就会到发码平台获取未发送的卡券。平时流量小，问题没有暴露出来，在9月30日这天因为并发量太大，导致有很多客户收到同一样的券码。发现问题时，第一时间去看业务流程（因为是接手的老项目所有并不是非常熟悉），发现流程很简单就是先按照商品Id与状态查询数据库未发送的券，然后更新状态为已发送，最后返回。当时就觉得这个问题很简单，就直接在方法上加了synchronized，然后更新重启。

  再次去观察兑换记录，我勒个xx，发现居然还出现了重复发码的情况。当时就慌了，synchronized怎么会不生效勒，马上就尝试使用this锁，class锁，静态变量锁，发现都没有用，问题并没有解决。最后实在没有办法在controller加上锁，重复发码的问题就没有再出现了，暂时处理掉问题。但是一般情况下项目经理是不会允许这样的，身为编程者也不会容许这样的代码出自自己之手。

 暂时补上漏洞继续查找问题，数据缓存，mabatis一级缓存各个地方都看过了，并没有发现问题出现的地方。最好在查看配置文件时发现了下面的配置:

我勒个圈圈，事物的传播行为，并且覆盖了整个业务实现，随之暂时注释掉这些配置，测试发现重复发码不再出现。随之修改对应代码，测试，更新服务器，观察运行情况，问题得到了解决。

总结：锁与事物同时存在时，锁是不生效的。出现synchronized失效的问题，多检查是否与事物存在冲突。

如果要需要与事物同时存在，那就就扩大锁的范围，用锁包含事物。参考链接：https://www.cnblogs.com/Java3y/p/10392645.html