не могу разобраться, что не так. цикл в цикле - это понятно. а итератор в итераторе ломает голову. где я ошибся? и ещё вопрос, не понимаю почему в методе removeItemFromMapByValue (он был написан изначально), прогон идёт по копии мапы? 
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) {
        Iterator<Map.Entry<String, String>> iterator = map.entrySet().iterator();
        while (iterator.hasNext()) {
            Map.Entry<String, String> pair = iterator.next();
            String value = pair.getValue();

            Iterator<Map.Entry<String, String>> iterator2 = map.entrySet().iterator();
            while (iterator2.hasNext()) {
                Map.Entry<String, String> pair2 = iterator2.next();
                String value2 = pair2.getValue();

                if (value.equals(value2)) {
                    removeItemFromMapByValue(map, value);
                }
            }

        }

    }

    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) {

    }
}