JavaRush/Помощь по Java задачам/Компилятор ругается на код, но программа работает как над...

41 уровень

26.03.2021
504просмотров
26комментариев

Компилятор ругается на код, но программа работает как надо

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

Обсуждается

Есть вот такой код

public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        Map<Integer,String> map=new HashMap<Integer,String>();
        try{
        while(true){
            Integer id = Integer.parseInt(reader.readLine());
            String name = reader.readLine();
            map.put(id, name);
            if(name.equals(""))throw new Exception();
        }
        }
        catch(Exception e){
            for (Map.Entry<Integer, String> pair : map.entrySet()) {
            int key = pair.getKey();
            String name = pair.getValue();
            System.out.println("Id=" + key + " Name=" + name);
            }
    }
}

Работает код в общем то идеально, все четко и по делу. Но есть одно большое НО. Компилятор выдает ошибку по поводу незакрытого бесконечного цикла. Хотя все обернуто в блок трай кэтч и в любой ситуации кэтч перехватывает ошибку и выводить мапу с введенными данными. Я понимаю, что где то тут зарыт косяк, но не могу понять где именно. И почему компилятор НетБинс прекрасно выполняет код, а компилятор на сайте не хочет?)я так понимаю, что есть какая то ситуация, которая может выбить мне ошибку. Но какая именно не понимаю. Раз 10-15 прогнал программу - все работает как должно.

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

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

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

Ksenia Volkova Java Developer в DXC

Master

26 марта 2021, 08:04

Я тут вижу другую проблему - по условию задачи, числа могут повторяться. Если ты тестил на данных из примера, ты не мог не заметить разницу в результате. Ну и прерывать цикл через выброс исключения - не лучшая практика. Исключения на то и исключения, что не должны быть частью нормальной логики.

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 08:08

аа вот тут я перепутал, у меня ключ это цифра, а вэлью это текст. Ну да это не суть, поменял местами - все работает как надо, это мелочи. Я не понимаю где кроется незакрытая ошибка... и почему компилятор мне подсвечивает бесконечный цикл, хотя вроде условия выхода без ошибки прописаны....

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 08:04

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

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 08:07

И вывод будет не правильный System.out.println("Id=" + key + " Name=" + name); Пример вывода: 1 Мыла - тут нет никаких Id и Name 2 Рама

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 08:08

да все я перепечатал код

public static void main(String[] args)  {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        Map<String,Integer> map=new HashMap<String,Integer>();
        try{
        while(true){
            Integer id = Integer.parseInt(reader.readLine());
            String name = reader.readLine();
            map.put(name,id);
            if(name.equals(""))throw new Exception();
        }
        }
        catch(Exception e){
        for (Map.Entry<String, Integer> entry : map.entrySet()) {
            System.out.println(entry.getValue() + " " + entry.getKey());
            }
    }
}

но вопросы остались те же самые. Где ошибка то?) Более того, в правильном решение все ужасно. Там нет никакой обработки на ввод цифры - и если мы введем пробел или букву - у нас программа крашнется с ошибкой. Так разве можно людей учить писать?)

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 08:19

if(name.equals(""))throw new Exception(); уберите условие, пустая строка может быть ввод заканчивается на пустой строке в числе, при NFE

Ksenia Volkova Java Developer в DXC

Master

26 марта 2021, 08:20

Более того, в правильном решение все ужасно. Говорит человек, который прерывает цикл с помощью throw new Exception() 😂😂 Даже Идея по этому поводу предупреждение напишет. Про netbeans не знаю, может, он любит такие извращения...

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 08:28

Предупреждение пишет. Так мое решение хотя бы работает. Ошибка вылетает - блок кэтч ее ловит и все чудно) Но я как раз написал этот пост потому что хочу разобраться как мне сделать то же самое по смыслу, но красиво и правильно по форме) Я понимаю, что это кривой код. Хочу понять как переписать правильно)

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 08:30

if(name.isEmpty())throw new Exception();

