Почему валидатор не принимает? Программа данные на экран выводит, но что-то не так.

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

Архивный

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

package com.javarush.task.task13.task1326;

/*
Сортировка четных чисел из файла
*/

import java.io.*;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;


public class Solution {
    public static void main(String[] args) throws IOException {
        // напишите тут ваш код
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String fileName = reader.readLine(); //считали имя файла
        reader.close(); //закрыли поток чтения с консоли
        InputStream in = new FileInputStream(fileName); //открыли поток чтения из файла
        List<Integer> list = new ArrayList<>(); //массив, в который будут записываться только четные числа
        List<Integer> num = new ArrayList<>(); //массив, в который будут записываться только
        while (in.available() > 0) {  //пока можем читать
            char c = (char) in.read(); //считываем символ
            if (!(c > 47 && c < 58)) { //если это не число, то
                int i = num.size() - 1; //получаем кол-во чисел в этом числе - 1
                if (i != -1) { //если массив не пустой, то
                    int ans = 0; //в эту переменную запишем число
                    for (int j = 0; j < num.size(); j++) { //проходимся по массиву
                        int k = num.get(j);  //в k пишем текущую цифру
                        ans += k * (int) Math.pow(10, i); //домножаем её на 10 в нужной степени
                        i--; //уменшаем степень для следующего числа
                    }
                    if (ans % 2 == 0) { //проверяем получившееся число на четность
                        list.add(ans); //если четное, то добавляем в существующий итоговый массив
                    }
                    num = new ArrayList<>(); //обнуляем массив цифр
                }
            } else { //если считанный символ - цифра (от 48 до 57), то
                num.add((int) c - 48); //довавляем его в виде числа в массив num
            }
        }
        int i = num.size() - 1; //проделываем 25-35 строки (перевод массива цифр в число)
        if (i != -1) { // для того, что сейчас хранится в массиве, если там что-то есть
            int ans = 0;
            for (int k : num) {
                ans += k * (int) Math.pow(10, i);
                i--;
            }
            if (ans % 2 == 0) {
                list.add(ans);
            }
        }
        in.close(); //закрываем поток чтения из файла
        if (list.size() == 0){ //если чисел в массиве нет, то завершаем программу
            return;
        }
        Collections.sort(list); //сортируем встроенным методом, поскольку мой вариант отказался работать на один из тестов
//        Object[] mas =  list.toArray();
//        for (int j = 0; j < mas.length; j++) {
//            for (int k = j; j < mas.length; j++) {
//                if ((Integer)mas[k] >(Integer) mas[j]) {
//                    Integer l = (Integer)mas[j];
//                    mas[j] = mas[k];
//                    mas[k] = l;
//                }
//            }
//        }
//        for (Object k:mas) {
//            System.out.println(k);
//        }
        for (int j = 0; j < list.size(); j++) { //выводим результат на экран
            System.out.println(list.get(j));
        }
    }
}

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

Ввести имя файла с консоли.
Прочитать из него набор чисел.
Вывести в консоли только четные, отсортированные по возрастанию.

Пример ввода:
5 8 -2 11 3 -5 2 10

Пример вывода:
-2 2 8 10

Требования:

Программа должна считывать данные с консоли.
Программа должна создавать FileInputStream для введенной с консоли строки.
Программа должна выводить данные на экран.
Программа должна вывести на экран все четные числа, считанные из файла, отсортированные по возрастанию.
Программа должна закрывать поток чтения из файла — FileInputStream.

