Что я делаю не правильно?

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

Чтение файла

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

Решен

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

Считать с консоли путь к файлу.
Вывести в консоли (на экран) содержимое файла.
Освободить ресурсы. Закрыть поток чтения с файла и поток ввода с клавиатуры.

Требования:

Программа должна считывать c консоли путь к файлу.
Программа должна выводить на экран содержимое файла.
Поток чтения из файла (FileInputStream) должен быть закрыт.
BufferedReader также должен быть закрыт.

package com.javarush.task.task13.task1318; import java.io.*; import java.util.Scanner; /* Чтение файла */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader br = new BufferedReader(new InputStreamReader(System.in)); String fileName = br.readLine(); br.close(); BufferedReader br1 = new BufferedReader(new InputStreamReader(new FileInputStream(fileName))); while(br1.readLine() != null) { System.out.println(br1.readLine()); } br1.close(); // напишите тут ваш код } }

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

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

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

Slava Olshanetskiy

Уровень 18

8 мая 2020, 10:01

Validator ругался каждый раз на то что поток FileInputStream не закрыт при строке вида: BufferedReader inputStream = new BufferedReader(new InputStreamReader(new FileInputStream(inputFileName))); Пришлось вынести часть FileInputStream is = new FileInputStream(inputFileName); и отдельно закрывать этот поток. По идее try catch сам закрывает все потоки и не нужно писать команду close()

Александр Трофимов

Уровень 23

27 мая 2019, 10:40

Я переделал так, чтобы принимал валидатор, но хочется понять что не так было в прошлом решении.

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

        InputStream is = new FileInputStream(fileName);



        while(is.available()>0)
        {
            System.out.print((char) is.read());
        }
        is.close();
        // напишите тут ваш код
    }
}

Justinian Judge в Mega City One

Master

27 мая 2019, 10:46

решение

while(br1.readLine() != null)   //читаем первую(каждую нечетную) строку
     {
         System.out.println(br1.readLine());// (еще раз читаем вторую (каждую четную) и печатаем
     }

обрати внимание на лекции. Там условие while вот так:

while(str=reader.readLine)!=null) {
  System.out.println(str);
}

таким образом мы не потеряем информацию. Каждый readLine читает следующу строку. У тебя было ДВА ридлайна. И один вывод в консоль. Плюс ридер открыт в трехэтажном конструкторе. Ты закрывал ридер, который в теории должен закрывать ресурсы открытые в нем, но это в теории, что делал твой предыдущий код ни ты, ни я не знаем, может ребята другие подскажут.

FileInputStream inputStream = new FileInputStream(fileName);
BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream));

....
reader.close;
inputStream.close;

Лично я обычно так делаю, и понятней что происходит и больше контроля над ресурсами. И валидатор доволен. И всем хорошо

Александр Трофимов

Уровень 23

27 мая 2019, 10:58

Спасибо большое. Жаль, что не могу проверить свой вариант с BufferedReader(( Но буду считать, что ошибка с двойным чтением была единственной. Почему не закрылся файл так и не понял. Тоже буду ждать подсказки. Уж очень удобно это всё в одну строку городить)))

Justinian Judge в Mega City One

Master

27 мая 2019, 11:22

Впереди еще миллион задач на BufferedReader и закрытие ресурсов, еще проверишь! :) Но на своем примере ты убедился какая цена ждет тех кто любит городит в одну строку. Это удобно, но ничем хорошим это не заканчивается ) Глянул еще немного, пишут что закрытие враппера (BufferedReader) приводит к закрытию ресурсов которые он оборачивает, правда могут быть нюансы при работе с файлами, garbage collectors и тд.