Core Java

How to Avoid ConcurrentModificationException when using an Iterator

Java Collection classes are fail-fast which means that if the Collection will be changed while some thread is traversing over it using iterator, the iterator.next() will throw a ConcurrentModificationException.

This situation can come in case of multithreaded as well as single threaded environment.

Lets explore this scenario with the following example:

import java.util.*;
 
public class IteratorExample {
 
    public static void main(String args[]){
        List<String> myList = new ArrayList<String>();
 
        myList.add("1");
        myList.add("2");
        myList.add("3");
        myList.add("4");
        myList.add("5");
 
        Iterator<String> it = myList.iterator();
        while(it.hasNext()){
            String value = it.next();
            System.out.println("List Value:"+value);
            if(value.equals("3")) myList.remove(value);
        }
 
        Map<String,String> myMap = new HashMap<String,String>();
        myMap.put("1", "1");
        myMap.put("2", "2");
        myMap.put("3", "3");
 
        Iterator<String> it1 = myMap.keySet().iterator();
        while(it1.hasNext()){
            String key = it1.next();
            System.out.println("Map Value:"+myMap.get(key));
            if(key.equals("2")){
                myMap.put("1","4");
                //myMap.put("4", "4");
            }
        }
 
    }
}

Output is:

List Value:1
List Value:2
List Value:3
Exception in thread "main" java.util.ConcurrentModificationException
    at java.util.AbstractList$Itr.checkForComodification(AbstractList.java:372)
    at java.util.AbstractList$Itr.next(AbstractList.java:343)
    at com.journaldev.java.IteratorExample.main(IteratorExample.java:27)

From the output stack trace, its clear that the exception is coming when we call iterator next() function. If you are wondering how Iterator checks for the modification, its implementation is present in AbstractList class where an int variable modCount is defined that provides the number of times list size has been changed. This value is used in every next() call to check for any modifications in a function checkForComodification().

Now comment the list part and run the program again.

Output will be:

Map Value:3
Map Value:2
Map Value:4

Since we are updating the existing key value in the myMap, its size has not been changed and we are not getting ConcurrentModificationException. Note that the output may differ in your system because HashMap keyset is not ordered like list. If you will uncomment the statement where I am adding a new key-value in the HashMap, it will cause ConcurrentModificationException.

To Avoid ConcurrentModificationException in multi-threaded environment:

1. You can convert the list to an array and then iterate on the array. This approach works well for small or medium size list but if the list is large then it will affect the performance a lot.

2. You can lock the list while iterating by putting it in a synchronized block. This approach is not recommended because it will cease the benefits of multithreading.

3. If you are using JDK1.5 or higher then you can use ConcurrentHashMap and CopyOnWriteArrayList classes. It is the recommended approach.

To Avoid ConcurrentModificationException in single-threaded environment:

You can use the iterator remove() function to remove the object from underlying collection object. But in this case you can remove the same object and not any other object from the list.

Let us run an example using Concurrent Collection classes:

package com.journaldev.java;
 
import java.util.Iterator;
import java.util.List;
import java.util.Map;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.CopyOnWriteArrayList;
 
public class ThreadSafeIteratorExample {
 
    public static void main(String[] args) {
 
        List<String> myList = new CopyOnWriteArrayList<String>();
 
        myList.add("1");
        myList.add("2");
        myList.add("3");
        myList.add("4");
        myList.add("5");
 
        Iterator<String> it = myList.iterator();
        while(it.hasNext()){
            String value = it.next();
            System.out.println("List Value:"+value);
            if(value.equals("3")){
                myList.remove("4");
                myList.add("6");
                myList.add("7");
            }
        }
        System.out.println("List Size:"+myList.size());
 
        Map<String,String> myMap = 
             new ConcurrentHashMap<String,String>();
        myMap.put("1", "1");
        myMap.put("2", "2");
        myMap.put("3", "3");
 
        Iterator<String> it1 = myMap.keySet().iterator();
        while(it1.hasNext()){
            String key = it1.next();
            System.out.println("Map Value:"+myMap.get(key));
            if(key.equals("1")){
                myMap.remove("3");
                myMap.put("4", "4");
                myMap.put("5", "5");
            }
        }
 
        System.out.println("Map Size:"+myMap.size());
    }
 
}

Output is:

List Value:1
List Value:2
List Value:3
List Value:4
List Value:5
List Size:6
Map Value:1
Map Value:null
Map Value:4
Map Value:2
Map Size:4

