how to avoid concurrentModificationException when removing an object from to ArrayLists"

Question

Consider the following code.

As you might expect, the deleteFruitByName method throws a ConcurrentModificationException when a fruit is removed inside a for-each loop.

How can i avoid that in such cases?

import java.util.ArrayList;

public class Stringplay {
    public static void main(String[] args) {
        ArrayList<Fruit> fruites = new ArrayList<Fruit>();
        new Fruit(32, "apple", "red");
        new Fruit(64, "orange", "orange");
        new Fruit(12, "banana", "red");
        new Fruit(42, "grape", "purple");
        fruites.addAll(Fruit.fruits);
        Fruit.deleteFruitByName("apple");
        for (Fruit fruit : fruites) {
            System.out.println(fruit.getName());
        }
    }

}

public class Fruit {
    public int weight;
    public String name;
    public String type;
    public static ArrayList<Fruit> fruits = new ArrayList<Fruit>();

    public Fruit(int weight, String name, String type) {
        this.weight = weight;
        this.name = name;
        this.type = type;
        fruits.add(this);
    }

    public String getName() {
        return name;
    }

    public static void deleteFruitByName(String fruitName) {
        for (Fruit fruit : fruits) {
            if (fruit.getName().equals(fruitName)) {
                fruits.remove(fruit);
            }
        }

    }
}

Boris Chistov · Accepted Answer · 2020-03-22 13:41:54Z

1

To avoid ConcurrentModificationException you need to use Iterator here.

    public static void deleteFruitByName(String fruitName) {
        Iterator<Fruit> it = fruits.iterator();
        while (it.hasNext()) {
            Fruit fruit = it.next();
            if (fruit.getName().equals(fruitName)) {
                it.remove();
            }
        }
    }

From java doc

The iterators returned by this class's iterator and listIterator methods are fail-fast: if the list is structurally modified at any time after the iterator is created, in any way except through the iterator's own remove or add methods, the iterator will throw a ConcurrentModificationException. Thus, in the face of concurrent modification, the iterator fails quickly and cleanly, rather than risking arbitrary, non-deterministic behavior at an undetermined time in the future.

Update: to iterate collection in class Fluit use this code

public class Main {

    public static void main(String[] args) {
        new Fruit(32, "apple", "red");
        new Fruit(64, "orange", "orange");
        new Fruit(12, "banana", "red");
        new Fruit(42, "grape", "purple");
        Fruit.deleteFruitByName("apple");
        for (Fruit fruit : Fruit.fruits) {
            System.out.println(fruit.getName());
        }
    }
}

edited Mar 22, 2020 at 13:41

answered Mar 22, 2020 at 9:40

Boris Chistov

9501 gold badge9 silver badges16 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

negar.n Over a year ago

does doing this removes object from fruites too?

Boris Chistov Over a year ago

yes, it removes. But your code has a little mistake. With this Fruit.deleteFruitByName("apple"); you remove elements from static collection in class Fluit, but with for loop in main - you iterete other collection, that initialized in main.

David Thielen · Accepted Answer · 2020-03-22 20:55:03Z

0

The following should work (just writing this here so may be a compile error):

public static void deleteFruitByName(String fruitName) { fruits.removeIf(fruit => fruit.getName().equals(fruitName)); }

answered Mar 22, 2020 at 20:55

David Thielen

33.7k42 gold badges158 silver badges294 bronze badges

Collectives™ on Stack Overflow

how to avoid concurrentModificationException when removing an object from to ArrayLists"

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related