Непонятное что-то.

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

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

Решен

Подскажите пож-та камрады в чем моя ошибка. Вроде распаковывает все правильно. Но что-то валидатору не подходит. Подозреваю, что упустил какой-то нюанс условия задачи.

Пора попробовать что-нибудь распаковать. Для этого добавим публичный метод void extractAll(Path outputFolder) throws Exception в класс ZipFileManager.
Path outputFolder - это путь, куда мы будем распаковывать наш архив. У тебя уже большой опыт работы с элементами архива и потоками. Так что, я дам только подсказки по реализации этого метода, а тебе придется хорошенько подумать, как это все сделать:
1. Проверь, есть ли zip файл вообще
2. Если директория outputFolder не существует, то ее нужно создать, как и все папки, внутри которых она лежит.
3. Внутри архива некоторые файлы могут лежат внутри папок, тогда метод getName() элемента архива ZipEntry, вернет не совсем имя, как может показаться из названия, а относительный путь внутри архива. Этот относительный путь должен сохраниться и после распаковки, но уже относительно той директории, куда мы распаковали архив
4. Реализуй метод execute() класса ZipExtractCommand, по аналогии с таким же методом класса ZipCreateCommand, сделай такой же блок try-catch, только поменяй сообщения выводимые пользователю, чтобы он понял, что сейчас мы будем распаковывать архив, и что нужно ввести полное имя архива и директорию, куда будем распаковывать.
Не забудь вызвать метод extractAll класса ZipFileManager, а не createZip, как это было в ZipCreateCommand
5. Запускай программу и наслаждайся результатом распаковки

Примечание:
Для получения потоков чтения и записи используй Files.newInputStream(Path path), Files.newOutputStream(Path path).

Требования:

В классе ZipFileManager нужно создать публичный метод void extractAll(Path outputFolder) throws Exception.
Метод extractAll(Path outputFolder) должен бросать исключение WrongZipFileException, если файл архива не существует.
Метод extractAll(Path outputFolder) должен создавать директорию outputFolder, если она не существует.
Метод extractAll(Path outputFolder) должен распаковывать все файлы из архива в директорию outputFolder.
Метод execute() в классе ZipExtractCommand должен получать объект ZipFileManager и разархивировать файлы в указанную пользователем директорию.

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)

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

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

Дмитрий

Уровень 37

15 октября 2019, 22:30

Ну смотри,

if (Files.notExists(outputFolder)) {
            Files.createDirectories(outputFolder);
        }

Если папка не существует то ты ее создаешь, а что если и сам путь к этой папке состоит из не созданных папок?

Elch

Уровень 32

17 октября 2019, 11:51

Но ведь тут Files.createDirectories(не createDirectory), который создает родительские каталоги сначала. Потестил сейчас - все корректно, если родительского каталога нет, то он создается, пробовал вложенные каталоги (test\test2\test3) - все корректно.

Pavlo Plynko Java-разработчик в CodeGym

Expert

17 октября 2019, 13:17

А если ентри будет типа: /output/folder1/file2.txt, и при этом folder1 не существует?

Дмитрий

Уровень 37

17 октября 2019, 21:08

Ты прав, я спутал с createDirectory. На момент моего ответа, я не знал об методе "Files.createDirectories" и вообще создавал отдельный метод, который выполнял работу Files.createDirectories)) Files.createDirectories - создает все несуществующие директории в переданном пути

Elch

Уровень 32

18 октября 2019, 11:23

Pavlo Сейчас потестил и такой вариант, тоже все корректно: каталог создан, файл распакован.

Pavlo Plynko Java-разработчик в CodeGym

Expert

21 октября 2019, 07:42

А как создаешь архив? Я создал архив с пом. твоей программы, включающий файл во вложенной папке, но распаковать его не могу, возникает NoSuchFileException

Создание архива.
Введите полный путь файла архива:
c:\myTemp\111out\1.zip
Введите полное имя файла или директории для архивации:
c:\myTemp\111\
Архив создан.

Введите полный путь файла архива:
c:\myTemp\111out\1.zip
Введите полное имя файла или директории для распаковки:
c:\myTemp\111out\
Произошла ошибка. Проверьте введенные данные.

Я так понимаю, что не создается именно вложенная папка "222", т.к. для нее в архиве нет отдельной ентри, есть только одна ентри для файла: 222\asd.txt

4
Просмотр содержимого архива.
Введите полный путь файла архива:
c:\myTemp\111out\1.zip
Содержимое архива:
222\asd.txt	0 Kb (0 Kb) сжатие: -100%
Содержимое архива прочитано.

Elch

Уровень 32

30 октября 2019, 08:43

решение

Спасибо Pavlo. Похоже, что в этом дело. Потестил так же - та же ошибка. Буду копать. Копал, копал, не докопал :-). Создает архив CreateZIp, который на каком-то этапе был заменен на вариант составителя задачи. Сдался, взял код extractAll из другого вопроса, вставил - работает, валидатор принял. При просмотре созданного архива, выдает то же самое, что и мой код. По коду мне мой вариант больше нравится :-). Разобраться интересно, но уже замучался, не хочу терять время.