JavaRush/Помощь по Java задачам/Выдает неизвестную ошибку компиляции в 27 строке.

38 уровень

03.01.2022
494просмотров
22комментариев

Выдает неизвестную ошибку компиляции в 27 строке.

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

Архивный

Уверен, что в коде также есть и другие ошибки, которые не дадут выполниться заданию верно, если кого-то не затруднит, то просьба указать на них

1. На вход метода main() подаются два параметра.
Первый - path - путь к директории, второй - resultFileAbsolutePath - имя (полный путь) существующего файла, который будет содержать результат.
2. Переименовать resultFileAbsolutePath в allFilesContent.txt (используй метод FileUtils.renameFile(), и, если понадобится, FileUtils.isExist()).
3. Для каждого файла в директории path и в ее всех вложенных поддиректориях выполнить следующее:
Если у файла длина в байтах НЕ больше 50, то записать его содержимое в allFilesContent.txt.
После каждого тела файла записать "\n".
Все файлы имеют расширение txt.
В качестве разделителя пути используй "/".
Для создания файлов используй конструктор File(String pathname).

Требования:

Файл, который приходит вторым параметром в main, должен быть переименован в allFilesContent.txt.
Нужно создать поток для записи в переименованный файл.
Содержимое всех файлов, размер которых не превышает 50 байт, должно быть записано в файл allFilesContent.txt.
Поток для записи в файл нужно закрыть.
Не используй статические переменные.

package com.javarush.task.task31.task3101; import java.io.BufferedWriter; import java.io.File; import java.io.FileWriter; import java.io.IOException; import java.nio.file.*; import java.nio.file.attribute.BasicFileAttributes; import java.util.TreeSet; /* Проход по дереву файлов */ public class Solution { public static void main(String[] args) { try (BufferedWriter writer = new BufferedWriter(new FileWriter("allFilesContent.txt", true))){ File path = new File(args[0]); File resultFileAbsolutePath = new File(args[1]); File allFilesContent = new File("allFilesContent.txt"); TreeSet<Path> treeSet = new TreeSet<>(); Path file = Paths.get(args[0]); if (!FileUtils.isExist(allFilesContent)) { FileUtils.renameFile(resultFileAbsolutePath, allFilesContent); } Files.walkFileTree(file, new SimpleFileVisitor<>() { @Override public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException { if (path.isFile() && path.length() < 50) { treeSet.add(file.getFileName()); return FileVisitResult.CONTINUE; } return super.visitFile(file, attrs); } }); while (!treeSet.isEmpty()) { for (Path files : treeSet) writer.write(new String(Files.readAllBytes(files))); } } catch (IOException e) { e.printStackTrace(); } } }

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

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

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

KIRAРазработчик в prosv.ru

Expert

3 января 2022, 09:27

