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

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

Чтение файла

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 Exception { InputStream input; BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String s = reader.readLine(); reader.close(); input = new FileInputStream(s); while(input.available()>0){ char c =(char)input.read(); System.out.println(c); } input.close(); } }

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

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

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

hidden #598481

Уровень 27

14 октября 2019, 18:01

полезный

char c =(char)input.read();

Эта штука сработает только для файла в однобайтовой кодировке. А если используется, например, UTF-8? Ну и ты зачем-то выводишь символ каждого байта с новой строки. Давай запишем файл перед тем, как его прочитать твоим алгоритмом:

public static void main(String[] args) throws IOException {
    try (PrintWriter fileWriter = new PrintWriter(new File("Test1318.txt"))) {
        fileWriter.print("ABCDE АБВГД");
        fileWriter.flush();
    }

    InputStream input;
    BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
    String s = reader.readLine();
    reader.close();
    input = new FileInputStream(s);
    while (input.available() > 0) {
        char c = (char) input.read();
        System.out.println(c);
    }
    input.close();
}

И вот что будет в консоли:

Test1318.txt // это мы  ввели название сохраненного файла
A
B
C
D
E

Ð

Ð

Ð

Ð

Ð

И это притом, что мы записали в файл всего одну строку. Так что, пожалуй, программа действительно выводит слишком много строк.

hidden #598481

Уровень 27

14 октября 2019, 18:05

Возможно, на этом этапе валидатор проверяет решение без кириллицы, в этом случае тебе достаточно убрать ln из println

Пирожок

Уровень 33

Expert

14 октября 2019, 18:25

Спасибо за ответ. Эта штука сработает только для файла в однобайтовой кодировке. А если используется, например, UTF-8? А как собственно выводить символы в любой кодировке?

hidden #598481

Уровень 27

14 октября 2019, 19:37

ну если коротко, то не переводить байт в чар, не зная кодировки. Если оставлять байты байтами, а символы символами, в зависимости от того, что имеешь на входе, то все будет хорошо. Это безотносительно этого задания, а в целом. Вообще, Java как раз работает с UTF-8. Поэтому как раз для этой кодировки байты в строку можно правильно преобразовать без указания чарсета:

try (PrintWriter fileWriter = new PrintWriter(new File("Test1318.txt"))) {
    fileWriter.print("ABCDE АБВГД");
    fileWriter.flush();
}
InputStream input;
BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
String s = reader.readLine();
reader.close();
input = new FileInputStream(s);
byte[] buffer = new byte[input.available()];
while (input.available() > 0) {
    int size = input.read(buffer);
}
System.out.println(new String(buffer));
input.close();

Но вообще для чтения текстовых файлов на практике не используют стримы напрямую: у них полно удобных оболочек, которые позволяют работать непосредственно со строками, что бесуловно намного привычнее.