项目中多线程修改同一引用引发的线程安全问题

问题描述

项目中存在如下伪代码：
其中customThreadPool变量是项目自定义的线程池，fileBeatInstanceDTO变量是线程池外部创建的一个对象。
最后issueOperator.operateWhenFirstDeploy方法接收fileBeatInstanceDTO参数。

FileBeatInstanceDTO fileBeatInstanceDTO = new FileBeatInstanceDTO();
 ......
 ......
 //赋值
 fileBeatInstanceDTO.setHosts(hostsMap.get(hosts.getHostId()));
 fileBeatInstanceDTO.setCollectInstanceMappingId(finalCollectInstanceMappingId);
 ......
 ......
 //自定义的线程池
 customThreadPool.execute(()->{
      try {

          ......
          ......
          //接收fileBeatInstanceDTO参数
          issueOperator.operateWhenFirstDeploy(fileBeatInstanceDTO);
      } catch (Exception e) {
          log.error("", e);
      }
  });

当customThreadPool提交多个任务时，issueOperator.operateWhenFirstDeploy方法运行的结果，偶尔会与预期不一致。

问题分析

在issueOperator.operateWhenFirstDeploy方法内部打印fileBeatInstanceDTO变量的值，发现有多个任务时，偶尔存在多个任务的fileBeatInstanceDTO里属性一致的情况。

但实际业务需要的是，不同任务的fileBeatInstanceDTO属性不一样。

分析

issueOperator.operateWhenFirstDeploy方法接收的fileBeatInstanceDTO是同一个引用。

当A任务的issueOperator.operateWhenFirstDeploy方法在运行时，B任务可能会通过如下代码修改这个引用，导致A任务预期的值错误

 fileBeatInstanceDTO.setHosts(hostsMap.get(hosts.getHostId()));
 fileBeatInstanceDTO.setCollectInstanceMappingId(finalCollectInstanceMappingId);

解决

将fileBeatInstanceDTO定义在线程池内部

 ......
 ......
 //自定义的线程池
 customThreadPool.execute(()->{
	 FileBeatInstanceDTO fileBeatInstanceDTO = new FileBeatInstanceDTO();
 		......
	 ......
	 //赋值
	 fileBeatInstanceDTO.setHosts(hostsMap.get(hosts.getHostId()));
	 fileBeatInstanceDTO.setCollectInstanceMappingId(finalCollectInstanceMappingId);
      try {

          ......
          ......
          //接收fileBeatInstanceDTO参数
          issueOperator.operateWhenFirstDeploy(fileBeatInstanceDTO);
      } catch (Exception e) {
          log.error("", e);
      }
  });