Бьюсь головой об стену, что валидатор хочет?

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

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

Решен

Скорее всего, что-то очевидное, что я упорно не замечаю.

Пришло время отрефакторить класс 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()]; } }

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

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

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

Jean Valjean

Уровень 41

27 августа 2020, 15:35

решение

в addNewZipEntry получи полный путь:

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

в подсказках к задаче писал также коммент недавно,посмотри, а метод copydata можно улучшить с помощью byte [] buffer если что, после первого уровня Collections решишь на раз-два

hidden #2322530

Уровень 41

28 августа 2020, 09:54

полезный

по поводу copyData метода. на днях был вопрос, там у человека все ок, но метод был описан как у автора этого вопроса. как только он поменял на вариант с использование буфера, валидатор пропустил.

Горох Антон

Уровень 29

28 августа 2020, 12:54

Действительно, проблема была в том, что не получал полный путь. Также была проблема с проверкой и созданием директории, а также в проходе по вложенным файлам. Вот конечный код метода:

public void createZip(Path source) throws Exception {
    if (!Files.isDirectory(zipFile.getParent())) Files.createDirectories(zipFile.getParent());
    try (
            ZipOutputStream out =
                    new ZipOutputStream(Files.newOutputStream(zipFile))
    ) {
        if (Files.isRegularFile(source))
            addNewZipEntry(out, source.getParent(), source.getFileName());

        else if (Files.isDirectory(source)) {
            FileManager fileManager = new FileManager(source);
            for (Path path : fileManager.getFileList()) {
                addNewZipEntry(out, source, path);
            }

        } else {
            throw new PathIsNotFoundException();
        }
    }
}