не проходит последний кейс тестов

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

Решен

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;


    private static List<Integer> list = new ArrayList<>();

    public static void main(String[] args) throws IOException {
        readConsole();
        list.sort(Collections.reverseOrder());

        System.out.println(list.get(0) + " " + list.get(1) + " " + list.get(2));
    }

    private static void readConsole() throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        for (int i = 0; i < 3; i++)
            list.add(Integer.parseInt(reader.readLine()));
    }

Ошибка последнего кейса. Если ввести 1, 2, 3, то должно быть 3 2 1 Не смотря на то что если ввести 1, 2, 3, то выводится 3 2 1 Думаю ошибка в валидаторе

Ввести с клавиатуры три числа, и вывести их в порядке убывания.
Выведенные числа должны быть разделены пробелом.

Требования:

Программа должна считывать числа c клавиатуры.
Программа должна выводить числа на экран.
Программа должна выводить три числа разделенных пробелами.
Программа должна выводить числа в порядке убывания.

package com.javarush.task.task04.task0420; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; import java.util.Collections; import java.util.Comparator; import java.util.List; /* Сортировка трех чисел */ public class Solution { static List<Integer> list = new ArrayList<>(); public static void main(String[] args) throws IOException { readConsole(); list.sort(Collections.reverseOrder()); System.out.println(list.get(0) + " " + list.get(1) + " " + list.get(2)); } public static void readConsole() throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); for (int i = 0; i < 3; i++) list.add(Integer.parseInt(reader.readLine())); } }

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

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

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

TORNADO

Уровень 12

19 февраля 2021, 13:31

решение

Я совсем не знаю как именно проверяются результаты, но... Запуск из браузера: Запуск IDE: Значит буду избегать использование статических переменных класса. Было бы не плохо сделать пометку что может повлиять на результат. решено, решение:

public static void main(String[] args) throws IOException {
    List<Integer> list = new ArrayList<>();
    readConsole(list);
    list.sort(Collections.reverseOrder());
    System.out.println(list.get(0) + " " + list.get(1) + " " + list.get(2));
}

public static void readConsole(List<Integer> list) throws IOException {
    BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
    for (int i = 0; i < 3; i++)
        list.add(Integer.parseInt(reader.readLine()));
}

hidden #2322530

Уровень 41

19 февраля 2021, 13:40

решение

использование static это уже твоя инициатива. если ты используешь инструмент, ты должен понимать как он работает и зачем ты его используешь. ты проверяешь интерактивно. загружаешь 1 раз программу, она работает и завершается. а валидатор это набор тестов, и он твою программу загружает 1 раз, и гоняет метод main несколько раз по разному набору тестовых данных. а так как static принадлежит классу, то эта переменная (и данные в ней) будут существовать всё время, пока класс загружен. Я совсем не знаю как именно проверяются результаты, но... можешь почитать про JUnit тестирование, чтобы получить некое представление о работе валидатора.

TORNADO

Уровень 12

19 февраля 2021, 13:52

Про работу JUnit я знаю. Не был уверен что проверки бегают по тестам. Да если честно, вообще не думал о том как работает тут тестирование. В следующих задачах буду иметь это ввиду.

hidden #2322530

Уровень 41

19 февраля 2021, 12:23

ошибка не в валидаторе, а вот здесь:

static List<Integer> list = new ArrayList<>();

for (int i = 0; i < 3; i++)
    list.add(Integer.parseInt(reader.readLine()));

валидатор загружает класс один раз, а метод main запускает много раз. в таком случае если метод main запустить 10 раз, то у тебя в списке list будет уже не 3 числа, а 30. а значит на экран будут выводиться совсем другие числа (не те что вводили)