try (BufferedWriter writer = new BufferedWriter(new FileWriter("allFilesContent.txt", true))){ вы создаете файл не там где просят, да и дописывать его не просят File resultFileAbsolutePath = new File(args[1]); вот у вас полный путь до файла, именно его нужно проверять (есть он или нет), его нужно переименовать в allFilesContent.txt 29 строка Files.walkFileTree(file, new SimpleFileVisitor<>() SimpleFileVisitor параметризированный класс, а вы ему параметр не передали. new SimpleFileVisitor<Path>()

KIRAРазработчик в prosv.ru

Expert

3 января 2022, 09:06

33 строка, не у той переменной методы вызываете 50 включительно проверять файл это или нет не нужно, visitFile вызывается только для файлов

ILYCHJava Developer в InDevLabs

4 января 2022, 07:42

Спасибо за замечания, вроде бы исправил, компиляция проходит, но все пункты красные, не могли бы вы подсказать в чем проблема?

public class Solution {
    public static void main(String[] args) {
        try (BufferedWriter writer = new BufferedWriter(new FileWriter(args[1]))){
            File path = new File(args[0]);
            File resultFileAbsolutePath = new File(args[1]);
            File allFilesContent = new File("allFilesContent.txt");
            TreeSet<Path> treeSet = new TreeSet<>();

            Path file = Paths.get(args[0]);

            if (!FileUtils.isExist(allFilesContent)) {
                FileUtils.renameFile(resultFileAbsolutePath, allFilesContent);
            }

            Files.walkFileTree(file, new SimpleFileVisitor<Path>() {
                @Override
                public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {

                    if (    file.toFile().length() < 50) {
                        treeSet.add(file.getFileName());
                        return FileVisitResult.CONTINUE;
                    }

                    return super.visitFile(file, attrs);
                }
            });

            while (!treeSet.isEmpty()) {
                for (Path files : treeSet)
                writer.write(new String(Files.readAllBytes(files)));
            }

        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

KIRAРазработчик в prosv.ru

Expert

4 января 2022, 15:42

решение

У вас есть полный путь к файлу
File resultFileAbsolutePath = new File(args[1]);
он состоит из пути и имени файла
// К примеру с:/folder/ + file.txt
Вам нужно из с:/folder/file.txt получить с:/folder/allFilesContent.txt
Переименовать с:/folder/file.txt в с:/folder/allFilesContent.txt учесть, что если такой файл уже есть  "с:/folder/allFilesContent.txt" произойдет ошибка (2 файла с одним именем быть не может)
Только когда с этим разобрались мы можем открыть поток для записи в файл
if (file.toFile().length() < 50) тут должно быть 50 включительно, и нет нужны переводить в файл, чтобы получить length(), у вас есть 2 параметр переданный в метод (attrs)
attrs.size() <= 50
Поскольку вам нужно переписать файлы в один без каких либо изменений лучше воспользоваться BufferedOutputStream(FileOutputStream)
тогда в 30 строке не нужно будет делать костыли
writer.write(Files.readAllBytes(files));
не забудьте дописать После каждого тела файла записать "\n".

ILYCHJava Developer в InDevLabs

5 января 2022, 15:32

Получилось что то типо такого

public class Solution {
    public static void main(String[] args) {

            File path = new File(args[0]);
            File resultFileAbsolutePath = new File(args[1]);
            File allFilesContent = new File(path.toString().replace(path.getName(), "allFilesContent.txt") );
            TreeSet<Path> treeSet = new TreeSet<>();

            Path file = Paths.get(args[0]);

            if (!FileUtils.isExist(allFilesContent)) {
                FileUtils.renameFile(resultFileAbsolutePath, allFilesContent);
            }

            try (BufferedOutputStream bufferedWriter = new BufferedOutputStream(new FileOutputStream(args[1]))) {
                Files.walkFileTree(file, new SimpleFileVisitor<Path>() {
                    @Override
                    public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {

                        if (attrs.size() <= 50) {
                            treeSet.add(file.getFileName());
                            return FileVisitResult.CONTINUE;
                        }

                        return super.visitFile(file, attrs);
                    }
                });

                while (!treeSet.isEmpty()) {
                    for (Path files : treeSet)
                        bufferedWriter.write(Files.readAllBytes(files));
                }
            } catch (IOException e) {
                e.printStackTrace();
            }
    }
}

KIRAРазработчик в prosv.ru

Expert

5 января 2022, 15:56

решение

вы же переименовали файл, а передаете args[1]

try (BufferedOutputStream bufferedWriter = new BufferedOutputStream(new FileOutputStream(args[1])))

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

treeSet.add(file.getFileName());
Files.readAllBytes(files) //где этот файл не известно

KIRAРазработчик в prosv.ru

Expert

5 января 2022, 16:08

решение

Цикл while тут не нужен и пропустили После каждого тела файла записать "\n".

while (!treeSet.isEmpty()) {
     for (Path files : treeSet)
         bufferedWriter.write(Files.readAllBytes(files));
}

ILYCHJava Developer в InDevLabs

5 января 2022, 16:31

Вроде бы все исправил

public class Solution {
    public static void main(String[] args) {

            File path = new File(args[0]);
            File resultFileAbsolutePath = new File(args[1]);
            File allFilesContent = new File(path.toString().replace(path.getName(), "allFilesContent.txt") );
            TreeSet<Path> treeSet = new TreeSet<>();

            Path file = Paths.get(args[0]);

            if (!FileUtils.isExist(allFilesContent)) {
                FileUtils.renameFile(resultFileAbsolutePath, allFilesContent);
            }

            try (BufferedOutputStream bufferedWriter = new BufferedOutputStream(new FileOutputStream(allFilesContent))) {
                Files.walkFileTree(file, new SimpleFileVisitor<Path>() {
                    @Override
                    public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {

                        if (attrs.size() <= 50) {
                            treeSet.add(file);
                            return FileVisitResult.CONTINUE;
                        }

                        return super.visitFile(file, attrs);
                    }
                });

                    for (Path files : treeSet) {
                        bufferedWriter.write(Files.readAllBytes(files));
                        bufferedWriter.write("\n".getBytes());
                    }


            } catch (IOException e) {
                e.printStackTrace();
            }
    }
}

KIRAРазработчик в prosv.ru

Expert

5 января 2022, 16:37

решение

Компаратора не хватает для сета, чтобы при сортировке учитывалось только имя файла

ILYCHJava Developer в InDevLabs

5 января 2022, 16:38

сейчас попробую добавить, прошу прощение за назойливость(

ILYCHJava Developer в InDevLabs

5 января 2022, 16:55

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

TreeSet<Path> treeSet = new TreeSet<>(new Comparator<Path>() {
                @Override
                public int compare(Path o1, Path o2) {
                    return o1.getFileName().compareTo(o2.getFileName());
                }
            });

KIRAРазработчик в prosv.ru

Expert

5 января 2022, 19:31

решение

вы переименовываете только если не существует, а нужно в любом случае.

if (!FileUtils.isExist(allFilesContent)) {
  FileUtils.renameFile(resultFileAbsolutePath, allFilesContent);
}

нужно проверить, если allFilesContent уже есть удалить его переименовать файл полученный в args[1] в allFilesContent

ILYCHJava Developer в InDevLabs

6 января 2022, 08:11

Не совсем понял, если файл с названием allFilesComtemt НЕ существует, то необходимо переименовать файл args1 в allFilesContent, а в случае если он существует то необходимо удалить его и переименовать файл полученный в args1 в allFilesContent?

KIRAРазработчик в prosv.ru

Expert

6 января 2022, 08:28

полезный

Проще на примере Путь к файлу с:/folder/result.txt (args[1]) в папке с:/folder/allFilesContent.txt уже есть файл allFilesContent.txt при попытке переименовать args[1] в с:/folder/allFilesContent.txt будет ошибка Чтобы избежать ошибки нужно проверить есть ли файл с:/folder/allFilesContent.txt, если есть удалить, а только после переименовать args[1] в с:/folder/allFilesContent.txt

ILYCHJava Developer в InDevLabs

6 января 2022, 08:31

Вроде сделал, но ошибка думаю не здесь

String path = args[0];
        String resultFileAbsolutePath = args[1];
        try {
            File resultFile = new File(resultFileAbsolutePath);
            File dest = new File(resultFile.getParentFile() + "/allFilesContent.txt");
            if (FileUtils.isExist(dest)) {
                FileUtils.deleteFile(dest);
            }
            FileUtils.renameFile(resultFile, dest);

KIRAРазработчик в prosv.ru

Expert

6 января 2022, 08:43

покажите полный код

ILYCHJava Developer в InDevLabs

6 января 2022, 08:44

public class Solution {
    public static void main(String[] args) {
        String path = args[0];
        String resultFileAbsolutePath = args[1];
        try {
            File resultFile = new File(resultFileAbsolutePath);
            File dest = new File(resultFile.getParentFile() + "/allFilesContent.txt");
            if (FileUtils.isExist(dest)) {
                FileUtils.deleteFile(dest);
            }
            FileUtils.renameFile(resultFile, dest);

            BufferedOutputStream bufferedWriter = new BufferedOutputStream(new FileOutputStream(dest));
            TreeSet<Path> treeSet = new TreeSet<>(new Comparator<Path>() {
                @Override
                public int compare(Path o1, Path o2) {
                    return o1.getFileName().compareTo(o2.getFileName());
                }
            });

            Path file = Paths.get(args[0]);

                  Files.walkFileTree(file, new SimpleFileVisitor<Path>() {
                      @Override
                      public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {

                          if (attrs.size() <= 50) {
                              treeSet.add(file);
                              return FileVisitResult.CONTINUE;
                          }

                          return super.visitFile(file, attrs);
                      }
                  });

                  for (Path files : treeSet) {
                      bufferedWriter.write(Files.readAllBytes(files));
                      bufferedWriter.write("\n".getBytes());
                  }
                  bufferedWriter.flush();
                  bufferedWriter.close();
              } catch (IOException e) {
                  e.printStackTrace();
              }
    }
}

KIRAРазработчик в prosv.ru

Expert

6 января 2022, 08:52

Ошибок не нашел, по каким пунктам не проходит?

ILYCHJava Developer в InDevLabs

6 января 2022, 08:56

2 и 4 пункт Может какие то проблемы с валидатором?

KIRAРазработчик в prosv.ru

Expert

6 января 2022, 09:12

решение

Откройте поток в режиме дозаписи, а еще лучше перенести пониже, перед записью непосредственно BufferedOutputStream bufferedWriter = new BufferedOutputStream(new FileOutputStream(dest, true));

KIRAРазработчик в prosv.ru

Expert

6 января 2022, 09:23

решение

В комментариях есть информация по поводу этих пунктов, говорят проходит если после того как переименовали файл открыть поток, закрыть, а потом открыть и уже писать в него информацию с FileWriter и PrintWriter вроде как такой проблемы нет, можно переписать на него

ILYCHJava Developer в InDevLabs

6 января 2022, 09:47

Все, видимо валидатор подлагал, через сайт задачу приняло, большое спасибо за помощь!