JavaRush/Помощь по Java задачам/Где ошибка?

10 уровень

09.08.2021
458просмотров
10комментариев

Где ошибка?

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

Решен

Решение вроде как правильное, менял данные ввода, задача выполняет свою функцию корректно, но валидатор ругается на выполнение третьего условия.

В этой задаче нужно:

Считывать целые числа с клавиатуры до тех пор, пока не будет введено что-то другое: например, строка или символ.
Должно быть введено минимум два числа. Если введено меньше двух целых чисел, то такую ситуацию обрабатывать не нужно, а программа может быть завершена с ошибкой.
Вывести на экран второе по величине целое число после минимального из введенных с клавиатуры. Если таких чисел несколько, то необходимо вывести любое из них.
Минимальных чисел тоже может быть несколько.

Пример ввода:

8

4 // минимальное число

7

4 // минимальное число

5 // второе по величине число (второе минимальное)

9

5 // второе по величине число (второе минимальное)

exit

Пример вывода:
5

Требования:

Программа должна считывать числа c клавиатуры.
Программа должна выводить число на экран.
Программа должна выводить на экран второе минимальное из введенных целых чисел.
Если введено несколько вторых минимальных чисел, необходимо вывести любое.
Считывать данные с клавиатуры необходимо в цикле while.

package com.javarush.task.pro.task04.task0410; import java.util.Scanner; /* Второе минимальное число из введенных */ public class Solution { public static void main(String[] args) { Scanner term = new Scanner(System.in); int buf, min1 = term.nextInt(), min2 = term.nextInt(); if ((min1<min2)==false) //присваиваем min1 мин. значение, если это не так { buf = min1; min1 = min2; min2 = buf; } while (term.hasNextInt()) { buf = term.nextInt(); if ((buf>min1) && (buf<min2))//находим второе минимальное по условию min2 = buf; } System.out.println(min2); } }

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

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

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

Гофф

Уровень 20

Master

9 августа 2021, 09:45

У тебя в цикле не меняется min1. А, по идее, ведь должен, нет? Тест: 6, 5, 4, 3, 2, 1. Ну и классическое if ((min1<min2)==false). Не ошибка, но улыбку вызывает. min1<min2 - это уже либо true либо false. Поэтому if ( ! min1 < min2)

Sumnise

Уровень 10

9 августа 2021, 09:58

С циклом согласен, упустил возможность ввода в цикле переменной buf меньше, чем min1. У if если условие принимает значение false, то он не выполняется? update. Поправил цикл, теперь тест на уменьшение чисел нормально проходит) С if тоже подправил, и действительно всё работает как надо. Но проблема по третьему требованию осталась.

Гофф

Уровень 20

Master

9 августа 2021, 11:23

А запили актуальный код, поглядим

Sumnise

Уровень 10

9 августа 2021, 11:32

public class Solution {
    public static void main(String[] args) {
        Scanner term = new Scanner(System.in);
        int buf, min1, min2;
        min1 = term.nextInt();
        min2 = term.nextInt();

        if (min2<min1)
        {
            buf = min2;
            min2 = min1;
            min1 = buf;
        }

        while (term.hasNextInt())
        {
            buf = term.nextInt();
            if (buf<min1)
            {
                min2 = min1;
                min1 = buf;
                System.out.println("if");
            }
            else if ((buf<=min2) && (buf>min1))
            {
                min2 = buf;
                System.out.println("else");
            }
            else
            {
                System.out.println("cont");
                continue;
            }

            System.out.print(min1 + " ");
            System.out.println(min2);
        }

        System.out.println(min2 + " end");

    }
}

Sumnise

Уровень 10

9 августа 2021, 11:32

Расставил маркеры чтобы видеть какие участки отрабатывают. В какой то момент валидатор стал выдавать ошибку с текстом "проверьте правильность последовательности 2 2 3", у меня как и положено выходила 3 вот в этом коде. Вот какую последовательность даю на ввод и что получаю на выходе: 8 4 7 4 5 9 5 2 2 3 exit else //отрабатывает 7 с записью в min2 4 7 cont //пропускает 4 else //отрабатывает 5 с записью в min2 4 5 cont //пропускает 9 else //повторно отрабатывает 5 что не критично 4 5 if //запись 2 в min1 2 4 cont //пропуск 2 else //запись 3 в min2 2 3 3 end Если честно не понимаю, что не нравится валидатору.

Гофф

Уровень 20

Master

9 августа 2021, 11:52

решение

Не забыл перед валидатором свой отладочный код убрать-закоментить? Включая и строки 29-33? Так в принципе код совершенно адекватный и логично выстроенный, должен работать. Единственное, что приходит в голову - в условии не сказано, что должно происходить, если второго минимального в принципе нет. Например, ты вводишь пять одинаковых чисел. Или если первые два одинаковые, например 1, 1, 2 должен вернуть 2. Попробуй эту ситуацию прочекать. Ифы вроде ок, я бы заменил в строке 24 else if ((buf<=min2) && (buf>min1)) на строгое else if (buf < min2 && buf > min1), но это не принципиально.

Sumnise

Уровень 10

9 августа 2021, 12:02

Чекнул последовательность из 5 одинаковых чисел, всё в порядке. При вводе 3 3 3 3 3, выдаёт только 3. Попробовал как и в условии дать на ввод только одно число, в поле вывода ошибка Scanner'a. А вот при первых двух одинаковых что-то идёт не так. Собственно на это наверно валидатор и ругается, вот текст: "Убедись, что при вводе цифр 2 2 3, выводится второе минимальное." Я давал эти пресловутые 2 2 3 в конце после всех остальных. Тогда насколько я понимаю надо править условие else if в 24 строке, т.к. там (buf<min2) возвращает false. Благодарю за наводку 😃

Гофф

Уровень 20

Master

9 августа 2021, 12:25

Ну при всех одинаковых второго минимального как бы нет, есть только одно, оно же первое. Поэтому надо ли что-то в этом случае выводить - непонятно. В условии не оговаривается никак. Попробуй только 2, 2, 3. Ну или любу. комбинацию, где первые два одинаковые. Если первые два одинаковые, то твой код не будет правильно работать, потому что оба ифа не сработают.

Гофф

Уровень 20

Master

9 августа 2021, 12:29

полезный

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

Sumnise

Уровень 10

9 августа 2021, 12:52

Решил вот таким образом, жёстким костылём защиты от повтора первых двух чисел, на мой взгляд криво, но что поделать, собственные идеи на этой иссякли. Кстати предложенный вариант с бесконечным вводом и моментальным сравнением был бы короче этой портянки и выполнял функцию даже лучше как мне кажется. Жаль не заглянул до того как отправил на проверку, затестил бы.

public class Solution {
    public static void main(String[] args) {
        Scanner term = new Scanner(System.in);
        int buf, min1, min2;
        min1 = term.nextInt();
        min2 = term.nextInt();

        if (min2<min1)
        {
            buf = min2;
            min2 = min1;
            min1 = buf;
        }

        while (term.hasNextInt())
        {
            buf = term.nextInt();
            if (buf<min1)
            {
                min2 = min1;
                min1 = buf;
            }
            else if (buf<min2 && buf>min1)
                min2 = buf;
            else if (buf>min2 && min1==min2)
                min2 = buf;
        }
        System.out.println(min2);
    }
}