新项目上线,测试的时候接口没有任何问题,上线两天后,发现有极个别用户的数据出现异常(金币变多的情况)。
发现出现问题为同一时间取消市场订单多次,执行了多次导致用户增加了多次资产。
再线下测试的时候 想过接口重复点击问题。自己测试,没法发现任何问题。前端做了接口防抖,后端也有redis锁来控制
接口重复提交:@Lock
部分代码:
我使用多线程请求接口 有几率不会被拦截,接口会被执行多次
public static void main(String[] args) {
// 创建一个固定大小为2的线程池
ExecutorService executorService = Executors.newFixedThreadPool(2);
// 提交两个任务到线程池执行
for (int i = 0; i < 2; i++) {
executorService.submit(() -> {
String body = HttpRequest.post("http://xxx.com/cancel")
.header("Authorization", "token")
.body("{\"id\":2562}")
.timeout(20000)
.execute().body();
System.out.println(LocalDateTime.now());
System.out.println(body);
});
}
executorService.shutdown();
while (!executorService.isTerminated()) {
}
System.out.println("所有任务已完成");
}
在看解决方法的时候,想过使用事务隔离级别控制顺序执行,这是花费最高代价但是最可靠的事务隔离级别。事务被处理为顺序执行。所以肯定不能这么做。
于是乎根据sql执行日志查询出,第二次执行方法修改的数据数量为0,所以我就在最后面判断如何updateById 返回为false的时候,返回异常,让sql回滚。
这个方法不行!
最终还是用上了事务控制:
@Transactional(rollbackFor = Exception.class, propagation = Propagation.REQUIRED, isolation = Isolation.SERIALIZABLE)
真的稳!