Помогите разобраться. закипаю уже

Вопрос по задаче

Архивный

package com.javarush.task.task08.task0817;

import java.util.HashMap;
import java.util.Map;


public class Solution {
    public static HashMap<String, String> createMap() {
        HashMap<String, String> map = new HashMap<>();
        map.put("Pupkin", "Pup");
        map.put("Vasilkov", "Alex");
        map.put("Grimm", "Brat");
        map.put("Konyshev", "Gosha");
        map.put("Petrov", "Izya");
        map.put("Zmeeva", "Olga");
        map.put("Rukin", "Alex");
        map.put("Dolina", "Larisa");
        map.put("Vasserman", "Anatoly");
        map.put("Rivz", "Kianu");
        return map;


    }

    public static void removeTheFirstNameDuplicates(Map<String, String> map) {
        int count = 0;
        for (Map.Entry<String, String> orig : map.entrySet()) {
            for (Map.Entry<String, String> camp : map.entrySet()) {
                if (orig.getValue().equals(camp.getValue())) {
                    count++;
                }
            }
            if (count > 1){
                Solution.removeItemFromMapByValue(map, orig.getValue());
            }
            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) {

        Solution.removeTheFirstNameDuplicates(createMap());
    }
}

Создать словарь (Map<String, String>) занести в него десять записей по принципу "фамилия" - "имя".
Удалить людей, имеющих одинаковые имена.

Требования:

Программа не должна выводить текст на экран.
Программа не должна считывать значения с клавиатуры.
Метод createMap() должен создавать и возвращать словарь Map с типом элементов String, String состоящих из 10 записей.
Метод removeTheFirstNameDuplicates() должен удалять из словаря всех людей, имеющие одинаковые имена.
Метод removeTheFirstNameDuplicates() должен вызывать метод removeItemFromMapByValue().

package com.javarush.task.task08.task0817; import java.util.HashMap; import java.util.Map; public class Solution { public static HashMap<String, String> createMap() { HashMap<String, String> map = new HashMap<>(); map.put("Pupkin", "Pup"); map.put("Vasilkov", "Alex"); map.put("Grimm", "Brat"); map.put("Konyshev", "Gosha"); map.put("Petrov", "Izya"); map.put("Zmeeva", "Olga"); map.put("Rukin", "Alex"); map.put("Dolina", "Larisa"); map.put("Vasserman", "Anatoly"); map.put("Rivz", "Kianu"); return map; } public static void removeTheFirstNameDuplicates(Map<String, String> map) { int count = 0; for (Map.Entry<String, String> orig : map.entrySet()) { for (Map.Entry<String, String> camp : map.entrySet()) { if (orig.getValue().equals(camp.getValue())) { count++; } } if (count > 1){ Solution.removeItemFromMapByValue(map, orig.getValue()); } 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) { Solution.removeTheFirstNameDuplicates(createMap()); } }

Комментарии (8)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

hidden #2054204

Уровень 14

13 апреля 2019, 11:49

полезный

Давайте еще я попробую другим путем донести мысль. Удаление элементов множества нужно вынести за пределы цикла, который это множество перебирает. Пришла в голову занятная (может и неудачная, но попробую) аналогия, даже не поленюсь набрать "многа букаф". :) Задача: Разрешено съесть самую маленькую конфету из всего пакета (из множества конфет). Вы прикинули, что, поскольку количество "подходов" не ограничено, то можно "утилизировать" весь пакет. Быстренько написали программу, включающую в себя цикл:

Цикл{
перебираем весь пакет  и находим самую маленькую;
Съедаем;
Хочу еще!
}

Программа ругается: Эй, чувак, я не смогу перебрать "весь" пакет с конфетами, когда ты их методично поджираешь! Так не пойдет! Программа не может перебрать в цикле весь пакет, поскольку на каждой петле цикла будет меняться размер пакета. К примеру, было 20 конфет, цикл пройдется и их станет 19, а программа, на новом проходе цикла, хочет сравнить весь пакет, т.е. все 20!!! Как программу успокоить и сожрать таки все конфеты? Нужно сделать так, что бы она выбирала для нас конфету, а мы ей при этом не мешали, выдергивая конфеты у нее из рук при переборе пакета:

Цикл1{
А какая конфета тут самая маленькая?
Цикл2{
Перебираем весь пакет и находим необходимую; // программа спокойно перебирает весь пакет
}
Получаем конфету и съедаем; //   размер пакета изменился, но во время следующего прохода цикла программа все равно сможет перебрать его весь
Хочу еще!
}

как-то так. :)

Уровень 40

13 апреля 2019, 11:59

Очень доходчивое объяснение. Благодарю!

Самуил Олегович

Уровень 41

3 мая 2019, 19:25

Весело и доходчиво, пойду переберу конфетки)

Andrei Lead Software Developer (

12 апреля 2019, 19:12

полезный

removeItemFromMapByValue(Map<String, String> map, String value)

можно сократить до одной строчки, если изучить метод values() у Map интерфейса и методы remove(Object o) и removeAll(Collection<?> c) у интерфейса Collection.

removeTheFirstNameDuplicates(Map<String, String> map)

сначала (в рамках одного цикла) собираем дубликаты (и тут нам так же может помочь метод values() ), затем, в рамках другого цикла, вызываем удаление для каждого из дубликатов.

Уровень 40

13 апреля 2019, 11:36

спасибо

--------

Уровень 40

Master

12 апреля 2019, 17:48

Выводить пробовал, что в результате остается?

Уровень 40

12 апреля 2019, 17:55

--------

Уровень 40

Master

12 апреля 2019, 18:03

У тебя итератор словаря задействован в removeTheFirstNameDuplicates, поэтому от копии в removeItemFromMapByValue толку немного - в первом методе тоже нужно как-то освободить итератор (тоже создавать копию или собирать куда-то удаляемые значения, после перебора уже удалять)