java8使用parallelStream遍历导致线程不安全问题

因为parallelStream使用了多线程所以难以保证线程安全,容易导致数据的丢失
原本代码:不安全

public List<List<Object>> getReportListBody(List<ReportList> ReportLists) {
	List<List<Object>> result = new LinkedList<>();
   	ReportLists.parallelStream().forEach(reportList -> {
   	List<Object> row = new LinkedList<>();
   	...处理代码
	result.add(row);
  });

现代码:

public List<List<Object>> getReportListBody(List<ReportList> ReportLists) {
	List<List<Object>> result = Collections.synchronizedList(new LinkedList<>());
   	ReportLists.parallelStream().forEach(reportList -> {
   	List<Object> row = new LinkedList<>();
   	...处理代码
	result.add(row);
  });

synchronizedList的源码:

 final Object mutex;     // Object on which to synchronize
 public void add(int index, E element) {
 	synchronized (mutex) {list.add(index, element);}
 }
 public E remove(int index) {
 	synchronized (mutex) {return list.remove(index);}
 }

synchronizedList的add和remove都是被synchronized修饰的所以可以保证线程安全,不会导致数据丢失

推荐的方式:

 List<List<Object>> result = ReportLists.parallelStream().map(reportList -> {
            List<Object> row = new LinkedList<>();
            ...处理代码
   return row;
 }).collect(Collectors.toList());

错误的版本:

public List<List<Object>> getReportListBody(List<ReportList> ReportLists) {
	List<List<Object>> result = new LinkedList<>();
	  synchronized (result){
   	ReportLists.parallelStream().forEach(reportList -> {
   	List<Object> row = new LinkedList<>();
   	...处理代码
	result.add(row);
  });
  }

我本以为给result加锁就能够保证线程安全,但是结果还是丢了数据
目前还不清楚为什么

  • 0
    点赞
  • 1
    收藏
    觉得还不错? 一键收藏
  • 3
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 3
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值