JavaRush/Помощь по Java задачам/Программа выводит слишком мало строк

14 уровень

01.05.2022
385просмотров
7комментариев

Программа выводит слишком мало строк

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

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

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.ArrayList; import java.util.Collections; import java.util.List; import java.util.Scanner; /* Сортировка четных чисел из файла */ public class Solution { public static void main(String[] args) throws IOException { InputStream inputStream = null; Reader inputStreamReader = new InputStreamReader(System.in); BufferedReader bufferedReader = new BufferedReader(inputStreamReader); String filename = bufferedReader.readLine(); try { inputStream = new FileInputStream(filename); ArrayList<Integer> arrayList = new ArrayList<>(); while (inputStream.available() > 0) { int c = inputStream.read(); if (c % 2 == 0) arrayList.add(c); Collections.sort(arrayList); for(int s : arrayList) { System.out.print(s + " "); } } } finally { inputStream.close(); bufferedReader.close(); } } }

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

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

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

Павел Безумный учёный

Expert

2 мая 2022, 00:40

полезный

В 23 строке из потока inputStream считываются байты; само же текстовое представление числа может состоять из одного или нескольких байтов, в зависимости от количества символов в этом числе. Поскольку каждое число в исходном файле находится на отдельной строке, целесообразно считывать содержимое файла построчно, используя для этого объект BufferedReader. Каждая считанная строка преобразуется в целое число при помощи метода Integer.parseInt(), которое затем проверяется на чётность и добавляется (или не добавляется) в список arrayList. Также обратите внимание, что числа из списка arrayList должны выводиться в консоль не через пробел, а каждое с новой строки.

Ольга Бахута

Уровень 14

2 мая 2022, 14:23

Все исправила, но теперь последний три пункта не проходят, и пишет, что "Программа выводит слишком много строк".

package com.javarush.task.task13.task1326;

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

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

public class Solution {
    public static void main(String[] args) throws IOException {
        InputStream inputStream = null;
        Reader inputStreamReader = new InputStreamReader(System.in);
        BufferedReader bufferedReader = new BufferedReader(inputStreamReader);
        String filename = bufferedReader.readLine();
        try {
             inputStream = new FileInputStream(filename);
            ArrayList<Integer> arrayList = new ArrayList<>();
            while (inputStream.available() > 0) {
                int c = Integer.parseInt(bufferedReader.readLine());
                if (c % 2 == 0) arrayList.add(c);
                Collections.sort(arrayList);
                for(int s : arrayList) {
                    System.out.println(s);
                }
            }
        } finally {
            bufferedReader.close();
            inputStreamReader.close();
            inputStream.close();



        }

    }

    }

Павел Безумный учёный

Expert

2 мая 2022, 15:19

полезный

Используйте разные объекты BufferedReader для ввода имени файла с консоли и считывания строки из самого файла:

public static void main(String[] args) throws IOException {
    BufferedReader consoleReader = new BufferedReader(new InputStreamReader(System.in));
    String fileName = consoleReader.readLine();

    try (FileInputStream fis = new FileInputStream(fileName);
         BufferedReader fileReader = new BufferedReader(new InputStreamReader(fis))) {

        // . . .
    }
}

Первым делом создаётся объект consoleReader, который предназначен исключительно для чтения данных из потока ввода System.in (то есть из консоли). С его помощью считывается имя исходного файла, которое сохраняется в переменную fileName. Далее на основе полученного имени файла открывается поток типа FileInputStream, на базе которого создаётся второй объект BufferedReader, предназначенный для чтения содержимого файла (переменная fileReader). Обратите внимание на специальный синтаксис оператора try: ресурсы, открытые в круглых скобках будут автоматически освобождены при выходе из оператора, то есть беспокоиться о закрытии потоков в этом случае нет необходимости. При этом системный поток ввода закрывать не требуется (если это не оговорено особо в условии задачи), поэтому объект consoleReader создаётся вне оператора try. В цикле while считывайте строки при помощи fileReader. Цикл for выенсите за пределы цикла while, чтобы содержимое списка выводилось в консоль перед завершением работы программы, а не на каждой итерации цикла while.

Ольга Бахута

Уровень 14

2 мая 2022, 20:15

Все исправила, но вернулась к тому, что было - "Программа выводит слишком мало строк". Проблема в том, что в IDEA не получается дебажить с плагином, непонятно, что имеется в виду под "слишком мало строк". Вывод даже не посмотреть. Вот пример "дебага" :) https://prnt.sc/0uifbJvuLmCO

package com.javarush.task.task13.task1326;

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

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

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader consoleReader = new BufferedReader(new InputStreamReader(System.in));
        String fileName = consoleReader.readLine();
        try (FileInputStream fis = new FileInputStream(fileName);
             BufferedReader fileReader = new BufferedReader(new InputStreamReader(fis))) {

            ArrayList<Integer> arrayList = new ArrayList<>();
            while (fileReader.readLine() != null) {
                int c = Integer.parseInt(fileReader.readLine());
                if (c % 2 == 0) arrayList.add(c);
                Collections.sort(arrayList);
            }

                for(int s : arrayList) {
                    System.out.println(s);
                }

        }

    }

    }

Павел Безумный учёный

Expert

2 мая 2022, 20:16

решение

В условии цикла while используйте метод ready() вместо проверки считанной строки на null. Иначе получается, что за одну итерацию цикла метод readLine() вызывается дважды, то есть из файла считываются две строки с числами. И если одно число парсится и сохраняется как положено (строка 22), то другое просто "улетает мимо кассы", потому что никуда не сохраняется (строка 21). Ошибка в консоли на скриншоте появляется закономерно. После запуска программа ожидает ввод в консоль, причём введённая строка должна представлять собой полное имя считываемого файла, например:

C:\Example\testfile.txt

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

Ольга Бахута

Уровень 14

3 мая 2022, 08:48

Спасибо за подробное объяснение! Работает 😊

Павел Безумный учёный

Expert

3 мая 2022, 09:01

На здоровье) Рад, что всё получилось.