Что с моим решением не так?

Готовые решения не принято постить, только если в личку)) И коллекции на этом уровне вроде как ещё не проходили.

понял, принял. Абсалютли согласен

Спасибо за помощь! И за совет на счет Integer.MAX_VALUE 😄✌👍

с твоих слов это вышлядит вот так public class Solution { public static void main(String[] args) { //напишите тут ваш код Scanner input = new Scanner(System.in); int min = input.nextInt(); int middle = min; do { int number = input.nextInt(); if (number < min) { middle = min; min = number; } } while (input.hasNextInt()); System.out.println(middle); } } только вот так не будет работать комбинации для проверки: 1. "1,2,3,4,5" 2. "5,4,3,2,1" 3. "5,1,2,3,4"

Чиорт, не будет, ты прав. Щяс.

Вот эту часть надо канеш по.другому

int min = input.nextInt();
int middle = min;

ду-вайл заменить на просто вайл и добавить в цикл ещё одну проверку else if (number < middle) middle = number;

хех, так тоже не работает)) Я вчера пол ночи просидел так и не решил как я только не крутил чтоб эта комбинация работала. 1. "1,2,3,4,5" 2. "5,4,3,2,1" 3. "5,1,2,3,4" потом плюнул написал через колекции.

Я просто в условие if добавил else if (number < min 2 && number != min) , и если это условие проходило то min2 = number 😄

Я в начале считывал только одно число и присваивал двум переменным. public class Solution { public static void main(String[] args) { //напишите тут ваш код Scanner s = new Scanner (System.in); int number = s.nextInt (); int min = number; int min2 = number; while (s.hasNextInt()) { int nbr = s.nextInt (); if (nbr < min) { min2 = min; min = nbr; } else if (nbr < min2 && nbr != min) min2 = nbr; }; System.out.println(min2); } }

Попробуй: 1 5 2 3 4

Здесь придётся, если не привлекать коллекции, два первых числа как то привести к порядку. Одним первым введённым числом, как Алексей Войнов очень правильно заметил, тут не обойтись, надо сначала оба первые два узнать и с ними разобраться. Поэтому оба два считываем, смотрим, кто из них поменьше, и потом начинаем с этими изначальными данными работать в цикле.

Намудрили... У меня проще получилось.

public static void main(String[] args) {
    Scanner scan = new Scanner (System.in);
    int min = Integer.MAX_VALUE;
    int min2 = 0;

    while (scan.hasNextInt()){
        int x = scan.nextInt();
        if (x<min){
            min2=min;
            min = x;

        }else if (x>min&&x<min2){
            min2=x;
        }
    }

Честь вам и хвала, что проще! Одно замечание: int min = Integer.MAX_VALUE; Использовать константы, зашитые в язык можно, но это просто не красиво. Для решения конкретной задачи ещё ок, но в общем - нет. Смысл ведь в чём? - проинициализировать эту переменную значением, которое точно не меньше, чем минимум. Хм, какое же значение не меньше минимума? Integer.MAX_VALUE? В джаве данной версии для типа int - да. Но может есть что-нибудь поближе и без этих условностей? Да есть! Вот первый элемент! Всегда, не зависимо ни от чего, ни от джавы, ни от типа, первый (на самом деле любой, не только первый) не меньше минимума. Или равен минимуму, или больше. И ты при этом не выходишь за рамки задачи и предоставленных тебе данных. Что бы найти минимальное значение среди элементов, надо их сравнить между собой, а не с посторонним, не относящимся к этим элементам, числом, с какой-то выдуманной, не существующей в природе константой MAX_VALUE. Привязывая решение к константе, которая существует в конкретном языке, конкретной реализации, в конкретной версии и для конкретного типа данных, ты ставишь себя в зависимость от того, что такая константа для такого типа данных есть. Но такое ведь не всегда бывает. Например, у BigInteger (с этим типом ты скоро столкнёшься, когда будешь факториалы вычислять) нет такой константы - и что ты будешь делать без BigInteger.MAX_VALUE? Хотелось здесь не просто сдать валидатору, а придумать чистый алгоритм, без костылей-констант, который легко бы подходил для реализации на любой платформе с любыми данными.

Ну и просто проследи, что у тебя произойдёт с переменными после первого прохода цикла: min = первый элемент min2 = Integer.MAX_VALUE Это без вариантов - именно так и будет. Но если мы знаем, что так произойдёт, зачем это вычислять? Может их лучше сразу этими значениями проинициализировать? Давай не будем торопится и посмотрим на второй проход цикла. После второго прохода два варианта: min = первый элемент min2 = второй элемент или min = второй элемент min2 = первый элемент. Хм... После второго прохода цикла у нас наши переменные min и min2 полюбак равны первым двум элементам... И дальше твой цикл именно с ними и работает. То-есть, получается, что константа Integer.MAX_VALUE нам для решения задачи и не нужна, можно обойтись просто первыми двумя отсканированными элементами. Ну да: "надо сначала оба первые два узнать и с ними разобраться. Поэтому оба два считываем, смотрим, кто из них поменьше, и потом начинаем с этими изначальными данными работать в цикле."

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

)) Ну просто это стандартный подход для поиска минимума/максимума, постарался объяснить максимально подробно, почему для инициализации лучше брать элементы из самой коллекции, а не константы. Удачи!

и даже эта последовательность сработала 1 5 2 3 4? у меня сомнения, я так же пытался проделать, но не всегда правильно отрабатывает

Я писал обработку min и min2 до вайла, присваивал им первые 2 введенных значения, чтоб отказаться от вшитых значений максвалуе, но тогда программа отрабатывает не корректно, например если ввести 3 2 1 , для корректной работы нужно минимум 4 ввода с клавиатуры, отрицательные значения тоже норм обрабатывает, но при условии что чисел 4 и более, поэтому решил не ломать голову и проинициализировал переменные через максвалуе

Scanner klava = new Scanner(System.in);
     // min = klava.nextInt();
     // min2 = klava.nextInt();
      while (klava.hasNextInt()) {
          i = klava.nextInt();
          if (i< min){
              min2=min;
              min=i;

2 закомментированные строчки это как раз обработка переменных первыми 2мя вводами с клавы

Чуток всё сложнее. 1.

min = klava.nextInt();
min2 = klava.nextInt();
if (min2 < min) свопнуть min  min2;

Если первые два: 5, 1, то min должен стать равен единице, а min2 пяти. 2.

if (i< min){
     min2=min;
     min=i;
} else if (i < min2) min2 = i;

Если введены 1, 5, 3, то трёшка должна попасть в min2. Насчёт минимума 4 вводов не понял. Если вот так подправить, то корректно работает с двумя и больше значениями