Не проходит по 4-му пункту

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

Решен

Ткните носом..

В метод main первым параметром приходит путь к файлу1, вторым - путь к файлу2.
Файл1 содержит слова, разделенные пробелом или переводом строки (в файле может быть несколько строк).
Все, что не относится к пробелу или переводу строки, разделителем не считать.
Записать в одну строку через запятую в Файл2 слова, длина которых строго больше 6.
В конце файла2 запятой не должно быть.
Закрыть потоки.

Пример выходных данных в файл2:
длинное,короткое,аббревиатура

Требования:

Программа НЕ должна считывать данные с консоли.
Программа должна считывать содержимое первого файла (используй FileReader c конструктором String).
Поток чтения из файла (FileReader) должен быть закрыт.
Программа должна записывать через запятую во второй файл все слова из первого файла длина которых строго больше 6(используй FileWriter).
Поток записи в файл (FileWriter) должен быть закрыт.

package com.javarush.task.task19.task1925; /* Длинные слова */ import java.io.*; public class Solution { public static void main(String[] args) throws IOException { BufferedReader fileReader = new BufferedReader(new FileReader(args[0])); BufferedWriter fileWriter = new BufferedWriter(new FileWriter(args[1])); while (fileReader.ready()) { String line = fileReader.readLine(); String newLine = ""; String[] temp = line.split(" "); for (String word : temp) { if (word.length() > 6) { newLine += word + " "; } } fileWriter.write(newLine.trim().replaceAll(" ", ",")); } fileReader.close(); fileWriter.close(); } }

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

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

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

Alexey Polyakov

Уровень 35

8 января 2019, 23:15

решение

Мне посоветовали переписать через StringBuilder. Совет, кстати, весьма полезен, потому как таким образом экономятся ресуры машины. Спасибо Ильяс!

BufferedReader fileReader = new BufferedReader(new FileReader(args[0]));
BufferedWriter fileWriter = new BufferedWriter(new FileWriter(args[1]));
StringBuilder stringBuilder = new StringBuilder();
while (fileReader.ready()) {
    stringBuilder.append(fileReader.readLine()).append(" ");
}
fileReader.close();

String content = stringBuilder.toString();
stringBuilder.setLength(0);

for(String s : content.split(" +")) {
    if (s.length() > 6) {
        stringBuilder.append(s).append(",");
    }
}

stringBuilder.setLength(stringBuilder.length()-1);
String words = stringBuilder.toString();
fileWriter.write(words);
fileWriter.close();

Уж не знаю в чем там был косяк. Может кто из опытных заглянет, да укажет. Было бы полезно в любом случае.

Sergey Khevoyan Android Developer в Freelancer

9 января 2019, 18:40

Косяк был в том, что вы вызываете метод replaceAll но не присваиваете его переменной. Я сделал ваше решение на реальных файлах - записывалось все с пробелами. После изменения кода - все записалось правильно.

newLine = newLine.trim().replaceAll(" ", ",");

              fileWriter.write(newLine);

Vasily Petrov

Уровень 35

22 января 2019, 11:23

А что значат эти строки? stringBuilder.setLength(0); String s : content.split(" +"))

Alexey Polyakov

Уровень 35

22 января 2019, 21:58

тут я "обнуляю" stringBuilder. Чтобы снова использовать этот же билдер дальше.

stringBuilder.setLength(0);

тут я строку content "режу" на кусочки, используя регулярное выражение "один или более пробелов".

String s : content.split(" +")

IceBerg

Уровень 41

Master

8 января 2019, 23:08

А строка точно одна? В условии об этом ни слова, но может...

Sergey Khevoyan Android Developer в Freelancer

8 января 2019, 21:50

newLine = newLine + "," + word;

Alexey Polyakov

Уровень 35

8 января 2019, 22:55

Тогда файл начнется с запятой. Понятно, что можно первое слово без запятой записать, а остальные уж разделить запятыми, но вывод отличаться не будет.

if (word.length() > 6) {
    if (newLine.length() < 6) {
        newLine = word;
    }
    else {
        newLine = newLine + "," + word;
    }
}