Не могу понять, почему не компилится

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

Решен

Помогите, пожалуйста, разобраться в причине

Программа вводит с клавиатуры пару (число и строку) и выводит их на экран.
Нужно сделать так, чтобы программа вводила с клавиатуры пары (число и строку) и сохраняла их в HashMap.

Ключевые требования:

Пустая строка - конец ввода данных.
Числа могут повторяться.
Строки всегда уникальны.
Введенные данные не должны потеряться!

Затем программа должна выводить содержание HashMap на экран.
Каждую пару - с новой строки.

Пример ввода:
1 Мама 2 Рама 1 Мыла

Пример вывода:
1 Мыла 2 Рама 1 Мама

Требования:

Программа должна считывать данные с клавиатуры.
Программа должна выводить текст на экран.
В методе main объяви переменную коллекции HashMap и сразу проинициализируй ee.
Программа должна помещать в HashMap пары считанные с клавиатуры.
Программа должна выводить на экран содержимое HashMap согласно условию. Ключ и значение разделены пробелом. Каждое значение с новой строки.

package com.javarush.task.task10.task1019; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.HashMap; import java.util.Map; /* Функциональности маловато! */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); HashMap<Integer,String> map=new HashMap<Integer,String>(); int id = Integer.parseInt(reader.readLine()); String name = reader.readLine(); while (!map.isEmpty()) { map.put(id, name); // map = reader.readLine(); } for (Map.Entry entry : HashMap.entrySet()) { System.out.println(entry.getKey() + " " + entry.getValue()); } } }

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

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

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

Jacob

Уровень 15

3 декабря 2021, 09:18

Хотел ответить, - но уже всё в принципе сказано. 1) Добавьте дополнительное условие выхода из цикла. 2) Цикл должен заканчиваться если следующая введённая строка пустая. 3) Вы не заполняете словарь согласно задаче.

Kirill

Уровень 18

2 декабря 2021, 23:23

полезный

while (!map.isEmpty())

1) Вы в мапу еще ничего не положили, а уже проверяете её на "не пустая". Т.е. в этот цикл вы никогда не зайдете. 2) Не компилируется, потому что ошибка в последнем for. У класса HashMap нет статического метода entrySet(). Сам метод там есть, но он не статический, значит вызывать его можно только у экземпляра класса HashMap , а не у самого класса. 3) И как написали ранее - "по условию числа могут повторяться. Значит, ключ в hashmap - строка." Но ввод, при этом осуществляется с цифр, а не со слов. В этом хитрость задания. 4) Подумайте, как сделать цикл бесконечным, а выходить из него только по условию ввода "пустой строки".

Ольга Бахута

Уровень 14

3 декабря 2021, 11:20

Вроде как исправила, но всё равно не проходит компиляцию, ругается на итератор.

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        HashMap<Integer,String> map=new HashMap<Integer,String>();
        int id = Integer.parseInt(reader.readLine());
        String name = reader.readLine();
        while (true) {
            map.put(id, name);
            if (reader.readLine()== null) break;
        }

        Iterator<Map.Entry<String, Integer>> iterator = map.entrySet().iterator();

        while (iterator.hasNext())
        {
            Map.Entry<String, Integer> pair = iterator.next();
            String key = pair.getKey();
            int value = pair.getValue();
            System.out.println(value + " " + key);
        }
    }
}

Ольга Бахута

Уровень 14

3 декабря 2021, 13:59

Окончательный вариант такой, поняла, что было не так выше, но вот вывод не проходит, всё еще ругается на итератор

HashMap<Integer,String> map=new HashMap<>();
        while (true){
        int id = Integer.parseInt(reader.readLine());
        String name = reader.readLine();
         for (HashMap.Entry<Integer, String> pair : map.entrySet()) {
            if(name.equals(pair.getValue())) {
                    name = reader.readLine();
                }

            if (name == null) break;}
            map.put(id, name);
        }

       Iterator<Map.Entry<String, Integer>> iterator = map.entrySet().iterator();

        while (iterator.hasNext())
        {
            Map.Entry<String, Integer> pair = iterator.next();
            String key = pair.getKey();
            int value = pair.getValue();
            System.out.println(value + " " + key);
        }
        }
}

Ольга Бахута

Уровень 14

3 декабря 2021, 19:11

решение

Надо без итератора, вместо налла пустая строка. И надо вставлять пустую строку, а не выходить из цикла

map.put("",id);

Kirill

Уровень 18

4 декабря 2021, 08:33

Дело было не только в итераторе, а еще и в том, что заполняли вы мапу с ключом Integer, а итератором шли по ней и ключом у вас была String. Итератором вы только выводили мапу в консоль и в нем не было ошибок. И по условию Пустая строка - конец ввода данных., вставлять в мап её как бы не нужно. Хотя все зависит от того как настроен валидатор. В любом случае, из цикла ввода данных вы выходите. Если решили - отлично. Главное, понять до конца все действия которые производите. Советую еще установить IDE на ПК и делать там с плагином от Javarush. И к IDE привыкните и многих мелких ошибок избежите.

Евгений

Уровень 28

2 декабря 2021, 21:21

Во-первых, по условию числа могут повторяться. Значит, ключ в hashmap - строка. Во-вторых, странный цикл - пока map не пуста добавлять пары.