Ребят, подскажите пожалуйста в чем дело. Как я понял нельзя проходится по коллекции и удалять с неё одновременно, поэтому создал копию мапы и прохожусь по ней. Далее создаю стринг и присваиваю ей значение первой пары ключ-значение из мапы и сравниваю её со всеми элементами во внутреннем цикле. Для того, чтобы исключить полное удаление элементов( из-за того, что в каждом прохождении цикла элемент будет "встречать" себя же во внутреннем цикле) добавил переменную count и условие удаления элемента в случае, если count больше 1. В чем я промахнулся?
package com.javarush.task.task08.task0817;
import java.util.HashMap;
import java.util.Iterator;
import java.util.Map;
/*
Нам повторы не нужны
*/
public class Solution {
public static HashMap<String, String> createMap() {
//напишите тут ваш код
HashMap<String, String> map = new HashMap<>();
map.put("Попов", "Андрей");
map.put("Сидоров", "Алексей");
map.put("Николаев", "Антон");
map.put("Волков", "Андрей");
map.put("Касаткин", "Ярослав");
map.put("Володарский", "Антон");
map.put("Циолковский", "Глеб");
map.put("Ванюшин", "Михаил");
map.put("Терешкова", "Галина");
map.put("Андропов", "Михаил");
return map;
}
public static void removeTheFirstNameDuplicates(Map<String, String> map) {
//напишите тут ваш код
HashMap<String, String> map1 = new HashMap<>(map);
int count = 0;
for (Map.Entry<String, String> a : map1.entrySet()) {
String value = a.getValue();
for (Map.Entry<String, String> b : map1.entrySet()) {
if (b.getValue().equals(value)) {
count++;
}
if (count > 1) {
removeItemFromMapByValue(map, value);
count = 0;
}
}
}
}
public static void removeItemFromMapByValue(Map<String, String> map, String value) {
HashMap<String, String> copy = new HashMap<String, String>(map);
for (Map.Entry<String, String> pair : copy.entrySet()) {
if (pair.getValue().equals(value))
map.remove(pair.getKey());
}
}
public static void main(String[] args) {
HashMap<String, String> a = createMap();
removeTheFirstNameDuplicates(a);
System.out.println(a);
}
}