JavaRush/Помощь по Java задачам/Бред какой-то

22 уровень

06.10.2020
429просмотров
11комментариев

Бред какой-то

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

Обсуждается

Говорит, что долго работает, но я старался следовать рекомендациям с SequenceInputStream и прочими буферами. Подскажите, где проглядел.

В метод main приходит список аргументов.
Первый аргумент - имя результирующего файла resultFileName, остальные аргументы - имена файлов fileNamePart.
Каждый файл (fileNamePart) - это кусочек zip архива. Нужно разархивировать целый файл, собрав его из кусочков.
Записать разархивированный файл в resultFileName.
Архив внутри может содержать файл большой длины, например, 50Mb.
Внутри архива может содержаться файл с любым именем.

Пример входных данных. Внутри архива находится один файл с именем abc.mp3:
C:/result.mp3 C:/pathToTest/test.zip.003 C:/pathToTest/test.zip.001 C:/pathToTest/test.zip.004 C:/pathToTest/test.zip.002

Требования:

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

package com.javarush.task.task31.task3106; import java.io.*; import java.util.ArrayList; import java.util.Arrays; import java.util.Collections; import java.util.List; import java.util.zip.ZipInputStream; /* Разархивируем файл */ public class Solution { public static void main(String[] args) throws IOException { File resultFile = new File(args[0]); BufferedWriter resultWriter = new BufferedWriter(new FileWriter(resultFile)); Arrays.sort(args, 1, args.length - 1); List<InputStream> inputStreams = new ArrayList<>(); for (int i = 1; i < args.length; i++) { inputStreams.add(new FileInputStream(args[i])); } SequenceInputStream seqIn = new SequenceInputStream(Collections.enumeration(inputStreams)); ZipInputStream zipIn = new ZipInputStream(seqIn); while (zipIn.available() == 1) { byte[] buffer = new byte[1000]; int data = zipIn.read(buffer); resultWriter.write(data); resultWriter.flush(); } resultWriter.close(); } }

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

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

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

Ksenia VolkovaJava Developer в DXC

Master

7 октября 2020, 06:56

int data = zipIn.read(buffer);
resultWriter.write(data);

И что тут у тебя происходить должно? И здесь ещё ошибка

Arrays.sort(args, 1, args.length - 1);

HackerPro

Уровень 22

7 октября 2020, 08:00

В первых двух строках я надеюсь считывать байты из ZipInputStream в buffer. Предполагаю, что я в resultWriter должен передать buffer, но не понимаю, почему ошибка с сортировкой. Почему?

HackerPro

Уровень 22

7 октября 2020, 08:19

Переделал, но всё равно долго.

public class Solution {
    public static void main(String[] args) throws IOException {
        String destinationFileName = args[0];

        byte[] buffer = new byte[2048];

        ByteArrayInputStream bufferInStream = new ByteArrayInputStream(buffer);

        BufferedWriter resultWriter = new BufferedWriter(new PrintWriter(destinationFileName));

        String[] zipFileNames = Arrays.copyOfRange(args, 1, args.length - 1);
        Arrays.sort(zipFileNames);

        List<FileInputStream> inputStreams = new ArrayList<>();

        for (String zipFileName : zipFileNames) {
            inputStreams.add(new FileInputStream(zipFileName));
        }

        SequenceInputStream sequenceOfInputStreams = new SequenceInputStream(Collections.enumeration(inputStreams));
        BufferedInputStream buffInputStream = new BufferedInputStream(sequenceOfInputStreams);
        ZipInputStream zipInputStream = new ZipInputStream(buffInputStream);

        while (zipInputStream.available() == 1) {
            zipInputStream.read(buffer);
            resultWriter.write(bufferInStream.read(buffer, 0, buffer.length - 1));
        }

        zipInputStream.close();
        bufferInStream.close();
        resultWriter.close();
    }
}

Ksenia VolkovaJava Developer в DXC

Master

7 октября 2020, 08:43

полезный

И в sort, и в copyOfRange ты не учитываешь, что indexTo (последний параметр) - не включается в диапазон. С записью тоже по-прежнему проблемы. Даже больше. Вот эта часть просто безумная какая-то ))

zipInputStream.read(buffer);
resultWriter.write(bufferInStream.read(buffer, 0, buffer.length - 1));

zipInputStream у тебя читает байты из buffInputStream и сохраняет их в buffer. После чего buffInputStream читает байты из файлов в тот же buffer. А что при этом пишет resultWriter - это ты посмотри сам, изучив в документации, что именно возвращает метод read (с аргументом-массивом).

HackerPro

Уровень 22

8 октября 2020, 06:35

То, что удалось нагуглить: ByteArrayInputStream при чтении возвращает целочисленное представление байта типа int, которые не нуждается в приведении к типу byte. Исходя из этого, zipInputStream читает сколько может, наполняя массив байт, затем bufferInStream записывает все байты, которые, в свою очередь, ByteArrayInputStream считывает из buffer, передавая интовые представления байтов на запись в файл resultWriter'ом. Описанное, приведено в коде ниже:

while (zipInputStream.available() == 1) {
         zipInputStream.read(buffer);
         resultWriter.write(bufferInStream.read());
}

Не могу найти ошибку.

Ksenia VolkovaJava Developer в DXC

Master

8 октября 2020, 07:52

При чем тут вообще ByteArrayInputStream ?? Вопрос был о методе read с аргументом-массивом, который ты использовал, например, тут

zipInputStream.read(buffer);

или тут

bufferInStream.read(buffer, 0, buffer.length - 1)

А еще подумай, как связаны между собой твои zipInputStream и bufferInStream. Откуда читает данные первый, откуда второй? И с учетом этого попробуй обычными словами объяснить, что именно у тебя происходит в куске кода в твоем комментарии.

hidden #2322530

Уровень 41

6 октября 2020, 18:45

while (zipIn.available() == 1) {

это что за условие. считывают до тех пор пока available() > 0

HackerPro

Уровень 22

6 октября 2020, 19:46

https://docs.oracle.com/javase/8/docs/api/java/util/zip/ZipInputStream.html

hidden #2322530

Уровень 41

6 октября 2020, 20:01

не очень помню как работает Collections.enumeration() но у тебя там был список с входящими файлами. ZipUnputStream.Entry и как бы надо с каждого считать по очереди, и записать в результирующий файл.

HackerPro

Уровень 22

7 октября 2020, 08:22

Collections.enumeration() возвращает объект типа Enumeration. Он нужен, чтобы использовать SequenceInputStream. А разве нельзя скопом прочесть весь ZipInputStream?

hidden #2322530

Уровень 41

7 октября 2020, 08:28

полезный

нет.