JavaRush/Помощь по Java задачам/что не так?

20 уровень

28.01.2019
612просмотров
19комментариев

что не так?

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

Решен

на тестовом файле у меня все норм. Не могу понять.

В метод main первым параметром приходит имя файла1, вторым - файла2.
Файл1 содержит строки со словами, разделенными пробелом.
Записать через пробел в Файл2 все слова, которые содержат цифры, например, а1, abc3d или 564.
Закрыть потоки.

Требования:

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

package com.javarush.task.task19.task1923; /* Слова с цифрами */ import java.io.*; import java.util.regex.Pattern; public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new FileReader(args[0])); //BufferedWriter wr = new BufferedWriter(new FileWriter(args[1])); FileWriter wr = new FileWriter(args[1]); String str = ""; String[] arr = new String[1000]; Pattern pattern = Pattern.compile(".*\\d+.*"); while ((str = reader.readLine())!= null) { arr = str.split(" "); for (String s: arr) { if( !pattern.matcher(s).matches() ) continue; wr.write(s); wr.write(" "); } wr.write("\r\n"); wr.flush(); } reader.close(); wr.close(); } }

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

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

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

Артем Divertitto Senior Android-разработчи в United Tech

Master

30 января 2019, 11:05

решение

Думаю wr.write("\r\n"); лишнее

IceBerg

Уровень 41

Master

29 января 2019, 00:47

Не переводи строку, просят просто всё слова через пробел, ну и регулярку поправь

String[] arr = new String[1000];

- зачем там делать? Почему не просто

String[] arr = str.split(" ");

Вадим

Уровень 20

29 января 2019, 21:44

не люблю в цикле объявлять... в С такого нельзя было делать вообще,

IceBerg

Уровень 41

Master

29 января 2019, 22:02

Так не объявляй, объяви за пределами, а в цикле используй, зачем присваивать за пределами пустой массив с ограниченным количеством элементов? А вообще, если переменная нужна только в цикле, то объявлять внутри цикла это нормальная практика, мне тоже сначала казалось это кривым кодом, но на деле так даже лучше для производительности, так как переменная сразу станет доступной для сборщика, как только завершиться цикл.

Вадим

Уровень 20

29 января 2019, 22:51

Это освободит ресурсы, но никак не ускорит работу. Вообще насчет объявления в цикле от гуру бы услышать чего. Впрочем изменил как вы хотели. Толку мало, ошибка вылетает все равно...

IceBerg

Уровень 41

Master

29 января 2019, 23:14

Читаю ваш код ниже, так перевод строки остался же

if (reader.ready()) wr.write("\r\n");

Вадим

Уровень 20

29 января 2019, 23:20

он в конец файле не пишется, только строчки переводит.

IceBerg

Уровень 41

Master

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

Ну так) Тебе же нужно просто все совпадения перечислить через пробел.

Вадим

Уровень 20

29 января 2019, 23:26

ну ладно, исправил, мимо :)

Pattern pattern = Pattern.compile("\\w*\\d+\\w*");
        boolean flag = false;
        while ((str = reader.readLine())!= null) {
            arr = str.split(" ");

            for (int i = 0; i < arr.length; i++) {
                if( !pattern.matcher(arr[i]).matches() ) continue;

                if (i != 0 && flag) wr.write(" ");
                wr.write(arr[i]);
                if (!flag) flag = true;
            }

            flag = false;

            if (reader.ready()) wr.write(" ");
        }

IceBerg

Уровень 41

Master

29 января 2019, 23:28

Нееее, символы верни)))

Вадим

Уровень 20

29 января 2019, 23:34

так запятые тогда если есть в словах, попадают :) а ведь ты прав, знаки препинания тоже считаются словами. Надо Сергею написать. :)

IceBerg

Уровень 41

Master

29 января 2019, 23:43

Так принял?

Вадим

Уровень 20

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

да, это ппц на самом деле. Я считаю этонеправильно, но меня никто не спрашивает. Вообще, у меня что то эти задания идут не очень весело... за весь курс такого небыло

IceBerg

Уровень 41

Master

30 января 2019, 18:51

Я на символах тоже погорел сначала.

Sergey Moroz

Уровень 26

Expert

29 января 2019, 00:12

Беда с пунктуацией и переводом строки - лепятся в файл на выходе

Вадим

Уровень 20

29 января 2019, 21:43

гм... и вот где правда брат :)

Вадим

Уровень 20

29 января 2019, 22:53

изменил, уже в файл ничего не дописывает, нет перевода строки

while ((str = reader.readLine())!= null) {
            arr = str.split(" ");

            for (String s: arr) {
                if( !pattern.matcher(s).matches() ) continue;

                wr.write(s);
               wr.write(" ");
            }

            if (reader.ready()) wr.write("\r\n");
            wr.flush();
        }

вот такой код как бы, а эффект такой же

Вадим

Уровень 20

29 января 2019, 23:39

Беда с пунктуацией

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

Sergey Moroz

Уровень 26

Expert

30 января 2019, 00:35

Да, я тоже натерпелся в этой задаче от валика, который как оказалось под словами с цифрами понимает любую последовательность любых символов (в том числе и с пунктуацией), главное промеж пробелов - то есть вот это нужно считать одним словом, а не тремя

ф№"%?58ы.ыф