Discussion :

[Mides]

Bonsoir,

comment peut on faire pour qu'une méthode retourne un ZipInputStream ?

J'ai bien essayé différents codes mais sans succès, par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
OutputStream out = new ByteArrayOutputStream();	
XMLOutputter sortie = new   XMLOutputter(Format.getPrettyFormat());
ZipOutputStream zipStream = new ZipOutputStream(out);
zipStream.putNextEntry(new ZipEntry("test.xml"));
sortie.output(xml, zipStream);
zipStream.close();
ZipInputStream in = new ByteArrayInputStream(((ByteArrayOutputStream) zipStream).toByteArray());		
return in;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

java.io.ByteArrayInputStream cannot be cast to java.util.zip.ZipInputStream

[tchize_]

on crée un ZipInputstream en utilisant le constructeur ZipInputstream, tout simplement. Tes casts n'ont ni queue ni tête. Revoie ton code, tu ne devrais de toutes façons pas avoir beosin de faire des casts.

Un cast, pour rappel, ça ne transforme pas un objet, il n'y a donc pas de raison pour que ByteArrayInputStream soit un ZipInputstream ou que zipStream soit un ByteArrayOutputStream

[Mides]

...et je suis entièrement d'accord avec toi.

D'où le problème d'arriver à faire un ZipInputstream en sortie pour que mon appel de méthode puisse ressembler à cela :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

ZipInputStream newXmlZip = pqZip.xmlZip(newXml);

Si tu as une façon de faire, je suis preneur.

[tchize_]

new ZipInputstream, comme déjà mentionné

[Mides]

Cela ne ressemblerait pas à un "truc" de ce style.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
public class OutputToInput {
	public static void main(String[] args) throws IOException {
		String txt = "ceci est un test";
		ByteArrayOutputStream output = new ByteArrayOutputStream();
		byte buf[] = txt.getBytes();
		output.write(buf);
		InputStream result = new ByteArrayInputStream(output.toByteArray());
		BufferedReader input = new BufferedReader(new InputStreamReader(result));
		String line = "";
		while ((line = input.readLine()) != null) {
			System.out.println(line);
		}
		output.close();
		input.close();
	}
}

[thelvin]

Cela ne ressemblerait pas à un "truc" de ce style.

À la limite t'as pas l'air d'avoir besoin du BufferedReader de cet exemple

Mais l'utilisation des InputStream et OutputStream, c'est le même principe, oui. Il faut juste ajouter la couche de zippage dans les deux sens.

[Mides]

Bon, je pense que ce code devrait faire l'affaire.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public class ReturnInputStream {
 
	public static InputStream zip(Document xml) throws IOException {
		ByteArrayOutputStream output = new ByteArrayOutputStream();
		XMLOutputter sortie = new XMLOutputter(Format.getPrettyFormat());
		ZipOutputStream zipStream = new ZipOutputStream(output);
		zipStream.putNextEntry(new ZipEntry("test.xml"));
		sortie.output(xml, zipStream);
		InputStream resultInputStream = new ByteArrayInputStream(output.toByteArray());
		zipStream.close();
		output.close();
		return resultInputStream;
	}
 
	public static void main(String[] args) throws IOException, JDOMException {
		String xml = "<root><data>Ceci est un test</data></root>";
		SAXBuilder sxb = new SAXBuilder();
		Document document = sxb.build(new StringReader(xml));
		InputStream result = zip(document);
	}
}

[thelvin]

En toute logique 'faudrait fermer zipStream avant d'appeler toByteArray().
Mais bon, si je ne me trompe pas, avec le format zip ça marchera quand même.

[Mides]

Merci pour ce conseil.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
zipStream.close();
InputStream resultInputStream = new ByteArrayInputStream(output.toByteArray());

Par contre, j'en profite pour demander un autre conseil : Concernant le resultInputStream, à quel moment dois je le fermer ?

[thelvin]

Un InputStream doit être fermé quand on a fini de le lire, c'est-à-dire :
- quand on est arrivé à la fin
OU
- quand on a reçu une erreur et qu'on ne peut plus le lire.

Concrètement, en Java 7, il faut donc le laisser se fermer implicitement avec un try-with-resources :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
try(InputStream stream = construireLeStream()) {
  // lire le stream;
}
// on a fini. Sortir du try appelle close() implicitement.

Et avant Java 7, l'appeler dans un finally :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
InputStream stream = construireLeStream();
try {
  // lire le stream;
} finally {
  stream.close();
}

Mais bon, tout ça c'est pour le cas général.
Un ByteArrayInputStream n'a pas besoin d'être fermé, puisqu'il n'a jamais rien ouvert.

[Mides]

Je vois un peu mieux la "mécanique" pour la fermeture des flux, par contre je découvre la syntaxe pour Java 7.

[Mides]

En toute logique 'faudrait fermer zipStream avant d'appeler toByteArray().
Mais bon, si je ne me trompe pas, avec le format zip ça marchera quand même.

Bien vu, cela ne fonctionne pas si zipStream n'est pas fermé avant toByteArray(). Tout le flux n'est pas écrit dans le resultInputStream.