[debutant] conversion byte[] -> String , avec StringBuilder

Version imprimable

09/07/2006, 11h37
absolut75

[debutant] conversion byte[] -> String , avec StringBuilder
Bonjour,

Je m'y perds un peu avec les Stream, les Reader, les Buffered ...etc ;-) mais ça commence à venir.

Comme je l'ai appris grâce à ce forum, il est nettement mieux de passer par StringBuilder que de concaténer des String...

Alors voilà :

je recois des paquets de byte[] de taille 1000, et l'ensemble de ces paquets represente une String.

Donc on peut faire çà :
Code:

1 2 3 4 5 6 7 String tmp; String s=""; while (il existe un paquet : byte[] paquet ) { tmp=new String(paq); s=s+tmp; }
Mais je n'arrive pas à réécrire ça avec des StringBuilder....si quelqu'un peut m'indiquer la soluce :-)

Merci, a+
09/07/2006, 12h05
Pragmateek

Pour construire un "StringBuilder" à partir d'un tableau de "byte":

Code:

StringBuilder sb=new StringBuilder(new String(paquet));

Pour concaténer:

Code:

sb.append(new String(paquetSuivant));

Et voilà pour info le code correspondant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
 
InputStream is=get.getResponseBodyAsStream();
BufferedInputStream bis=new BufferedInputStream(is);
ZipInputStream zis = new ZipInputStream(bis);
 
//Décompression du zip vers tmp.txt
zis.getNextEntry();
int BUFFER=1000;
int count;
byte data[] = new byte[BUFFER];
 
String rep=System.getProperty("user.dir");
String sep=System.getProperty("file.separator");
FileOutputStream fos = new FileOutputStream(rep+sep+"tmp.txt");
BufferedOutputStream bos = new BufferedOutputStream(fos, BUFFER);
 
while ((count = zis.read(data, 0, BUFFER)) != -1) {
	bos.write(data, 0, count);
}
bos.close();
zis.close();
fos.close();
get.releaseConnection();
 
 
// Import du .txt en String
BufferedReader in = new BufferedReader(new FileReader(rep+sep+"tmp.txt"));
StringBuilder buffer=new StringBuilder();
String s = new String();
while((s = in.readLine())!= null)
	buffer.append(s).append(nl);
in.close();
 
 
//Effacement du fichier tmp.txt
File  tmptxt;
tmptxt=new File(rep+sep+"tmp.txt");
tmptxt.delete();
 
 
String resultatVoulu=buffer.toString();

Le but de ma question étant de supprimer le passage par un fichier temporaire (tmp.txt).

Si les pros ont des commentaires à faire sur le code, qu'ils n'hésitent pas : c'est encore un peu confus.

Merci

09/07/2006, 12h16
absolut75

merci de ta réponse seriousme :-)

Mais est-ce que le fait d'instancier une String pour chaque paquet ne revient pas à concatener des String comme je le fais ?

Code:

1 2 3 4 5 6 7 8 byte data[] = new byte[BUFFER]; StringBuffer buffer = new StringBuffer(); while ((count = zis.read(data, 0, BUFFER)) != -1) { buffer.append(new String(data)); } return buffer.toString();
Niveau mémoire c'est surement pas la meilleure chose à faire, mais je pense que cette solution fonctionne. Après si quelqu'un a mieux, n'hésitez pas !

09/07/2006, 12h25
Pragmateek

Citation:

Mais est-ce que le fait d'instancier une String pour chaque paquet ne revient pas à concatener des String comme je le fais ?

Oui mais au début tu parlais de séquences de "byte", or la méthode "append" de la classe "StringBuilder" n'est pas surchargée pour les séquences de "byte" d'où le passage par le constructeur de la classe "String" qui les accepte et renvoie une "String"; mais comme la méthode "readLine" renvoie une "String" directement pas besoin de le faire.
09/07/2006, 12h27
Pragmateek

Code:

StringBuffer buffer = new StringBuffer();

Si on n'utilise pas le "threadsafety" de la classe "StringBuffer" mieux vaut utiliser des "StringBuilder" plus performants.
09/07/2006, 12h37
absolut75

Ok merci pour vos réponses,

dbprog, j'ai effectivement essayé cette solution après les indications de seriousme ; (en utilisant StringBuilder).

Alors ça marche, mais à moitié ;-). J'intuite peut être un problème de charSet...
Ce qui est curieux, c'est que mon code (celui qui passe par un tmp.txt) marche très bien et il n'y a aucune référence aux charSet....

J'investigue. (je plane)
09/07/2006, 12h58
Pragmateek

Si c'est un problème de charset il y a la classe "CharsetDecoder".
09/07/2006, 16h30
DBProg

Citation:

Envoyé par seriousme

Si on n'utilise pas le "threadsafety" de la classe "StringBuffer" mieux vaut utiliser des "StringBuilder" plus performants.

Tu peux m'expliquer un petit peu stp ? Qu'est-ce que tu entends par threadsafety ?

