На сколько я понимаю, в методе removeItemFromMapByValue() уже осуществляется одной переменной из Map и нужно в методе removeFirstNameDuplicates() организовать перебор всех значений, а удалять уже вызовом removeItemFromMapByValue()?
package en.javarush.task.jdk13.task08.task0817;
import java.util.HashMap;
import java.util.Map;
/*
We don't need repeats
*/
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 removeFirstNameDuplicates(Map<String, String> map) {
Map<String, String> copy = new HashMap<>(map);
for (String key : copy.keySet()) {
String value = copy.get(key);
removeItemFromMapByValue(map, value);
}
}
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) {
}
}