.cbzアーカイブから画像を読み取る

Question

.CBZアーカイブ内の画像を読み取り、それらをArrayList内に保存したいと考えています。私は以下の解決策を試しましたが、少なくとも2つの問題があります。

1. 私は、ArrayListの内の画像を得る代わりに、一時ファイルにそれらを書き込み、それを再度読んでのより良い方法があるに違いありませんArrayListの
2. に10-15画像を追加した後のOutOfMemoryエラーが発生します。

---

public class CBZHandler { 
final int BUFFER = 2048; 
ArrayList<BufferedImage> images = new ArrayList<BufferedImage>(); 

public void extractCBZ(ZipInputStream tis) throws IOException{ 
    ZipEntry entry; 
    BufferedOutputStream dest = null; 
    if(!images.isEmpty()) 
     images.clear(); 
    while((entry = tis.getNextEntry()) != null){ 
     System.out.println("Extracting " + entry.getName()); 
     int count; 
     FileOutputStream fos = new FileOutputStream("temp"); 
     dest = new BufferedOutputStream(fos,BUFFER); 
     byte data[] = new byte[BUFFER]; 
     while ((count = tis.read(data, 0, BUFFER)) != -1) { 
       dest.write(data, 0, count); 
      } 
     dest.flush(); 
     dest.close(); 
     BufferedImage img = ImageIO.read(new FileInputStream("temp")); 
     images.add(img); 
    } 
    tis.close(); 
} 
} 

Answer 1

「OutOfMemoryErrorが」よく、またはあなたがメモリに格納しようとしているデータの量に固有ではないかもしれません。最大ヒープサイズを変更する必要があるかもしれません。しかし、確実にディスクへの書き込みを避けることができます。代わりにByteArrayOutputStreamに書き込むだけで、データをバイト配列として取得することができます。必要に応じてByteArrayInputStreamを作成する可能性があります。それぞれをbyte[]として保存するのではなく、BufferedImageとしてリストに追加する必要がありますか？あなたがGuavaを使用することができるしている場合、それは少し非常に簡単「InputStreamから抽出データ」を作ること

注：

byte[] data = ByteStreams.toByteArray(tis); 

Answer 2

は各BufferedImageは、典型的には、それがあるからbyte[]よりも有意に多くのメモリが必要になります構築された。 byte[]をキャッシュし、必要に応じてそれぞれの画像にスタンプする。