Проверь, корректно ли метод createZip сохраняет директорию с вложенными файлами.

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

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

Решен

public void createZip(Path source) throws Exception {
        if (!Files.isDirectory(zipFile.getParent())) Files.createDirectory(zipFile.getParent());
        try (ZipOutputStream zos = new ZipOutputStream(Files.newOutputStream(zipFile))) {
            if (Files.isRegularFile(source)) {
                addNewZipEntry(zos, source.getParent(), source.getFileName());
            } else if (Files.isDirectory(source)) {
                FileManager fileManager = new FileManager(source);
                List<Path> fileNames = fileManager.getFileList();
                for (Path path : fileNames) {
                    addNewZipEntry(zos, path.getParent(), path.getFileName());
                }
            } else throw new PathIsNotFoundException();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    private void addNewZipEntry(ZipOutputStream zipOutputStream, Path filePath, Path fileName) throws Exception {
        Path path = filePath.resolve(fileName);
        try (InputStream inputStream = Files.newInputStream(path)) {
            //ZipEntry zipEntry = new ZipEntry(fileName.getFileName().toString());
            ZipEntry zipEntry = new ZipEntry(fileName.toString());
            zipOutputStream.putNextEntry(zipEntry);
            copyData(inputStream, zipOutputStream);
            zipOutputStream.closeEntry();
        } catch (Exception e) {
            e.printStackTrace();
        }
    }

Пробовал addNewZipEntry(zos, path.getParent(), path.getFileName()); менять на addNewZipEntry(zos, source, path.getFileName()); не помогло. Убрал еще catch (Exception e). Не помогло.

Пришло время отрефакторить класс ZipFileManager.
В методе createZip есть код, который нам также понадобится в методах, которые будут добавлять или удалять файл в архив, распаковывать его и т.д.
Эти методы мы будем реализовывать позже, но уже сейчас можем вынести общие части кода в отдельные методы.
Кроме того, метод createZip мог создавать архив только из одного файла, а хотелось бы уметь архивировать всю папку целиком.
Создавать отдельный метод для этого не будем, т.к. в createZip(Path source) можно передавать и директорию и обычный файл.

Задания на сегодня:
1. Реализуй приватный метод void addNewZipEntry(ZipOutputStream zipOutputStream, Path filePath, Path fileName) throws Exception в классе ZipFileManager.

Он должен:
1.1. Создавать InputStream, для файла с именем fileName, расположенным в директории filePath
1.2. Создавать новый элемент архива ZipEntry, в качестве имени используй fileName, преобразовав его в String
1.3. Добавить в переданный zipOutputStream новый элемент архива ZipEntry (из п.1.2)
1.4. Копировать данные из InputStream (из п.1.1) в переданный zipOutputStream
1.5. Закрывать в zipOutputStream элемент архива ZipEntry (из п.1.2)
1.6. Закрывать InputStream, сделай это с помощью try-with-resources
2. Замени часть кода метода createZip вызовом нового метода addNewZipEntry.
Передай значение source.getParent() в параметр filePath, а source.getFileName() в filename.
3. Реализуй приватный метод void copyData(InputStream in, OutputStream out) throws Exception.
Он должен читать данные из in и записывать в out, пока не вычитает все.
4. Замени часть кода метода addNewZipEntry на вызов метода copyData
5. Вернемся к createZip:
5.1. В начале метода проверь, что существует директория (zipFile.getParent()), в которой мы будем создавать zipFile, если ее нет, то создай ее.
5.2. Если source является обычным файлом (для проверки используй Files.isRegularFile), то оставим просто вызов addNewZipEntry
5.3. Если source является директорией (для проверки используй Files.isDirectory), то:
5.3.1. Создай объект класса файловый менеджер FileManager, в конструктор передадим source
5.3.2. Получи список файлов у файлового менеджера, сохраним его в переменную fileNames
5.3.3. Для всех элементов fileNames, вызови метод addNewZipEntry(zipOutputStream, source, fileName)
5.4. Если source не является ни папкой, ни файлом, то кинь исключение PathIsNotFoundException.

Требования:

В классе ZipFileManager добавь приватный метод void addNewZipEntry(ZipOutputStream zipOutputStream, Path filePath, Path fileName) throws Exception.
Метод addNewZipEntry должен быть реализован следуя описанию в задании.
Замени часть кода метода createZip вызовом нового метода addNewZipEntry.
В классе ZipFileManager добавь приватный метод void copyData(InputStream in, OutputStream out) throws Exception.
Метод copyData должен читать данные из in и записывать в out, пока не вычитает все байты.
Замени часть кода метода addNewZipEntry вызовом нового метода copyData.
Метод createZip в классе 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()]; } }

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

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

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

