Запутался, что-то не понимаю.

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

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

Решен

Как запустить тест программы, с Winrar архивами не работает.

if(entry.isDirectory()){
                    Files.createDirectories(outputFolder.resolve(entry.getName()));
                     os = new FileOutputStream(outputFolder.resolve(entry.getName()).toString());
                }

здесь я проверяю является ли энтри директорией если да создаю директории по пути (outputFolder.resolve(entry.getName())

Пора попробовать что-нибудь распаковать. Для этого добавим публичный метод 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()]; } }

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

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

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

Галкин Юрий

Уровень 41

14 августа 2021, 14:45

Чаще всего можно самому придумать тест, который завалит программа. Попробуйте начать с этого. И напишите, какие тесты проводили.

Алексей

Уровень 35

14 августа 2021, 15:02

попробую) спасибо что откликнулись !

Алексей

Уровень 35

15 августа 2021, 15:42

После этой строки вылетает

os =  Files.newOutputStream(outputFolder/*.resolve(entry.getName())*/);

Галкин Юрий

Уровень 41

15 августа 2021, 21:16

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

outputFolder/*.resolve(entry.getName())*/

А ошибка эта всегда вылезает, на каких данных смотрели? Я так понял только от path зависит. Так? Вт он же разный бывает. На разных вариантах пути всегда тут вылетает?

Алексей

Уровень 35

18 августа 2021, 20:23

честно говоря, сейчас все упаковывает и распаковывает. Не понимаю почему валидатор не принимает по 4 пункту.

Галкин Юрий

Уровень 41

19 августа 2021, 08:36

решение

> Внутри архива некоторые файлы могут лежат внутри папок, тогда метод getName() элемента архива ZipEntry, вернет не совсем имя, как может показаться из названия, а относительный путь внутри архива. Этот относительный путь должен сохраниться и после распаковки, но уже относительно той директории, куда мы распаковали архив Это тоже выполняется?

Галкин Юрий

Уровень 41

19 августа 2021, 08:39

а "всё упаковывает и распаковывает"... всё, это какие варианты проверялись?

Галкин Юрий

Уровень 41

19 августа 2021, 08:56

Да, судя по тому, как люди отвечают, как я сам ищу темы с вопросами... Я предлагаю ещё раз задать вопрос по этой задаче, чтобы кто-то новый посмотрел. Данная тема уже уехала далеко вниз и в ней идёт обсуждение... Полагаю это существенно снижает шанс, что кто-то новый посмотрит это обсуждение.