Не проходит 4 пункт

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

Решен

Всем привет. Прошу подсказать, код работает, но не проходит 4 пункт, в чем может быть ошибка?

В метод main приходит список аргументов.
Первый аргумент - полный путь к файлу fileName.
Второй аргумент - путь к zip-архиву.
Добавить файл (fileName) внутрь архива в директорию 'new'.
Если в архиве есть файл с таким именем, то заменить его.

Пример входных данных:
C:/result.mp3 C:/pathToTest/test.zip

Файлы внутри test.zip:
a.txt b.txt

После запуска Solution.main архив test.zip должен иметь такое содержимое:
new/result.mp3
a.txt b.txt

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

Требования:

В методе main создай ZipInputStream для архивного файла (второй аргумент main). Нужно вычитать из него все содержимое.
В методе main создай ZipOutputStream для архивного файла (второй аргумент main).
В ZipOutputStream нужно записать содержимое файла, который приходит первым аргументом в main.
В ZipOutputStream нужно записать все остальное содержимое, которое было вычитано из ZipInputStream.
Потоки для работы с архивом должны быть закрыты.

111

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

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

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

Natasha Martova

Уровень 41

15 апреля 2021, 15:34

я бы поменяла местами строки 50-54 и 47-49 и еще убрала бы 49 строку совсем

Сергей Backend Developer

15 апреля 2021, 18:57

Поменял, теперь по 3му пункту не проходит.

try (ZipOutputStream zout = new ZipOutputStream(new FileOutputStream(args[1]))) {
    for (Map.Entry<String, ByteArrayOutputStream> entry : filesInArchive.entrySet()) {//Запись файлов из мапы
        zout.putNextEntry(new ZipEntry(entry.getKey()));
        entry.getValue().writeTo(zout);
        //zout.closeEntry();
    }
    zout.putNextEntry(new ZipEntry(newPathInArchive));//Помещаем файл в архив
    Files.copy(Paths.get(args[0]), zout);//Наполняем файл архива
    //zout.closeEntry();//Переходим к следующей записи
} catch (Exception ex) {
    System.out.println(ex.getMessage());
}

Natasha Martova

Уровень 41

16 апреля 2021, 11:08

а сели 4 строку поменять относительно zout: zout.write(entry.getValue().toByteArray()); или как ест, но в конце toByteArray() У меня 1 вариант сработал

Сергей Backend Developer

16 апреля 2021, 14:37

1 вариант с zout.write(entry.getValue().toByteArray()); не сработал, таже ошибка 2 вариант с "ест" не понял

Natasha Martova

Уровень 41

16 апреля 2021, 20:13

решение

а если проверку вставить: File file = new File(args[0]); for (Map.Entry<String, ByteArrayOutputStream> entry : filesInArchive.entrySet()) {//Запись файлов из мапы if (entry.getKey().substring(entry.getKey().lastIndexOf("/") + 1).equals(file.getName())) continue; zout.putNextEntry(new ZipEntry(entry.getKey())); zout.write(entry.getValue().toByteArray()); }

Сергей Backend Developer

17 апреля 2021, 11:52

Прошло, спасибо! В данном случае, зачем такая проверка? Я считаю её избыточной