Citation:

Envoyé par absolut75

Alors ça marche, mais à moitié ;-). J'intuite peut être un problème de charSet...

Montre nous ton problème (code + sortie) que l'on comprenne ce qu'il ne va pas ;)
09/07/2006, 16h44
Pragmateek

Citation:

Tu peux m'expliquer un petit peu stp ? Qu'est-ce que tu entends par threadsafety ?

En contexte multi-thread les "StringBuilder" ne sont pas sûr car des threads peuvent accéder aux données de manière désordonnée ce qui conduit à un comportement indéterminé.
En revanche les "StringBuffer" sont synchronisés empêchant que plusieurs threads n'accèdent en même temps à la même donnée.
09/07/2006, 16h48
afrikha

Citation:

Tu peux m'expliquer un petit peu stp ? Qu'est-ce que tu entends par threadsafety ?

un StringBuffer peut étre manipuler par plusieurs threads sans que cela entraine une corruption des données (pas de synconisation à faire ), et cela implique une perte de performance face à StringBuilder qui elle n'est pas thread-safe .

Voilà, j'éspère que c'est suffisamment clair, sinon n'hésite pas à poser des questions ;)

[EDIT] grillé :aie:
09/07/2006, 16h51
DBProg

Merci pour l'info, je n'avais jamais fais attention à cette suptilité :oops:
10/07/2006, 09h51
absolut75
Citation:

Montre nous ton problème (code + sortie) que l'on comprenne ce qu'il ne va pas

Salut !

Ben le code je l'ai mis un peu plus haut (version ok en passant par un fichier tmp.txt).
Pour la version qui bugue, je remplace :
Code:

1 2 3 4 while ((count = zis.read(data, 0, BUFFER)) != -1) { bos.write(data, 0, count); }
par
Code:

1 2 3 4 while ((count = zis.read(data, 0, BUFFER)) != -1) { buffer.append(new String(data)); }
(et suppression de tout ce qui est relatif au .txt)

Pour le resultat, j'ecris dans deux fichier les deux String obtenues. Et celui de la deuxième version est "légèrement corrompu" (malheureusement je ne peux pas vous les transmettre).

Si j'ouvre ces deux fichiers en Hexadecimal, un premier indice est que dans la version ok, mes sauts de lignes sont codés par "0D 0A" tandis que dans l'autre c'est juste "0A" ... et une ou deux autres bizarreries ....

Je suis perdu, help ;-)
10/07/2006, 10h07
dude

Déjà, avec le StringBuffer, tu risques d'avoir des problèmes en fin de stream :
lors du dernioer passage dans read, si count < BUFFER, tous les bytes compris entre les index count et BUFFER ne sont pas écrasés.
Avec le bos, tu n'écriras que les n premiers bytes de ton array (entre 0 et count) fraichement lus, tandis qu'avec le StringBuffer tu écriras le tableau dans son intégralité (les n premiers + ceux entre count et BUFFER qui datent de la lecture précédente)

j'espere avoir ete clair...

[edit] j'ai lu le stream un peu en travers, mais pourquoi ne pas utiliser un StringWriter ?
[re-edit] d'ailleurs pour manipuler du texte, il est recommandé d'utiliser les Writer plutot que les OutputStream
10/07/2006, 11h27
absolut75

Je crois que tu as mis le doigt sur un des soucis....en effet, j'ai des series de bytes qui se retrouvent "dans le désordre". J'ai pas tout saisi, mais je vais tacher d'apronfondir.

En ce qui concerne l'utilisation de Stream, ben en fait j'utilise ce que j'ai pu trouver dans le package ZIP de la JDK...C'est de la bricole, car je comprends pas encore bien tout ce fourbi (stream, writer, reader, buffered et les autres).

En gros le prog lance une requete https (httpClient jakarta) sur un site qui renvoie un .zip, que je veux decompresser "à la volée" et mettre le resultat dans une String. En passant par un .txt sur le dur ça marche...
10/07/2006, 11h43
afrikha

J'éspère que ce tutoriel sur java.io pourra t'aider ;)

normalement, ceci devrait marcher...
je ne vois pas ou est le probleme

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
....
 
ByteArrayOutputStream baos = new ByteArrayOutputStream();
BufferedOutputStream bos = new BufferedOutputStream(baos, BUFFER);
while ((count = zis.read(data, 0, BUFFER)) != -1) {
   bos.write(data, 0, count);
}
bos.flush();
System.out.println(baos.toString());
 
....

11/07/2006, 17h33
absolut75

Un grand merci à vous !

dude, effectivement ça marche :-) . Tu ne voyais pas le problème, je ne voyais pas la solution.

merci aussi pour le tuto, bien utile.

A+
11/07/2006, 17h41
absolut75

en fait, je crois que c'est le flush() qui change tout ;-)
11/07/2006, 17h43
dude

pas forcement : il est appelé automatiquement par le close()