多线程下ConcurrentHashMap一次诡秘的应用失败

         一次聊天服的压力测试中,偶然发现某个用户竟然收不到别的用户广播的消息,跟踪日志,该用户已成功登录,难道用户加入该消息广播组失败了?然而日志显示用户也已经成功加入了该消息广播组。。。
       经过仔细检查,最终锁定了以上代码, 有什么问题么?</
 		//key为组名,value为该组所有在线用户
		ConcurrentHashMap<String, CopyOnWriteArrayList
  
  
   
   > groupSessions
		
		
  
  
		List groupList = groupSessions.get(groupName);
		//检查是否已经存在该组,如果不存在,创建该组list并加入该组
		if (groupList == null) {
			groupList=new CopyOnWriteArrayList();
			groupSessions.putIfAbsent(groupName,groupList);
		}
		//将用户session加入到该组
		groupList.add(session);

       单机测试或者用户很少的时候,这段代码应该没什么问题(只是应该哦)。但是如果并发的话,线程A和线程B几乎同时进入if (groupList == null) 这段代码,线程A先执行完,所以groupList加入groupSessions,线程B执行groupSessions.putIfAbsent(groupName,groupList)由于该组已经存在(线程A加入),所以线程B的groupList没有加入到groupSessions组,而该用户的session加入的groupList不存在groupSessions组。那么广播该组消息时自然就该用户收不到消息了。

      修改如下:

		List groupList = groupSessions.get(groupName);
		//检查是否已经存在该组,如果不存在,创建该组list并加入该组
		if (groupList == null) {
			groupList=new CopyOnWriteArrayList();
			groupList=groupSessions.putIfAbsent(groupName,groupList);
		}
		//将用户session加入到该组
		groupList.add(session);
       一行只差啊。
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 打赏
    打赏
  • 0
    评论

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

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

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

无心的雨

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

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

余额充值