JavaRush/Помощь по Java задачам/Помогите пожалуйста

1 уровень

12.11.2020
503просмотров
19комментариев

Помогите пожалуйста

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

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()]; } }

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

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

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

hidden #2322530

Уровень 41

12 ноября 2020, 09:35

ZipEntry zipEntry = zipInputStream.getNextEntry(); while (zipEntry != null){ в конце итерации надо в zipEntry положить следующий Entry

zipEntry = zipInputStream.getNextEntry();

сделай метод например copyData и в нём распиши копирование байтов из одного потока в другой, с помощью буфера.

VitalyK #1116124

Уровень 1

12 ноября 2020, 09:37

Спасибо, буду пробовать

VitalyK #1116124

Уровень 1

13 ноября 2020, 12:23

использовал copyDate который я уже использовал в getFilesList() private void copyData(InputStream in, OutputStream out) throws Exception { byte[] buffer = new byte[8 * 1024]; int len; while ((len = in.read(buffer)) > 0) { out.write(buffer, 0, len); } } вот как получилось НО не проходит try (ZipInputStream zipInputStream = new ZipInputStream(Files.newInputStream(file))) { ZipEntry zipEntry = zipInputStream.getNextEntry(); while (zipEntry != null) { String name = zipEntry.getName(); FileOutputStream fos = new FileOutputStream(String.valueOf(Files.newOutputStream(outputFolder)) + name); copyData(zipInputStream, fos); fos.close(); zipEntry = zipInputStream.getNextEntry(); } zipInputStream.closeEntry(); zipInputStream.close(); }

hidden #2322530

Уровень 41

13 ноября 2020, 12:51

какой то странный у тебя способ сделать поток на запись

FileOutputStream fos = new FileOutputStream(String.valueOf(Files.newOutputStream(outputFolder)) + name);

ты получил поток, а потом его перевёл в строку, а потом добавил к нему имя файла и все это в новый поток 🤔

VitalyK #1116124

Уровень 1

13 ноября 2020, 13:30

ннн да ..... зациклился :) не обновил валик, вот последний вариант FileOutputStream fos = new FileOutputStream(String.valueOf(outputFolder));

hidden #2322530

Уровень 41

13 ноября 2020, 13:34

ты каждый файл в архиве записываешь с одним выходным путем.

VitalyK #1116124

Уровень 1

13 ноября 2020, 13:56

если я правильно понимаю нужны относительные пути относительно введённого пути (outputFolder) ?

hidden #2322530

Уровень 41

13 ноября 2020, 14:09

тебе нужно соединить outputFolder и имя конкретного Entry, таким образом у тебя получится полный путь для потока. я бы даже так попробовал:

FileOutputStream fos = Files.newOutputStream(outputFolder + File.separator + name);

под рукой кода нет, глянуть как правильно.

VitalyK #1116124

Уровень 1

13 ноября 2020, 17:30

не проходит try (ZipInputStream zipInputStream = new ZipInputStream(Files.newInputStream(file))) { ZipEntry zipEntry = zipInputStream.getNextEntry(); String name = zipEntry.getName(); Path parent = Paths.get(outputFolder + File.separator + name); while (zipEntry != null) { FileOutputStream fos = (FileOutputStream) Files.newOutputStream(parent); // BufferedOutputStream fos = (BufferedOutputStream) Files.newOutputStream(parent); copyData(zipInputStream, fos); fos.close(); zipEntry = zipInputStream.getNextEntry(); } zipInputStream.closeEntry(); zipInputStream.close(); }

hidden #2322530

Уровень 41

13 ноября 2020, 17:31

это должно быть в цикле! сразу после while,а то ты занёс туда значения первого Entry и больше не меняешь их.

String name = zipEntry.getName();
Path parent = Paths.get(outputFolder + File.separator + name);

VitalyK #1116124

Уровень 1

13 ноября 2020, 17:50

были в цикле, не помогает try (ZipInputStream zipInputStream = new ZipInputStream(Files.newInputStream(file)) { ZipEntry zipEntry = zipInputStream.getNextEntry(); while (zipEntry != null) { String name = zipEntry.getName(); Path parent = Paths.get(outputFolder + File.separator + name); FileOutputStream fos = (FileOutputStream) Files.newOutputStream(parent); // BufferedOutputStream fos = (BufferedOutputStream) Files.newOutputStream(parent); copyData(zipInputStream, fos); fos.close(); zipEntry = zipInputStream.getNextEntry(); } zipInputStream.closeEntry(); zipInputStream.close(); }

VitalyK #1116124

Уровень 1

14 ноября 2020, 09:09

меня смущает эта строка --- Path parent = Paths.get(outputFolder + File.separator + name); Если я правильно понимаю то мы задаём свою папку outFolder -- допустим C:/ARC/ дальше мы задаём имя файла которое поднимаем из архива zipEntry.getName(); допустим 111.txt И вопрос, а где за архивированные папки ? Я так понимаю что они должны быть созданы относительно папки outFolder. Так ?

hidden #2322530

Уровень 41

14 ноября 2020, 09:15

по идее да

VitalyK #1116124

Уровень 1

14 ноября 2020, 09:16

но так тоже не проходит Path parent = outputFolder.resolve(name);

hidden #2322530

Уровень 41

14 ноября 2020, 09:20

его надо добавить к outputFolder наверное.

VitalyK #1116124

Уровень 1

14 ноября 2020, 09:24

так ? Path parent = Paths.get(outputFolder.resolve(zipEntry.toString()) + name); не прошло :(

VitalyK #1116124

Уровень 1

14 ноября 2020, 13:05

вот где ошибка -- Path file = zipFile.getFileName(); -- это не нужно делать нужно просто zipFile вставлять в zip поток :) try (ZipInputStream zipInputStream = new ZipInputStream(Files.newInputStream(zipFile)))

hidden #2322530

Уровень 41

14 ноября 2020, 13:06

решение

ну тебя с именами выходных файлов бы всё равно развернули потом)

VitalyK #1116124

Уровень 1

14 ноября 2020, 13:10

да ... но похоже там не нужно создавать относительные директории а в указанную мною папку ,кладутся файлы все в кучу. Я вот так прошел и проверку еще сделал (не знаю нужна ли была) Path parent = outputFolder.resolve(zipEntry.getName()); if (Files.notExists(parent.getParent())) { Files.createDirectories(parent.getParent()); } OutputStream fos = Files.newOutputStream(parent);