Коллекции, ошибка Exception in thread "main" java.util.ConcurrentModificationException

@Shersh1k · Регистрация: 20.10.2016

Author24 — интернет-сервис помощи студентам

Есть задача удалить из хэшмэп элементы с интом меньше 500
непонимаю почему без хэшмэп "copy" не пашет?

Java

public class Solution {
    public static HashMap<String, Integer> createMap() {
        HashMap<String, Integer> vedom = new HashMap<>();
        vedom.put("adsad", 100);
        vedom.put("asdfdsf", 150);
        vedom.put("asd", 569);
        vedom.put("dsad", 124);
        vedom.put("ddfg", 1213);
        vedom.put("dfbr", 1012);
        vedom.put("brer", 600);
        vedom.put("ergbefrv", 400);
        vedom.put("dfvwfb", 500);
        vedom.put("dfgerwfge", 300);
        return vedom;
    }
 
    public static void removeItemFromMap(HashMap<String, Integer> map) {
        HashMap<String, Integer> copy = new HashMap(map);
        for (Map.Entry<String, Integer> pair : copy.entrySet()) {
            if (pair.getValue() < 500) {
                map.remove(pair.getKey());
            }
        }
        System.out.println(map);
    }
 
 
 
    public static void main(String[] args) {
        removeItemFromMap(createMap());
 
    }

если сделать

Java

public class Solution {
    public static HashMap<String, Integer> createMap() {
        HashMap<String, Integer> vedom = new HashMap<>();
        vedom.put("adsad", 100);
        vedom.put("asdfdsf", 150);
        vedom.put("asd", 569);
        vedom.put("dsad", 124);
        vedom.put("ddfg", 1213);
        vedom.put("dfbr", 1012);
        vedom.put("brer", 600);
        vedom.put("ergbefrv", 400);
        vedom.put("dfvwfb", 500);
        vedom.put("dfgerwfge", 300);
        return vedom;
    }
 
    public static void removeItemFromMap(HashMap<String, Integer> map) {
 
        for (Map.Entry<String, Integer> pair : map.entrySet()) {
            if (pair.getValue() < 500) {
                map.remove(pair.getKey());
            }
        }
        System.out.println(map);
    }
 
 
 
    public static void main(String[] args) {
        removeItemFromMap(createMap());
 
    }
}

то вот что показывает:
Exception in thread "main" java.util.ConcurrentModificationException
at java.util.HashMap$HashIterator.nextNode(HashMap.java:1437)
at java.util.HashMap$EntryIterator.next(HashMap.java:1471)
at java.util.HashMap$EntryIterator.next(HashMap.java:1469)
at com.javarush.task.task08.task0818.Solution.removeItemFromMap(Solution.java:28)
at com.javarush.task.task08.task0818.Solution.main(Solution.java:39)

Добавлено через 1 минуту
Я еще учусь, так что просьба не кидаться ничем)
Просто не пойму зачем мы передаем в метод наш список, после чего его копируем и с копией работаем? почему напрямую то нельзя, чего хотит эта дьявольщина)
Спасибо

Добавлено через 36 секунд
Это ведь по идее если работать с большим списком то скушает памяти дофига?

@lignium · 12.07.2017, 02:36

Для этого созданы итераторы, https://docs.oracle.com/javase... rator.html

Добавлено через 4 минуты
Map.entrySet().iterator()
Так как entrySet() возвращает "представление" словаря, можно смело вызывать Iterator.remove(), зная что пара удалится из словаря.

Добавлено через 1 минуту
А так получается что Вы пытаетесь удалить пару при том что со словарем работает (прогоняет) цикл for.

@Shersh1k · 12.07.2017, 08:40 **[ТС]**

lignium, проблема в том что с англ беда) поэтому покамись на русские источники опираюсь то там то сям. И вот где-то вычитал что в цикле foreach как-то там создается неявный итератор, и я думал что опираясь на него должно все получится. В общем немного непонятно. Ладно на данный момент пусть будет так, надеюсь в будущем это прояснится больше. (как много раз я уже себе так сказал, за это недолгое время что посвящаю изучению

)

@lignium 4 / 4 / 3 Регистрация: 17.11.2015 Сообщений: 57
	12.07.2017, 02:36	2
	Для этого созданы итераторы, https://docs.oracle.com/javase... rator.html Добавлено через 4 минуты Map.entrySet().iterator() Так как entrySet() возвращает "представление" словаря, можно смело вызывать Iterator.remove(), зная что пара удалится из словаря. Добавлено через 1 минуту А так получается что Вы пытаетесь удалить пару при том что со словарем работает (прогоняет) цикл for. 0

@Shersh1k 1 / 1 / 0 Регистрация: 20.10.2016 Сообщений: 12 Записей в блоге: 3
	12.07.2017, 08:40 [ТС]	3
	lignium, проблема в том что с англ беда) поэтому покамись на русские источники опираюсь то там то сям. И вот где-то вычитал что в цикле foreach как-то там создается неявный итератор, и я думал что опираясь на него должно все получится. В общем немного непонятно. Ладно на данный момент пусть будет так, надеюсь в будущем это прояснится больше. (как много раз я уже себе так сказал, за это недолгое время что посвящаю изучению ) 0