并发踩坑:list共享变量的addAll

起名不费劲2023-12-27 16:47

背景：

某业务报错了，提示：Caused by: org.apache.ibatis.exceptions.PersistenceException:

Error querying database. Cause: java.util.ConcurrentModificationException

分析：

这是执行查询时报的并发修改异常。大概逻辑是在多线程场景下，有一个共享变量list对象，每个线程都对它执行addAll操作。导致MySQL查询语句执行的时候，这个list还在并发修改，所以抛出的错误。

java 复制代码

public void restList() {
    
    Thread thread1 = new Thread("线程1"){
        @Override
        public void run() {
            innerSide();
        }
    };
    thread1.start();

    Thread thread2 = new Thread("线程2"){
        @Override
        public void run() {
            innerSide();
        }
    };
    thread2.start();                
    
}

public void innerSide(){
    List<String> list = nacos.getList();
    //List<String> list = new ArrayList<>();
    List<String> querys = mapper.selectList(queryWrapper);
    if(CollectionUtils.isNotEmpty(querys)){
        list.addAll(querys);
    }
    System.out.println(Thread.currentThread().getName() +JSONObject.toJSONString(list));

    List<Object> objectList = otherMapper.queryList(list);
}

代码逻辑如上所示，大概就是多个线程并发执行，调用innerSide()方法，但是这个方法里的list，是从nacos上读到的。所以list变量指向的是一个共享的引用地址。所以后续对list进行 addAll操作，每个线程都执行一次。导致list里的值呈指数级别增长。导致后边代码 otherMapper.queryList()方法执行的时候，mybatis报错并发修改。

修改：

list改成new的方式，不要用共享变量。