被自己蠢哭。。。。 怎么肥事!!!!这种问题还犯。。。
1.需求
有一个集合,不确定里面排序顺序是怎么样的,不确定里面是否有重复元素,数据根据每次请求不同数据也不同 现在就是需要将重复的去除,只取非重复元素放入新的集合。 下面是我的错误代码。。。公开判刑
2.逻辑进展
2.1错误1:再集合循环的时候取改动集合
List<Map<String,Object>> queryList1 = queryFromDB(id);
List<Map<String,Object>> queryList2 = new ArrayList<>();
if(queryList1!=null && queryList1.size!=0){
foreach(Map<String,Object> map1 : queryList1 ) {
if(queryList2 ==null){
queryList2.add(map1);
}else{
foreach(Map<String,Object> map2 : queryList2 ) {
if(!map2.get("ID").equals(map1.get("ID"))
queryList2 .add(map1);
}
}
}
}
错误显而易见,在queryList2 循环的时候去add()元素。。。。 报错:ConcurrentModificationException
Concurrent:并存的,同时发生的
2.2 错误2 :集合并非有序,对比逻辑错误
经过错误1 之后,我想这用一个临时的list 去缓存一下元素,在queryList2 循环外添加不就好了。确实可以避免错误1 。但是对比逻辑错误了,导致产生了多条数据且重复。。。
List<Map<String,Object>> temp = new ArrayList<>();
if(queryList1!=null && queryList1.size!=0){
foreach(Map<String,Object> map1 : queryList1 ) {
if(queryList2 ==null){
queryList2.add(map1);
}else{
foreach(Map<String,Object> map2 : queryList2 ) {
if(!map2.get("ID").equals(map1.get("ID"))
temp.add(map1);
}
queryList2 .addALL(temp);
}
}
}
假设:queryList2里面存了一个map2了,来了map1对比,不一样,放入queryList2。 map1再循环变化,再进入queryList2循环,取第一个map2,不一样就,放入queryList2;queryList2再循环取第二个map2,还是不一样,再次放入。。。 以此类推,错了个大离谱。。。
2.3 正确的:map转实体类,然后用Map<Integer,User>的contain()方法比较
List<Map<String,Object>> queryList1 = queryFromDB(id);
List<User> userList = new ArrayList<User>();
Map<Integer,User> map_user = new HashMap();
if(queryList1!=null && queryList1.size!=0){
foreach(Map<String,Object> map1 : queryList1 ) {
User user = JSON.parseObject(JSON.toJSONString(map1),User.class);
userList.add(user);
}
}
if(userList!=null){
Iterator i = userList.iterator();
while(i.hasNext()){
User user=(User)i.next();
Interger id= user.getID_I();
if(!map_user.containsKey(id)){
map_user.put(id,user);
}
}
}
Map<id,user>=这样直接取出key就很方便对比数据。 感谢优秀代码,学习学习,加把劲啊,这种错误可是不要再犯了。。。
|