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

**absolut75** · 09/07/2006, 11h37

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 :

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

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+

**Pragmateek** · 09/07/2006, 12h05

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

Code :

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

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

Pour concaténer:

Code :

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

sb.append(new String(paquetSuivant));

**absolut75** · 09/07/2006, 12h11

Et voilà pour info le code correspondant :

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
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

**absolut75** · 09/07/2006, 12h16

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 ?

**DBProg** · 09/07/2006, 12h22

Code :

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

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 !

**Pragmateek** · 09/07/2006, 12h25

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.

**Pragmateek** · 09/07/2006, 12h27

Code :

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

StringBuffer buffer = new StringBuffer();

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

**absolut75** · 09/07/2006, 12h37

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)

**Pragmateek** · 09/07/2006, 12h58

Si c'est un problème de charset il y a la classe "CharsetDecoder".

**DBProg** · 09/07/2006, 16h30

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 ?

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

**Pragmateek** · 09/07/2006, 16h44

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.

**afrikha** · 09/07/2006, 16h48

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é

**DBProg** · 09/07/2006, 16h51

Merci pour l'info, je n'avais jamais fais attention à cette suptilité

**absolut75** · 10/07/2006, 09h51

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 :

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

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

par

Code :

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

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 ;-)

**dude** · 10/07/2006, 10h07

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

**absolut75** · 10/07/2006, 11h27

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...

**afrikha** · 10/07/2006, 11h43

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

**dude** · 10/07/2006, 12h04

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

Code :

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

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());
 
....

**absolut75** · 11/07/2006, 17h33

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+

**absolut75** · 11/07/2006, 17h41

en fait, je crois que c'est le flush() qui change tout ;-)

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

Langage Java

Discussions similaires

Partager

Partager