Господа обьясните пожалуйста почему мой цикл в методе createMap() не срабатывает

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

Решен

Господа обьясните пожалуйста почему мой цикл в методе createMap() не срабатывает, при выводе на экран

"Kozlov" 291

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

Требования:

Программа не должна выводить текст на экран.
Программа не должна считывать значения с клавиатуры.
Метод createMap() должен создавать и возвращать словарь Map с типом элементов String, Integer состоящих из 10 записей по принципу «фамилия» - «зарплата».
Метод removeItemFromMap() должен удалять из словаря всех людей, у которых зарплата ниже 500.

package com.javarush.task.task08.task0818; import java.util.*; /* Только для богачей */ public class Solution { public static Map<String, Integer> createMap() { Map<String,Integer> map = new HashMap<>(); int a = (int) (Math.random()*(1000+1)) + 0; for (int i = 0; i < 10; i++){ map.put("Kozlov",a); } return map; } public static void removeItemFromMap(Map<String, Integer> map) { map.values().removeIf(integer -> integer < 500); } public static void main(String[] args) { } }

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

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

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

Павел Безумный учёный

Expert

13 апреля 2020, 17:48

решение

Цикл добавляет в отображение пару с различными значениями, но с одинаковыми ключами ("Kozlov"), поэтому фактически в отображение попадает только пара, создаваемая на первой итерации цикла, а все последующие пары игнорируются. Для разрешения этой проблемы можно, к примеру, создать пул фамилий, и сделать так, чтобы цикл на каждой итерации выбирал очередную фамилию из этого пула:

public static Map<String, Integer> createMap() {
    Map<String,Integer> map = new HashMap<>();
    int a = (int) (Math.random()*(1000+1)) + 0;

    String[] peoples = {"Kozlov", "Ivanov", "Petrov", "Ovsov"};

    for (int i = 0; i < 4; i++) {
        map.put(peoples[i], a);
    }

    return map;
}

P. S. В примере уменьшено количество итераций цикла, чтобы не расписывать массив peoples полностью.

Nikita

Уровень 9

13 апреля 2020, 17:59

ага понял спасибо большое!

Nikita

Уровень 9

13 апреля 2020, 18:21

а с Math.random я так понимаю такая же ситуация надо создавать такой же пул

int [] a = {(int) (Math.random()*(1000+1)) + 0,(int) (Math.random()*(1000+1)) + 0,(int) (Math.random()*(1000+1)) + 0,
                (int) (Math.random()*(1000+1)) + 0

так как при выводе дает такой результат

Kozlov 729
Ivanov 729
Petrov 729
Ovsov 729

может есть какая то более уменьшенная конструкция?

Павел Безумный учёный

Expert

13 апреля 2020, 18:59

Можно даже обойтись без дополнительной переменной a, генерируя очередное число "по месту" прямо при создании пары:

for (int i = 0; i < 4; i++) {
    map.put(peoples[i], (int) (Math.random() * 1000));
}

Nikita

Уровень 9

13 апреля 2020, 19:20

вот то что нужно спасибо !

Sevod

Уровень 26

14 апреля 2020, 07:21

полезный

Как вариант строчку 3 перенести внутрь цикла, между 7 и 8 строкой. Просто сейчас "int a" вне цикла, один раз заполняется и дальше не меняется. А вот внутри цикла, при каждой итерации, данные в "int a" будут обновляться. С Козловым можно к примеру так сделать, что бы он разным был (пишу без проверки, возможно где то синтаксическая ошибка) map.put("Kozlov" + i,a); И тогда будет Kozlov0, Kozlov1 и т.д. Рендом тоже можно убрать, что бы упростить. for (int i = 0; i < 10; i++){ map.put("Kozlov" + i, i + 495); } Вариантов решения огромное. Например можно map.put повторить 10 раз.

Nikita

Уровень 9

13 апреля 2020, 17:32

точнее я более меннее понимаю почему но как привязать условие цикла к счетчику?