Всем привет, подскажите, пожалуйста, почему не работает такое решений с вложенным циклом.
Кажется, всё логично:
- создаю лист;
- запихиваю в него все значения (имена) из мапы;
- в цикле беру первый элемент листа и с помощью вложенного цикла сравниваю его со всеми остальными элементами листа;
- если есть совпадения - вызываю метод removeItemFromMapByValue и передаю в него мапу и повторяющийся элемент;
package com.javarush.task.task08.task0817;
import java.util.HashMap;
import java.util.Map;
import java.util.*;
/*
Нам повторы не нужны
*/
public class Solution {
public static Map<String, String> createMap() {
Map<String, String> map = new HashMap<String, String>();
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) {
//Iterator<Map.Entry<String, String>> iter = map.entrySet().iterator();
ArrayList<String> list = new ArrayList<String>();
list.addAll(map.values());
String first = "";
int i = 0;
while ( i < list.size()) {
first = list.get(i);
for(int j = 0; j < list.size() - 1; j++){
if ( first.equals(list.get(j + 1)) )
removeItemFromMapByValue(map, first);}
i++;
}
/*while(iter.hasNext()) {
Map.Entry<String, String> pair = iter.next();
removeItemFromMapByValue(map, pair.getValue());
}
*/
}
public static void removeItemFromMapByValue(Map<String, String> map, String value) {
Map<String, String> copy = new HashMap<>(map);
for (Map.Entry<String, String> pair : copy.entrySet()) {
if (pair.getValue().equals(value)) {
map.remove(pair.getKey());
}
}
}
public static void main(String[] args) {
}
}