сделал так но компилятору все равно не нравится бесконечный цикл :(

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 08:30

Читаете строку, проверяете ее isEmpty() если все ок парсите в число и читаете name, добавляете в map

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 08:32

Я же говорю, name может быть пустым id не должно быть пустым if(name.isEmpty())throw new Exception(); вам вообще не нужна эта строка

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 08:34

Так если нэйм может быть пустым, то как у меня цикл ввода закроется? А по поводу строк, вы предлагаете сделать 2 стринговых переменных и парсить в число после проверки?

hidden #598481

Уровень 27

26 марта 2021, 08:37

естественно, так и нужно поступить

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 08:40

Так если нэйм может быть пустым, то как у меня цикл ввода закроется? Учитывая ваш код то при попытке спарсить пустую строку получите NumberFormatException и выйдите из цикла

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 08:44

String idStr;
String name;
int id;
try{
while(true){
    idStr = reader.readLine();
    if(!(idStr.isEmpty())){
        id=Integer.parseInt(idStr);
    }
    else break;
    name = reader.readLine();
    map.put(name,id);

}
}

так вообще все перестало работать)

idStr = reader.readLine();
            if(!(idStr.isEmpty())){
                id=Integer.parseInt(idStr);
            }
            else throw new Exception();
            name = reader.readLine();
            map.put(name,id);

вот так работает, но ровно так же как и до всех изменений. Такое же предупреждение компилятор дает Мне не дает покоя красный восклицательный знак с его reached end of while while parsing Обычно эта штука указывает на бесконечный цикл. Значит есть где то ситуация где я не выхожу из него или выхожу с ошибкой. Вот че мне надо - найти эту ситуацию)

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 08:53

Полный код лучше выложите, а то додумывать приходится :)

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 08:58

public static void main(String[] args)  {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        Map<String,Integer> map=new HashMap<String,Integer>();
        try{
        while(true){
            Integer id = Integer.parseInt(reader.readLine());
            String name = reader.readLine();
            //if(name.isEmpty())throw new Exception();
            map.put(name,id);

        }
        }
        catch(Exception e){
        for (Map.Entry<String, Integer> entry : map.entrySet()) {
            System.out.println(entry.getValue() + " " + entry.getKey());
            }
    }
}

сейчас получается, что когда я ввожу в интеджер пустоту или символ или строку - вылетает ошибка, ее ловит кэч и печатает мапу. Когда я ввожу пустоту в стринг - все норм)Все работает как часики. Но где то зарыта собака)

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 09:08

Попробуйте заменить тип ссылки Map<String, Integer> HashMap<String, Integer>

Ksenia Volkova Java Developer в DXC

Master

26 марта 2021, 09:17

В задании сказано, что пустая строка - конец ввода. Думаю, без разницы, в какой именно позиции.

Ksenia Volkova Java Developer в DXC

Master

26 марта 2021, 09:20

Правильно по форме: - прерывать цикл либо с помощью самого условия цикла, либо с помощью break. - не запихивать вывод результата в catch, потому что это часть нрмальной логики программы, а не обработка исключительной ситуации.

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 09:27

Введенные данные не должны потеряться! Думаю это к ситуации если строка пустая, мы выйдем из цикла потеряв введенное число

KIRA Разработчик в prosv.ru

Expert

26 марта 2021, 09:28

В решении от JR цикл не завершается

while (true) {
      String number = reader.readLine();
      if (number.isEmpty()) {
         break;
      }
      int id = Integer.parseInt(number);
      String name = reader.readLine();
      map.put(name, id);
}

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 09:31

public static void main(String[] args) throws IOException  {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        Map<String,Integer> map=new HashMap<String,Integer>();

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

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

Ksenia Volkova Java Developer в DXC

Master

26 марта 2021, 09:33

try-catch можно было и оставить. Просто не засовывать в него вывод.

Ksenia Volkova Java Developer в DXC

Master

26 марта 2021, 09:35

Введенные данные не должны потеряться! Думаю это к ситуации если строка пустая, мы выйдем из цикла потеряв введенное число Можно добавить число с пустой строкой - и выйти из цикла. Тут нет противоречия.

Maks Panteleev Java Developer в Bell Integrator

26 марта 2021, 10:15

да я просто не понял почему в изначальном примере компилятор показывал ошибку