Не принимает, кучу попыток уже слил, не могу понять в чём причина

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

Java Multithreading, 7 уровень, 15 лекция

Решен

Приветствую! Вроде всё корректно сделал, отвечает заданию и логике, нашёл метод relativize(), сделал с ним, валли не принял, почитал комменты, все тоже с relativize() делают, но почему-то всё-равно не принимает.. Подскажите кому не сложно :)

Теперь займемся другой, не менее важной частью нашего архиватора.
Очень часто пользователь хочет создать архив не из одного файла, а из целой папки.
Тогда архивация сводится к поочередному добавлению элемента ZipEntry для каждого файла в архив.
Нам нужно написать класс FileManager для получения списка всех файлов в какой-то папке.
У него должен быть конструктор, который будет принимать путь Path rootPath, указывающий на корень (папку, файлы в которой нас интересуют) и метод List<Path> getFileList(), который должен возвращать список относительных путей всех файлов, которые находятся по пути rootPath, включая файлы в подпапках.
Директория, в которой будем искать, может задаваться только один раз в конструкторе.
Поэтому, прямо в конструкторе мы сразу можем сформировать список файлов (их относительных путей), сохранив его в переменную класса List<Path> fileList.
Осталось только решить, как нам собрать все файлы, даже те, которые могут быть в папке, которая в папке, которая в папке, которая в интересующей нас папке. Фух, чуть не вошел рекурсию.
Хотя нам-то она и поможет! Сделаем метод collectFileList(Path path), который будет складывать в переменную класса fileList все файлы, обнаруженные внутри переданного пути path, вызывая сам себя для всех объектов, в обнаруженных директориях.

1. Создай класс FileManager с конструктором FileManager(Path rootPath) throws IOException
2. Объяви и проинициализируй приватные переменные класса:
2.1. Path rootPath - корневой путь директории, файлы которой нас интересуют
2.2. List<Path> fileList - список относительных путей файлов внутри rootPath
3. Создай геттер для fileList
4. Реализуй метод void collectFileList(Path path) throws IOException, который должен:
4.1. Проверить, если переданный путь path является обычным файлом (используй метод Files.isRegularFile), то получить его относительный путь относительно rootPath и добавить его в список fileList.
4.2. Если переданный путь path, является директорией (узнать это поможет метод Files.isDirectory), то пройтись по всему содержимому директории и вызвать collectFileList(Path path), передав в path обнаруженные элементы.
Пройтись по всему содержимому директории можно предварительно получив DirectoryStream с помощью метода newDirectoryStream класса Files.
Не забудь закрыть созданный DirectoryStream.
5. Добавь вызов метода collectFileList(rootPath) в конструкторе FileManager.
6. Примени все свои знания об инкапсуляции к этому классу.
Выполняя это задание, ты написал алгоритм, который обходит дерево файлов.
Но в Java есть специальный интерфейс FileVisitor для этих целей.
Очень рекомендую разобраться как им пользоваться.

Требования:

В корне задачи должен быть создан класс FileManager c конструктором FileManager(Path rootPath) throws IOException.
В классе FileManager должны быть созданы приватные поля Path rootPath и List<Path> fileList, которые инициализируются в конструкторе.
В классе FileManager должен быть создан getter для поля fileList.
В классе FileManager нужно реализовать приватный метод void collectFileList(Path path) throws IOException согласно заданию.
Нужно добавить вызов метода collectFileList(rootPath) в конструктор FileManager.

package com.javarush.task.task31.task3110; import com.javarush.task.task31.task3110.exception.WrongZipFileException; import java.io.IOException; public class Archiver { public static void main(String[] args) throws IOException { Operation operation = null; do { try { operation = askOperation(); CommandExecutor.execute(operation); } catch (WrongZipFileException e) { ConsoleHelper.writeMessage("Вы не выбрали файл архива или выбрали неверный файл."); } catch (Exception e) { ConsoleHelper.writeMessage("Произошла ошибка. Проверьте введенные данные."); } } while (operation != Operation.EXIT); } public static Operation askOperation() throws IOException { ConsoleHelper.writeMessage(""); ConsoleHelper.writeMessage("Выберите операцию:"); ConsoleHelper.writeMessage(String.format("\t %d - упаковать файлы в архив", Operation.CREATE.ordinal())); ConsoleHelper.writeMessage(String.format("\t %d - добавить файл в архив", Operation.ADD.ordinal())); ConsoleHelper.writeMessage(String.format("\t %d - удалить файл из архива", Operation.REMOVE.ordinal())); ConsoleHelper.writeMessage(String.format("\t %d - распаковать архив", Operation.EXTRACT.ordinal())); ConsoleHelper.writeMessage(String.format("\t %d - просмотреть содержимое архива", Operation.CONTENT.ordinal())); ConsoleHelper.writeMessage(String.format("\t %d - выход", Operation.EXIT.ordinal())); return Operation.values()[ConsoleHelper.readInt()]; } }

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

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

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

Anton Stezhkin

Уровень 41

11 марта 2020, 12:05

rootPath.relativize(path);

- у тебя. А в задании - "путь относительно rootPath "

Evgeniy

Уровень 40

Expert

8 июня 2018, 12:28

решение

А зачем сливать попытки... во всем винить вали... ждать пока тебе ответят... ведь можно просто один раз запустить! >> Exception in thread "main" java.lang.IllegalStateException: Iterator already obtained

Сергей

Уровень 30

9 июня 2018, 11:08

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

Evgeniy

Уровень 40

Expert

9 июня 2018, 12:29

Должно быть так

Iterator<Path> iter = directoryStream.iterator();
while (iter.hasNext()) {
    collectFileList(iter.next());
}

Сергей

Уровень 30

9 июня 2018, 16:55

Да, попробовал так, всё работает, почитал немного про исключение, скорее всего ошибка была в двух запросах итератора.

Alisa Lamekhova

Уровень 40

8 июня 2018, 11:35

// Чтобы не писать код по вызову close для DirectoryStream, обернем вызов newDirectoryStream в try-with-resources

Сергей

Уровень 30

9 июня 2018, 11:10

Спасибо за совет) думаю, что в данной задаче это не обязательное условие, но вприципе его можно и провернуть)