JavaRush/Помощь по Java задачам/Валидатор не пропускает решение

40 уровень

19.06.2022
421просмотров
10комментариев

Валидатор не пропускает решение

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

Решен

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

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

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

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

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 scanner = new Scanner(System.in); while (scanner.hasNextInt() && scanner.hasNextInt()) { int a = scanner.nextInt(); int b = scanner.nextInt(); int firstMin = a <= b ? a : b; int secondMin = a <= b ? b : a; while (scanner.hasNextInt()) { int c = scanner.nextInt(); if (c <= firstMin) { secondMin = firstMin; firstMin = c; } else if (c < secondMin) { secondMin = c; } } System.out.println(secondMin); } scanner.close(); } }

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

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

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

Pavel Martynov

Уровень 6

19 июня 2022, 10:38

Судя из написанного я вижу, что если будет введено 4 5 е 6 7 е , каждый раз "е" будет пропущено, тк внутренний цикл будет ложным и мы вернемся к внешнему циклу. Кстати, подход интересный:

while (scanner.hasNextInt() && scanner.hasNextInt())

Лично я вывел 2 значения перед циклом и сравнил между собой. Потом перешел в цикл, где перебирал каждое следующее введенное значение и сравнивал его с минимальным. И как и у вас указано, присваивал мин2=мин1, когда мин1=минНовое. Зачем закрывать сканер? scanner.close(); , когда у нас по условию Считывать целые числа с клавиатуры до тех пор, пока не будет введено что-то другое: например, строка или символ.. Понял, наверное потому что у вас сначала во внешнем цикле забирается первые 2 числа, потом идет переход во внутренний цикл, где вы перебираете каждое введенное число, и когда вы натыкаетесь на "ВЫХОД", вас перебрасывает на внешний цикл, а новых вводных нет, и цикл бесконечный?

System.out.println(secondMin);

System.out.println у Вас внутри цикла, т.е. вам будут выводить каждое новое значение мин2. Вынесите за пределы внешнего цикла.

Aman Efer

Уровень 40

19 июня 2022, 13:15

Привет! Да, спасибо. Я чет проглядел этот момент (4 5 е). Хотел уже плюнуть и переписать код полностью, но тут включилось "дело принципа")) Изначально неправильно понял условие про вывод второго минимального числа. В итоге изменил условия в "else-if" во внутреннем цикле, и валидатор пропустил. Хотя очень странно как это решение прошло с учетом возврата на внешний цикл при внутреннем "false". Про "scanner.close()"я прочитал в лекции про класс Scanner, что его нужно всегда закрывать, чтобы не потреблять ресурсы компьютера. Поэтому я автоматом его всегда закрываю.

Aman Efer

Уровень 40

19 июня 2022, 13:24

В общем, вот это решение прошло

import java.util.Scanner;

/*
Второе минимальное число из введенных
*/

public class Solution {
    public static void main(String[] args) {
        //напишите тут ваш код
        Scanner scanner = new Scanner(System.in);

        while (scanner.hasNextInt() && scanner.hasNextInt()) {
            int a = scanner.nextInt();
            int b = scanner.nextInt();
            int firstMin = a <= b ? a : b;
            int secondMin = a <= b ? b : a;

            while (scanner.hasNextInt()) {
                int c = scanner.nextInt();
                if (c < firstMin) {
                    secondMin = firstMin;
                    firstMin = c;
                } else if (c < secondMin) {
                    if (c == firstMin)
                    continue;
                    secondMin = c;
                } else if ((firstMin == secondMin) && (c > secondMin)) {
                    secondMin = c;
                }
            }
            System.out.println(secondMin);
        }
        scanner.close();
    }
}

Pavel Martynov

Уровень 6

19 июня 2022, 13:55

Можно и короче сделать)

if (c < firstMin) {
                    secondMin = firstMin;
                    firstMin = c; }
else if (c> firstMin && c<secondMin) {
secondMin = c;
}

Профит) потому что идёт перекрытие других условий, которые нам не интересны и они пропускаются автоматически, нам даже не нужно использовать continue; ;)

Aman Efer

Уровень 40

19 июня 2022, 14:25

У меня так изначально было прописано. Но из-за того, что я сразу неправильно понял условия задачи, начал экспериментировать с условиям, и меня унесло в дебри))

Aman Efer

Уровень 40

19 июня 2022, 15:51

Посмотрел щас правильное решение. Оказывается переменные ввода данных с клавиатуры можно было объявить перед циклом, а я руководствовался требованием к задаче "Считывать данные с клавиатуры необходимо в цикле while", поэтому и запихнул объявление переменных ввода внутрь цикла. А можно было решить в разы быстрее и проще.

Александр

Уровень 40

19 июня 2022, 05:35

полезный

попробуй последовательность 4 4 5 5 7 7 8 8

Aman Efer

Уровень 40

19 июня 2022, 12:47

Спасибо большое, разобрался. Неправильно понял условие про вывод второго минимального.

Eduard Agafonov

Уровень 9

20 июня 2022, 11:31

Простите, можете объяснить что это значит, совсем не понимаю.

Aman Efer

Уровень 40

20 июня 2022, 17:41

Если вводить подряд два наименьших одинаковых числа, а затем число больше, то на печать будет выведено одно из двух минимальных. Т.е. вводя 4 4 5 5.... в консоль выведет 4. А по условию задачи должно быть выведено второе наименьшее число (в нашем случае 5).