JavaRush/Помощь по Java задачам/Помогите кто чем может..

16 уровень

01.12.2018
747просмотров
9комментариев

Помогите кто чем может..

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

Решен

Почему ошибка вместо вывода?

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

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

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

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

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

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

Требования:

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

package com.javarush.task.task10.task1019; import java.io.*; 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<String, Integer> map = new HashMap<>(); while (true) { Integer id = Integer.parseInt(reader.readLine()); String name = reader.readLine(); map.put(name, id); if (name.isEmpty() && id.equals(" ")) break; } for(Map.Entry<String, Integer> pair : map.entrySet()) { System.out.println("Id=" + pair.getValue() + " Name=" + pair.getKey()); } } }

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

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

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

Andzha Saksykov

Уровень 20

1 декабря 2018, 20:52

Обратите внимание на условия : 1. Пустая строка - конец ввода данных. Числа могут повторяться. Строки всегда уникальны. Только пустая строка заканчивает ввод данных! 2. Программа должна выводить на экран содержимое HashMap согласно условию. Ключ и значение разделены пробелом. Каждое значение с новой строки. Ничего более этого выводить на печать нет необходимости (смотри пример вывода)!

Александр

Уровень 16

1 декабря 2018, 21:07

Исправил.

while (true) {
             Integer id = Integer.parseInt(reader.readLine());
             String name = reader.readLine();
             map.put(name, id);
             if (name.isEmpty()) break;
         }
         for(Map.Entry<String, Integer> pair : map.entrySet()) {
            System.out.println(pair.getValue() + " " + pair.getKey());
         }

У меня вообще ничего кроме ошибок не выводит..

Andzha Saksykov

Уровень 20

1 декабря 2018, 21:13

У вас в цикле первым вводится значение Value, а уже позже ключ Key. Думаю есть необходимость поменять очередность ввода.

Andzha Saksykov

Уровень 20

1 декабря 2018, 21:17

И проверку на пустую строку лучше сделать сразу после её ввода.

hidden #598481

Уровень 27

2 декабря 2018, 10:55

Тебе не совсем верно подсказывают. По условию задачи ты должен действительно считывать сначала значение, потом ключ, но в исправленном варианте у тебя нет проверки на пустоту считанного id. Поэтому раздели строчку с id на несколько: 1. считай строку 2. если пустая - выходи 3. в противном случае парси число. что касается проверки на пустоту name - ты все сделал верно. если name пустое, ты кладешь в мапу считанное id с пустым ключом name и выходишь из цикла.

Александр

Уровень 16

2 декабря 2018, 13:25

Как я понял пустое ID должно заносится в мапу, а потому в проверке не нуждается. Но тем не менее если, не ввести ID, то вместо вывода получаем кучу ошибок. С другой стороны, ничего не меняется, если сделать так:

while (true) {
             Integer id = Integer.parseInt(reader.readLine());
             if (id.equals(" ")) break;
             String name = reader.readLine();
             map.put(name, id);
             if (name.isEmpty()) break;
         }
for(Map.Entry<String, Integer> pair : map.entrySet()) {
            System.out.println(pair.getKey() + " " + pair.getValue());
         }

Александр

Уровень 16

2 декабря 2018, 13:57

гм.. а я так понял что строки у нас ключи, так как не повторяются, а числа значения. В примере вывода, выводятся сначала значения...

hidden #598481

Уровень 27

2 декабря 2018, 21:11

решение

да, в выводе идет сначала значение, потом ключ. но

Integer id = Integer.parseInt(reader.readLine());
if (id.equals(" ")) break;

это неправильно. у тебя id никогда не будет равно пробелу, т.к. id это целое число. кстати, тебе необязательно делать его типом Integer, можно использовать примитив int. но прежде чем парсить - нужно проверить, не пустая ли там строка: сначала считать: String s = reader.readLine(); потом проверить s.isEmpty() и выйти если true потом парсить id = Integer.parseInt(s);

Александр

Уровень 16

5 декабря 2018, 12:57

Спасибо большое!