Виталик

Уровень 41

18 февраля 2019, 13:38

решение

for (Path path : fileNames) { addNewZipEntry(zos, sourse, path); Попробуй так.

rotarru

Уровень 34

19 февраля 2019, 03:04

Да. Так прошло. Сварганил такой класс для понимания:

class Solution2 {
    static Path rootPath = Paths.get("D:\\Java\\");;
    static List<Path> fileList = new ArrayList<>();
    static {
        try {
            collectFileList(rootPath);
        } catch (IOException e) {
            e.printStackTrace();
        }
    }

    public static void main(String[] args) {
        for(Path path:fileList){
            Path needPath1 = rootPath.resolve(path);
            Path needPath2 = rootPath.resolve(path.getFileName());
            System.out.println(path);
            System.out.println(path.getFileName());
            System.out.println(needPath1);
            System.out.println(needPath2);
            System.out.println("*****************************");
        }
    }

    private static void collectFileList(Path path) throws IOException {
        // Добавляем только файлы
        if (Files.isRegularFile(path)) {
            Path relativePath = rootPath.relativize(path);
            fileList.add(relativePath);
        }

        // Добавляем содержимое директории
        if (Files.isDirectory(path)) {
            // Рекурсивно проходимся по всему содержмому директории
            // Чтобы не писать код по вызову close для DirectoryStream, обернем вызов newDirectoryStream в try-with-resources
            try (DirectoryStream<Path> directoryStream = Files.newDirectoryStream(path)) {
                for (Path file : directoryStream) {
                    collectFileList(file);
                }
            }
        }
    }
}

rotarru

Уровень 34

19 февраля 2019, 03:04

Он даёт такой вывод:

path:                      JavaRushTasks\out\production\3.JavaMultithreading\com\javarush\task\task31\task3110\command\ZipExtractCommand.class
path.getFileName():        ZipExtractCommand.class
.
resolve path:              D:\Java\JavaRushTasks\out\production\3.JavaMultithreading\com\javarush\task\task31\task3110\command\ZipExtractCommand.class
resolve path.getFileName():D:\Java\ZipExtractCommand.class
*****************************
path:                      JavaRushTasks\out\production\3.JavaMultithreading\com\javarush\task\task31\task3110\command\ZipRemoveCommand.class
path.getFileName():        ZipRemoveCommand.class
.
resolve path:              D:\Java\JavaRushTasks\out\production\3.JavaMultithreading\com\javarush\task\task31\task3110\command\ZipRemoveCommand.class
resolve path.getFileName():D:\Java\ZipRemoveCommand.class
*****************************
path:                      JavaRushTasks\out\production\3.JavaMultithreading\com\javarush\task\task31\task3110\CommandExecutor.class
path.getFileName():        CommandExecutor.class
.
resolve path:              D:\Java\JavaRushTasks\out\production\3.JavaMultithreading\com\javarush\task\task31\task3110\CommandExecutor.class
resolve path.getFileName():D:\Java\CommandExecutor.class
*****************************
path:                      JavaRushTasks\out\production\3.JavaMultithreading\com\javarush\task\task31\task3110\ConsoleHelper.class
path.getFileName():        ConsoleHelper.class
.
resolve path:              D:\Java\JavaRushTasks\out\production\3.JavaMultithreading\com\javarush\task\task31\task3110\ConsoleHelper.class
resolve path.getFileName():D:\Java\ConsoleHelper.class
*****************************

rotarru

Уровень 34

19 февраля 2019, 03:06

Я задачу понимал так, что все архивируемые файлы мы вытаскиваем в одну папку, а подпадки, в которых лежат файлы, и которых может быть много, игнорируем. Но по коду, который проходит валидацию, мы тащим в архив все подпапки. Надо будет в конце задачи посмотреть, как оно в итоге работает. PS Хотя нет. Это я вообще не в туда.

Path path = filePath.resolve(fileName);
        try (InputStream inputStream = Files.newInputStream(path)) {

Этот path мы передаём в инпутстрим, где при path.getFileName() передастся неправильный путь. PPS Да, в архиве все подпапки сохраняются.