package com.javarush.task.task13.task1326; /* Сортировка четных чисел из файла */ import java.io.*; import java.util.ArrayList; import java.util.List; public class Solution { public static void main(String[] args) throws IOException { // напишите тут ваш код BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String fileName = reader.readLine(); reader.close(); InputStream in = new FileInputStream(fileName); List<Integer> list = new ArrayList<>(); List<Integer> num = new ArrayList<>(); while (in.available() > 0) { char c = (char) in.read(); if (!(c > 47 && c < 59)) { int i = num.size() - 1; if (i != -1) { int ans = 0; for (int k : num) { ans += k * (int) Math.pow(10, i); i--; } if (ans % 2 == 0) { list.add(ans); } num = new ArrayList<>(); } } else { num.add((int) c - 48); } } int i = num.size() - 1; if (i != -1) { int ans = 0; for (int k : num) { ans += k * (int) Math.pow(10, i); i--; } if (ans % 2 == 0) { list.add(ans); } } reader.close(); in.close(); if (list.size() == 0){ return; } Object[] mas = list.toArray(); for (int j = 0; j < mas.length; j++) { for (int k = j; j < mas.length; j++) { if ((Integer)mas[k] >(Integer) mas[j]) { Integer l = (Integer)mas[j]; mas[j] = mas[k]; mas[k] = l; } } } // for (Object k:mas) { // System.out.println(k); // } for (int j = 0; j < mas.length-1; j++) { System.out.println(mas[j]); } System.out.print(mas[mas.length-1]); } }

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

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

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

y-grek

Уровень 20

31 мая 2019, 20:35

этот код у меня прекрассно сработал.. но валидатор запросил побайтно

public static void main(String[] args) throws IOException, NullPointerException {
    BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
    FileInputStream fileInputStream = new FileInputStream(bufferedReader.readLine());
    BufferedReader fileReader = new BufferedReader(new FileReader(fileInputStream.getFD()));

    ArrayList<Integer> list = new ArrayList<>();
    while (fileReader.ready()) {
        String x = fileReader.readLine();
        if ((Integer.parseInt(x) % 2) == 0) {
            list.add(Integer.parseInt(x));
        }
    }
    Collections.sort(list);
    for (int i = 0; i < list.size(); i++) {
        System.out.println(list.get(i));
    }
    fileInputStream.close();
    bufferedReader.close();
    fileReader.close();
}

y-grek

Уровень 20

31 мая 2019, 20:31

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

public static void main(String[] args) throws IOException, NullPointerException {
    BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));
    FileInputStream fileInputStream = new FileInputStream(bufferedReader.readLine());

    ArrayList<Integer> list = new ArrayList<>();
    byte[] byteTxt = new byte[fileInputStream.available()];
    fileInputStream.read(byteTxt);
    String x = "";
    for (int i = 0; i < byteTxt.length; i++) {
        Character s = (char) byteTxt[i];
        if (s.toString().equals("-") || Character.isDigit(s)) {
            x = x + s;
        }
        else {
            if (!x.isEmpty() && ((Integer.parseInt(x) % 2) == 0)) {
                list.add(Integer.parseInt(x));
            }
            x = "";
        }
    }
    if (!x.isEmpty()) {
        list.add(Integer.parseInt(x));
    }

    Collections.sort(list);
    for (int i = 0; i < list.size(); i++) {
        System.out.println(list.get(i));
    }
    fileInputStream.close();
    bufferedReader.close();

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

Danila

Уровень 39

9 января 2018, 18:03

полезный

О, Collection.sort(), отлично! Теперь осталось избавиться от ненужного возведения в степень и будет супер)

Kemol

Уровень 3

9 января 2018, 18:11

Так и до написания всего кода через

BufferedReader fileReader = new BufferedReader(new InputStreamReader(new FileInputStream(reader.readLine())));

недалеко) Хорошо, вечером переделаю тогда.

Danila

Уровень 39

9 января 2018, 11:32

Сомнительно, что в этом дело, но проверка в строке 23 не работает как задумано(?) - интервал от 47 до 59 включает в себя, помимо цифр, двоеточие (код ascii 58). Нужны комментарии к коду. Что делает код в строчках 24-34? Похоже, какая-то неясная логика, которая выполняется, если попался символ, не являющийся цифрой - определенно, ничего подобного в задании не было. Что делает этот код?

int ans = 0;
            for (int k : num) {
                ans += k * (int) Math.pow(10, i);
                i--;
            }

Kemol

Уровень 3

9 января 2018, 17:22

Немного поправил и добавил код с комментариями в описание моей проблемы. Тот код на тест выводил числа в неверном порядке из-за ошибок в сортировке(58-59 строки были ошибочны)