Quelle est la meilleure façon de fermer un ByteArrayOutputStream?

Question

J'ai besoin d'optimiser une application qui utilise trop de mémoire de tas. Je rencontre un problème lors de la fermeture d'une variable ByteArrayOutputStream après l'avoir utilisée. J'ai essayé de faire en utilisant close() mais ça ne marche pas. c'est le code:

ByteArrayOutputStream zipOutTempStream = new ByteArrayOutputStream(); 
//arquivo.getZipStream() has the XML received by FTP. 
//STreamEtils is the function who transfers the XML to zipOutTempStream 
StreamUtils.copiarStream(arquivo.getZipStream(), zipOutTempStream); 

      //Creating a new XML to write over this. 
      File arquivo1 = new File("C:/XML.xml"); 
      if (arquivo1.exists()) { 
       System.out.println("ele existe"); 
      } else { 
       if (arquivo1.createNewFile()) { 
        System.out.println("arquivo criado"); 
       } else { 
        System.out.println("arquivo não criado"); 
       } 
      } 

      FileOutputStream arquivo2 = new FileOutputStream(arquivo1); 
      //Copy the unziped XML to the new xml created. 
      StreamUtils.copiarStream(StreamUtils    .uncompressXmlFromZipStream(new ByteArrayInputStream(zipOutTempStream.toByteArray())), arquivo2); 
      arquivo.setZipStream(null); 
      arquivo.setXmlStream(null)  
return arquivo; 

Answer 1

Vous ne pouvez pas fermer une ByteArrayOutputStream, car il est la méthode close() est documentée comme

Fermeture d'une ByteArrayOutputStream n'a pas d'effet. Les méthodes de cette classe peuvent être appelées après que le flux a été fermé sans générant une IOException.

Ce flux de sortie est soutenu par un tableau; c'est PAS un flux tamponné. Si vous pensez qu'il utilise trop de mémoire, vous devez générer des octets directement sur un point de terminaison, tel qu'un fichier ou une socket, en utilisant un OutputStream approprié.

Answer 2

Je pense que vous utilisez trop peu de mémoire. close() n'a rien à voir avec ça. En fait, il n'est pas nécessaire de fermer ByteArrayOutputStream. Ici, vous copiez le fichier ZIP dans enveloppées tableau byte[]:

ByteArrayOutputStream zipOutTempStream = new ByteArrayOutputStream(); 
StreamUtils.copiarStream(arquivo.getZipStream(), zipOutTempStream); 

et quelques lignes vous convertir plus tard le tableau byte[] Retour à InputStream:

StreamUtils.copiarStream(StreamUtils.uncompressXmlFromZipStream(
    new ByteArrayInputStream(zipOutTempStream.toByteArray()) 
), arquivo2); 

On dirait que ce produit byte[] tableau est assez énorme (confirmer avec journalisation). Au lieu de stocker le fichier ZIP entier en mémoire (dans byte[]), stockez-le dans un fichier temporaire et relisez-le.