Оптимизация процесса разархивирования в Java?

Question

Оптимизация процесса разархивирования в Java?

1

У меня есть проблема с временем, требуемым для файлов.gz, чтобы получить несжатый код с помощью следующего кода:

import java.io.*;
import java.util.zip.*;

public class UnGunzipClass{

    public static boolean ungunzip(String compressedFile, String decompressedFile){

        try{

            // in
            FileInputStream fileIn          = new FileInputStream(compressedFile);
            GZIPInputStream gZipIn          = new GZIPInputStream(fileIn);
            BufferedInputStream in          = new BufferedInputStream(gZipIn);

            // out
            FileOutputStream fileOut        = new FileOutputStream(decompressedFile);
            BufferedOutputStream out        = new BufferedOutputStream(fileOut);

            int n = 0;
            int len = 1024*1024*1024;
            byte[] buffer = new byte[len];

            while((n = in.read(buffer,0,len)) > 0){
                out.write(buffer,0,n);
            }

            gZipIn.close();
            fileOut.close();

            return true;

        } catch (IOException e){
            e.printStackTrace();
            return false;
        }
    }
}

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

display_name 23 янв. 2015, в 07:24

Источник

1

Вы действительно нуждаетесь в 1 ГБ буфера памяти (не это должно иметь значение по скорости)?
Thilo 23 янв. 2015, в 05:42
0

n = in.read(buffer,0,n) . Этот последний n кажется неправильным. должен быть buffer.length .
Thilo 23 янв. 2015, в 05:49
0

Это вообще работает? Если вы начнете с n=0 , согласно документации, он не должен читать никаких байтов и выходить из цикла.
Thilo 23 янв. 2015, в 05:50
0

@Тило, ты прав. Должно быть n = in.read(buffer,0,len))
display_name 24 янв. 2015, в 20:07

Показать ещё 2 комментария

Теги:

java

optimization

performance

gunzip

1 ответ

Ещё вопросы

Вы действительно нуждаетесь в 1 ГБ буфера памяти (не это должно иметь значение по скорости)?
n = in.read(buffer,0,n) . Этот последний n кажется неправильным. должен быть buffer.length .
Это вообще работает? Если вы начнете с n=0 , согласно документации, он не должен читать никаких байтов и выходить из цикла.
@Тило, ты прав. Должно быть n = in.read(buffer,0,len))

Elliott Frisch · Accepted Answer · 2015-01-23T03-55-00.000Z

Вы создали BufferedInputStream из GZIPInputStream, для производительности вы бы сделали это в обратном порядке. Кроме того, я предлагаю вам уменьшить размер вашего буфера (и использовать ваши буферизованные потоки). Наконец, я бы использовал приложение try-with-resources которое могло бы выглядеть примерно так:

public static boolean ungunzip(String compressedFile,
        String decompressedFile) {
    final int BUFFER_SIZE = 32768;
    try (InputStream in = new GZIPInputStream(new BufferedInputStream(
            new FileInputStream(compressedFile)), BUFFER_SIZE);
            OutputStream out = new BufferedOutputStream(
                    new FileOutputStream(decompressedFile), BUFFER_SIZE)) {
        int n = 0;
        byte[] buffer = new byte[BUFFER_SIZE];
        while ((n = in.read(buffer, 0, BUFFER_SIZE)) > 0) {
            out.write(buffer, 0, n);
        }
        return true;
    } catch (IOException e) {
        e.printStackTrace();
    }
    return false;
}

Это все еще медленно, но я думаю, что это так же быстро, как и получается :) спасибо!