From the above example its clear that:

1. Concurrent Collection classes can be modified avoiding ConcurrentModificationException.

2. In case of CopyOnWriteArrayList, iterator doesn’t accomodate the changes in the list and works on the original list.

3. In case of ConcurrentHashMap, the behavior is not always the same.

For condition:

if(key.equals("1")){
    myMap.remove("3");

Output is:

Map Value:1
Map Value:null
Map Value:4
Map Value:2
Map Size:4

It is taking the new object added with key “4? but not the next added object with key “5?.

Now if I change the condition to

if(key.equals("3")){
    myMap.remove("2");

Output is:

Map Value:1
Map Value:3
Map Value:null
Map Size:4

In this case its not considering the new added objects.

So if you are using ConcurrentHashMap then avoid adding new objects as it can be processed depending on the keyset. Note that the same program can print different values in your system because HashMap keyset is not in any order.

Extra Toppings:

for(int i = 0; i<myList.size(); i++){
    System.out.println(myList.get(i));
    if(myList.get(i).equals("3")){
        myList.remove(i);
        i--;
        myList.add("6");
    }
}

If you are working on single-threaded environment and want your code to take care of the extra added objects in the list then you can do so using following code and avoiding iterator.

Note that I am decreasing the counter because I am removing the same object, if you have to remove the next or further far object then you don’t need to decrease the counter.

Try it yourself.

Reference : How to Avoid ConcurrentModificationException when using an Iterator from our JCG partner at JournalDev.

    Related Articles:

    Related Snippets :
    Subscribe
    Notify of
    guest

    This site uses Akismet to reduce spam. Learn how your comment data is processed.

    14 Comments
    Oldest
    Newest Most Voted
    Inline Feedbacks
    View all comments
    Rajesh
    Rajesh
    11 years ago

    Awesome solution. thanks much.. :)

    Zbonger Masangu
    Zbonger Masangu
    10 years ago

    I use to come across this issue every now and then until someone suggested that I loop backwards and modify the list It’s working but I think CopyOnWriteArrayList is the best. Thanks!!!!!

    Peter Jerald
    Peter Jerald
    10 years ago

    Good approach. Thanks a lot :)

    Jim Nolan
    10 years ago

    Pankaj,

    Thanks for the clear and concise solution. This solved my problem on the first search. Keep up the excellent work!

    Jim

    Shwetank
    Shwetank
    10 years ago

    why we can update a value in a hasMap but we can not update in a arraylist while looping it.

    1- arrylist.add(1,2); // updating a existing value in arraylist
    2- map.put(1,2); // updating a existing value in map

    Minnum Vairam
    Minnum Vairam
    10 years ago

    Thanks for the clearly articulating solution.

    Phil
    Phil
    10 years ago

    Popped right up on google and explained the problem, thank you so much!

    Binh Thanh Nguyen
    Binh Thanh Nguyen
    9 years ago

    Thanks, really nice.

    Harsh Maheshwari
    Harsh Maheshwari
    9 years ago

    If you use Iterator as follows, concurrent modification exception is not thrown.. May be you can update your post to include these conditions –

    Iterator<Entry> i = m.entrySet().iterator();

    while (i.hasNext()) {
    Entry entry = i.next();
    if (entry.getKey().equals(“key10”)) {
    i.remove(); // removes current entry from map
    m.remove(“key12”); // removes another entry
    // m.put(“key221”, “newnewval”); // this also works
    break;
    } else {
    System.out.println(entry);
    }
    }

    // Key10 and key12 are removed from the map. Size is reduced by 2

    Anwesh
    Anwesh
    8 years ago

    Thanks for Solution.

    bala
    8 years ago

    great article.good approach sir

    części zamienne do wózków widłowych

    Thank you for the good writeup. It actually was a leisure account it.

    Glance complex to more added agreeable from you!
    By the way, how can we keep in touch?

    producent palet
    5 years ago

    Thank you for this article. Thank you for this article. I create my website on my own and want to know as much as possible.

    Fauzi
    Fauzi
    5 years ago

    Hi, i already implement your method for removing value of ConcurrentHasMap but i still got error ConcurrentModificationException, the log is pointed to iterator.next();

    Iterator iterator = data.keySet().iterator();
    while (iterator.hasNext()) {
    String key = iterator.next();
    if (name.equals(key))
    data.remove(key);
    }

    Back to top button