有时会抛出 java.util.ConcurrentModificationException

Question

我执行下面的代码，但有时我得到 java.util.ConcurrentModificationException 异常..但有时工作正常。请让我知道我在哪里做错了代码。下面是我的代码，如果有更好的，请检查我的逻辑那么请告诉我。

    public  String saveSkills(HttpServletRequest request,@RequestParam String skills,@RequestParam String Email) throws IOException
    {

        Domain domain1 = (Domain)request.getSession().getAttribute("Domain");
        Long domanId =domain1.getDomainId();
        System.out.println(skills);
        String[] skillsParts = skills.split(",");
        UserProfile user = userProfileManager.getUserByEmail(domain1.getPrimary_Domain_Id(), Email);
        if(user.getSkillsList().size()>0){
            Iterator it = user.getSkillsList().iterator();
            while (it.hasNext())
            {
                Skills skillsitereator = (Skills) it.next();
                int count=0;

                for(int i =0;i<skillsParts.length;i++){

                    if((skillsParts)[i].equals(skillsitereator.getSkillName())){

                        break;

                    }else{

                        count++;
                    }
                }
                if(count == skillsParts.length){
                    it.remove();
                    userProfileManager.update(user);
                }
            }
        }else{

            for(int i =0;i<skillsParts.length;i++){

                Skills skillObj = new Skills();
                skillObj.setSkillName(skillsParts[i]);
                user.getSkillsList().add(skillObj);

            }
            userProfileManager.update(user);
        }
        skillsParts = skills.split(",");
        System.out.println(skillsParts);
        ArrayList<Integer> values =new ArrayList<Integer>();
        for(int i =0;i<skillsParts.length;i++){
            Iterator it = user.getSkillsList().iterator();
            while (it.hasNext())
            {
                Skills skillsitereator = (Skills) it.next();
                if((skillsParts)[i].trim().equals(skillsitereator.getSkillName().trim())){
                    break;
                }
                else{

                    Skills skillObj = new Skills();
                    skillObj.setSkillName(skillsParts[i]);
                    user.getSkillsList().add(skillObj);
                    userProfileManager.update(user);
                }
            }
        }
        Gson gson = new Gson();
        return gson.toJson(user);
    }

Answer 1

这是来自ConcurrentModificationException 的 JavaDoc：

例如，如果一个线程在使用 fail-fast 迭代器对集合进行迭代时直接修改了一个集合，则迭代器将抛出此异常。

在你的最后一个循环中，你有时会这样做

user.getSkillsList().add(skillObj);

在使用 user.getSkillsList().iterator() 进行迭代时。

Answer 2

ConcurrentModificationExceptions 在迭代 Collection 时发生，而不是使用 Iterator.remove() 方法。

所以，执行的时候会抛出：

user.getSkillsList().add(skillObj);

来自 Java 教程，The Collection interface：

请注意，Iterator.remove 是在迭代期间修改集合的唯一安全方法；如果在迭代过程中以任何其他方式修改了底层集合，则行为未指定。