тр...сь, простите, третий день

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

Новый

траухажуюсь. код полностью рабочий. - не использует файловую систему - в случае, если файл имя файла совпадает с именем файла в папке new - файл заменяется, в случае если файла с таким именем нет - новый файл добавляется посмотрел комменты, решение не смотрел. у меня вопрос, неужели валидатору не нарвится мой класс Pair и он категорически желает видеть мапу? или у меня есть ошибки в коде? PS то, что окошечный архиватор создаёт битые архивы - я понял, написал класс, который создаёт и заполняет тестовый вариант архива. приложу ее ниже

В метод 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.
Потоки для работы с архивом должны быть закрыты.

package com.javarush.task.task31.task3105; import java.io.FileInputStream; import java.io.FileOutputStream; import java.io.IOException; import java.nio.file.Path; import java.nio.file.Paths; import java.util.ArrayList; import java.util.List; import java.util.zip.ZipEntry; import java.util.zip.ZipInputStream; import java.util.zip.ZipOutputStream; /* Добавление файла в архив */ public class Solution { private static final List<Pair> savedEntries = new ArrayList<>(); public static void main(String[] args) throws IOException { Path fileName = Paths.get(args[0]); Path oldZipArchive = Paths.get(args[1]); String newDerictoryInArchive = "new/"; String absolutePathOfResultFile = newDerictoryInArchive + fileName.getFileName(); FileInputStream fis = new FileInputStream(fileName.toFile()); boolean flag = false; try (ZipInputStream zis = new ZipInputStream(new FileInputStream(oldZipArchive.toFile()))) { ZipEntry currentEntry = null; int currentByte = 0; Pair pair = null; while ((currentEntry = zis.getNextEntry()) != null) { List<Integer> currentContent = new ArrayList<>(); if (!currentEntry.isDirectory()) { if (currentEntry.getName().equals(absolutePathOfResultFile)) { while ((currentByte = fis.read()) != -1) currentContent.add(currentByte); flag = true; } else { while ((currentByte = zis.read()) != -1) currentContent.add(currentByte); } } pair = new Pair(currentEntry.getName(), currentEntry.isDirectory() ? null : currentContent); savedEntries.add(pair); zis.closeEntry(); } } try (ZipOutputStream zos = new ZipOutputStream(new FileOutputStream(oldZipArchive.toFile()))) { for(Pair entry : savedEntries) { zos.putNextEntry(new ZipEntry(entry.getZipEntryName())); if (entry.getEntryContent() != null) { for (Integer currentByte : entry.getEntryContent()) { if (currentByte < 0 || currentByte > 255) { throw new IllegalArgumentException("Element is not a byte value"); } zos.write(currentByte); } } } if(!flag) { zos.putNextEntry(new ZipEntry(absolutePathOfResultFile)); int currentByte = 0; while ((currentByte = fis.read()) != -1) { zos.write(currentByte); } zos.closeEntry(); } } } }

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

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

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

Денис Кокшаров

Уровень 1

1 апреля 2025, 06:12

так, значит с частью проблем я разобрался. следующие сообщения будут для будущих поколений. валидатору не нравится то, что я использую для создания FileInputStream метод Paths.get(args[1]).toFile(). тут видимо проверка чисто синтаксическая ибо абсолютно не используется файловая система при тако создании FileInputStream. спокойно проходит Paths.get(args[1]).toString или совсем просто сразу строка args[1]

Денис Кокшаров

Уровень 1

31 марта 2025, 16:45

public class Solution {

    public static void main(String[] args) throws IOException {
        Map<ZipEntry, ByteArrayOutputStream> entries = new HashMap<>();
        Path fileName = Paths.get(args[0]);
        Path oldZipArchive = Paths.get(args[1]);
        String newDerictoryInArchive = "new/";
        String absolutePathOfResultFile = newDerictoryInArchive + fileName.getFileName();

        try (ZipInputStream zis = new ZipInputStream(new FileInputStream(oldZipArchive.toFile()))) {
            ZipEntry currentEntry = null;
            while ((currentEntry = zis.getNextEntry()) != null) {
                ByteArrayOutputStream baos = new ByteArrayOutputStream();
                int currentByte = 0;
                while ((currentByte = zis.read()) != -1) {
                    baos.write(currentByte);
                }
                entries.put(new ZipEntry(currentEntry.getName()), baos);
                zis.closeEntry();
            }
        }

        try (ZipOutputStream zos = new ZipOutputStream(new FileOutputStream(oldZipArchive.toFile()))) {
            for (Map.Entry<ZipEntry, ByteArrayOutputStream> entry : entries.entrySet()) {
                zos.putNextEntry(entry.getKey());
                if (!entries.containsKey(absolutePathOfResultFile)) {
                    zos.write(entry.getValue().toByteArray());
                } else {
                    if (entry.getKey().getName().equals(absolutePathOfResultFile)) {
                        Files.copy(fileName, zos);
                    } else {
                        zos.write(entry.getValue().toByteArray());
                    }
                }
                zos.closeEntry();
            }

            zos.putNextEntry(new ZipEntry(absolutePathOfResultFile));
            Files.copy(fileName, zos);
            zos.closeEntry();
        }
    }
}

Денис Кокшаров

Уровень 1

31 марта 2025, 16:45

кароч, переписал, всё работает и в таком варианте, валидатор ставит везде кресты, шо я делаю не так - не понимаю

Денис Кокшаров

Уровень 1

31 марта 2025, 14:11

/*
/ this class is a creator and filler of ZIP Archive
*/

import java.io.FileOutputStream;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.zip.ZipEntry;
import java.util.zip.ZipOutputStream;

public class Solution {

    public static void main(String[] args) throws IOException {
        String first = "FIRST";
        String name1 = "1.txt";
        String second = "second";
        String name2 = "2.txt";
        String directoryName = "new/";
        String fileName = "result.txt";
        Path zipArchiveName = Paths.get("D:\\study\\pathToZip\\archive.zip");

        Files.deleteIfExists(zipArchiveName);
        try (ZipOutputStream zos = new ZipOutputStream(new FileOutputStream(zipArchiveName.toFile()))) {
            zos.putNextEntry(new ZipEntry(directoryName));
            zos.closeEntry();
            zos.putNextEntry(new ZipEntry(directoryName + fileName));
            zos.write(first.getBytes());
            zos.closeEntry();
            zos.putNextEntry(new ZipEntry(name1));
            zos.write(first.getBytes());
            zos.putNextEntry(new ZipEntry(name2));
            zos.write(second.getBytes());
            zos.closeEntry();
        }
    }
}