JavaRush/Помощь по Java задачам/нужна помощь

40 уровень

10.12.2019
834просмотров
8комментариев

нужна помощь

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

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) { 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)

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

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

hidden #1403413

Уровень 41

11 декабря 2019, 15:49

По третьей ошибке, у тебя метод createZip() и addNewEntryZip() делают одно и тоже, обрати внимание на требование, что нужно сделать. Последняя ошибка из-за этого же.

Kirill Kushnaryov

Уровень 40

12 декабря 2019, 03:41

А можно поподробнее объяснить где именно они одно и то же делают? Я эти требования уже раз 30 перечитал и перепроверил всё, сравнил мой вариант со всеми вариантами из раздела "помощь", попробовал попереправить всякую мелочугу. Всё без изменений. Так что я немного в отчаянии, и мне не помешало бы побольше конкретики)

hidden #1403413

Уровень 41

12 декабря 2019, 07:31

Ты в методе createZip() создаешь поток вывода в файл и в addNewEntryZip() ты делаешь тоже самое, зачем тебе дублировать функционал, когда ты из метода createZip() можешь вызвать просто addNewEntryZip() этого и хочет валидатор.

Kirill Kushnaryov

Уровень 40

12 декабря 2019, 15:15

мы с тобой как будто в разный код смотрим... В методе createZip() я создаю ZipOutputStream и вызываю addNewEntryZip(ZipOutputStream,path,path), в котором я создаю InputStream. В первом запись, во втором чтение... Где именно дубль кода?

hidden #1403413

Уровень 41

13 декабря 2019, 08:06

решение

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

!Files.exists(zipFile.getParent())

Использовать:

Files.notExists(zipFile.getParent())

Вместо:

int count;
        byte[] buff = new byte[1024*10];
        while ((count = in.read(buff)) != -1) {
            out.write(buff, 0 , count);
        }
        out.flush();

Сделай так:

byte[] buffer = new byte[8 * 1024];
        int len;
        while ((len = in.read(buffer)) > 0) {
            out.write(buffer, 0, len);
        }

И у тебя кстати, метод createZip() приватный, а должен быть public.

Kirill Kushnaryov

Уровень 40

13 декабря 2019, 08:19

Просто эпик фейл какой-то) И у тебя кстати, метод createZip() приватный, а должен быть public. всё было в этом) Спасибо)

hidden #1403413

Уровень 41

13 декабря 2019, 08:21

Бывает, сам не раз тупил над очевидными вещами)

Kirill Kushnaryov

Уровень 40

10 декабря 2019, 16:57

добавил zipOutputStream.closeEntry(); в add метод вторая ошибка ушла. Остальные остались