При тестировании получаю NumberFormatException - не понимаю в чем проблема с этим кодом, хелп, плиз.

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

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

Java Core, 3 уровень, 11 лекция

Решен

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

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

Пример ввода:
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.*; import java.util.stream.Stream; /* Сортировка четных чисел из файла */ //C:\Users\Артём\Desktop\test.txt public class Solution { public static void main(String[] args) { // напишите тут ваш код try (Scanner sc = new Scanner(System.in); FileInputStream reader = new FileInputStream(sc.nextLine())){ byte [] arr = reader.readAllBytes(); String [] allBytes = new String(arr).split("\n"); for (String s:allBytes) System.out.println(s); int [] result = Arrays.stream(allBytes).mapToInt(Integer::parseInt).filter(x->(x%2==0)).toArray(); Arrays.sort(result); Arrays.stream(result).forEach(System.out::println); } catch (IOException e) { throw new RuntimeException(e); } } }

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

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

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

Денис Enterprise Java Developer

3 мая 2024, 11:42

решение

Я тебе больше скажу, твой код у валидатора даже не скомпилится, потому как метода readAllBytes() не существовало в 8й Java. Перенос строки в разных системах может быть разным. В частности на винде это CRLF а не просто LF Вполне возможно что этот самый возврат каретки тебе 25ю строку и разваливает. К слову, что помешало использовать BufferedReader? То, что в условии требуется использовать FileInputStream ведь не запрещает использовать обвязку над ним.

Артём

Уровень 28

Expert

3 мая 2024, 13:23

Спасибо! Перепроверил уже все, что мог - не подумал о разнице версий Решил обойтись без буфера просто, чтобы поменьше кода писать, раз есть такой хороший метод как ReadAllBytes() :)

Денис Enterprise Java Developer

3 мая 2024, 14:10

Не нужно пытаться писать меньше кода, пиши код так, чтобы его было удобно читать и видоизменять по необходимости. В данном конкретном примере куда проще было бы использовать BufferedReader, чтобы читать файл построчно (судя по примеру ввода, одна строка - одно число). После чего их парсить и накинуть простенькую логику по нахождению четных чисел. Что-то такое:

public static void main(String[] args) {
    List<Integer> evenNumbers = new ArrayList<>();

    Scanner scanner = new Scanner(System.in);
    String fileName = scanner.nextLine();
    try (BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream(fileName)))) {
        while (reader.ready()) {
            int num = Integer.parseInt(reader.readLine());
            if (num % 2 == 0) {
                evenNumbers.add(num);
            }
        }
    } catch (IOException ignore) {
    }

    evenNumbers.stream().sorted().forEach(System.out::println);
}

Артём

Уровень 28

Expert

5 мая 2024, 09:55

Так действительно удобнее читать, благодарю