Провел тесты, разархивация многоуровневой иерархии папок происходит корректно, но валидатор не принимает решение. Что я упускаю ?

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

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

Решен

/**
     * Извлечение содержимого архива
     */
    public void extractAll(Path outputFolder) throws Exception {
        if (!Files.isRegularFile(zipFile))
            throw new WrongZipFileException();

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

//поток в сторону архива на чтение
        try (ZipInputStream zis = new ZipInputStream(Files.newInputStream(zipFile))) {
            ZipEntry zipEntry;
            while ((zipEntry = zis.getNextEntry()) != null) { //получение следующего элемента из архива и проверка на null
                Path file_name_abs = outputFolder.resolve(zipEntry.getName()); //полный путь для разархивации с учётом вложенности в архиве

                if (zipEntry.isDirectory()) {  // проверка zipEntry на дирректорию
                    if (Files.notExists(file_name_abs)) {  //если это дирректория и она не добавлена...
                        Files.createDirectories(file_name_abs);  // то создаем её
                    }
                    zis.closeEntry(); //завершаем работу с текущим zipEntry - элементом массива, на следующим цикле получим новый
                    continue; //завершаем текущую итерацию на этой строке, сразу переходим на следующую. Так как код ниже только для файлов !!!
                }
                copyData(zis, Files.newOutputStream(file_name_abs));  //если проверка выше на папку не пройдена => это файл, разархивируем его
                zis.closeEntry();  //завершаем работу с текущим zipEntry
            }
        }
    }

Пора попробовать что-нибудь распаковать. Для этого добавим публичный метод 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; //C:\JavaProjects\Trash_Projects\ZIP_TEST\ZIP_ARCH\arch1.zip //C:\JavaProjects\Trash_Projects\ZIP_TEST\ZIP_UNARCH 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()]; } }

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

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

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

Андрей Гузанов

Уровень 37

29 декабря 2021, 09:05

Со своей стороны отрефакторил немного код (для себя, валидацию не проходит, мб кому то интересно):

/**
 * Извлечение содержимого архива
 */
public void extractAll(Path outputFolder) throws Exception {
    if (!Files.isRegularFile(zipFile))
        throw new WrongZipFileException();

    try (ZipInputStream zis = new ZipInputStream(Files.newInputStream(zipFile))) {
        if (Files.notExists(outputFolder))
            Files.createDirectories(outputFolder);

        ZipEntry zipEntry;
        while ((zipEntry = zis.getNextEntry()) != null) {
            Path file_name_abs = outputFolder.resolve(zipEntry.getName());

            if (zipEntry.isDirectory()) {
                if (Files.notExists(file_name_abs)) {
                    Files.createDirectories(file_name_abs);
                }
            } else {
                copyData(zis, Files.newOutputStream(file_name_abs));
            }
            zis.closeEntry();
        }
    }
}

Андрей Гузанов

Уровень 37

29 декабря 2021, 09:05

решение

Мда, подсмотрел "правильное решение", в итоге понял, что валидатор принимает только не правильное решение... то есть решение, без обработки вложений. Вот правильное решение от JR:

public void extractAll(Path outputFolder) throws Exception {
        // Проверяем существует ли zip файл
        if (!Files.isRegularFile(zipFile)) {
            throw new WrongZipFileException();
        }

        try (ZipInputStream zipInputStream = new ZipInputStream(Files.newInputStream(zipFile))) {
            // Создаем директорию вывода, если она не существует
            if (Files.notExists(outputFolder))
                Files.createDirectories(outputFolder);

            // Проходимся по содержимому zip потока (файла)
            ZipEntry zipEntry = zipInputStream.getNextEntry();

            while (zipEntry != null) {
                String fileName = zipEntry.getName();
                Path fileFullName = outputFolder.resolve(fileName);

                // Создаем необходимые директории
                Path parent = fileFullName.getParent();
                if (Files.notExists(parent))
                    Files.createDirectories(parent);

                try (OutputStream outputStream = Files.newOutputStream(fileFullName)) {
                    copyData(zipInputStream, outputStream);
                }
                zipEntry = zipInputStream.getNextEntry();
            }
        }
    }

Видно, что нет проверки на ПАПКУ. Попытки затестить такой код приводят к ошибке (в моем архиве я создал папки), так как пытается вычитать папку как файл, чего, на сколько я знаю, делать низя. Ошибки нет только если нет папок в архиве. Вывод, нужно решить не правильно, что бы продвинуться :)

Виктор

Уровень 1

1 октября 2022, 17:01

Господи, какая бредовая задача! Спасибо тебе, дружище, за совет: Вывод, нужно решить не правильно, что бы продвинуться :) Оставлю своё решение здесь:

public void extractAll2(Path outputFolder) throws Exception {
    // проверяем существует ли zip=file
    if (!Files.isRegularFile(zipFile)) {
        throw new WrongZipFileException();
    }

    // Если директория outputFolder не существует, то ее нужно создать,
    // как и все папки, внутри которых она лежит
    if (Files.notExists(outputFolder)) {
        Files.createDirectories(outputFolder);
    }

    try (ZipInputStream zipInputStream = new ZipInputStream(Files.newInputStream(zipFile))) {
        ZipEntry zipEntry = zipInputStream.getNextEntry();

        while (zipEntry != null) {
            Path fullPath = outputFolder.resolve(zipEntry.getName());
            Path elementPath = Paths.get(zipEntry.getName());

            if (zipEntry.getName().endsWith("/") || zipEntry.getName().endsWith("\\")) {
                if (Files.notExists(fullPath)) Files.createDirectories(fullPath);
            } else {
                OutputStream out = Files.newOutputStream(fullPath);
                copyData(zipInputStream, out);
                out.flush();
                out.close();
            }

            // Считываем новый элемент
            zipEntry = zipInputStream.getNextEntry();
        }
    }
}