Discussion :

[rige47]

Bonjour,

Je test la communication entre plusieurs programmes via des sockets. Et je me pose la question de la nécessité de procéder comme suis pour effectuer une lecture :

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
 
...
 
DataInputStream dis = null;
 
...
 
dis = new DataInputStream(serviceSocket.getInputStream());
 
...
 
// code pour la lecture
 
byte[] tempBytes = new byte[dis.readInt()];    // lit un int qui représente la longueur du message qui suit
int length = 0;
while(length < tempBytes.length){
	length += dis.read(tempBytes, length, tempBytes.length-length);
}
String msgRecu = new String(tempBytes, "ISO-8859-1");

plutot que comme ca :

Code :

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

1
2
3
 
byte[] tempBytes = new byte[dis.readInt()];    // lit un int qui représente la longueur du message qui suit
dis.read(tempBytes, 0, tempBytes.length);

Je me demande si de faire des boucles comme ca est nécessaire ou si ca fait double emploi avec l'implémentaion des flux (haut niveau). Meme question pour l'envoi : Est il judicieux de faire une boucle afin de ne pas envoyer de trop gros message en une fois.

pour infos le code de l'ecriture

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
 
ByteArrayOutputStream baos = null;
DataOutputStream dos = null;
 
...
 
dos = new DataOutputStream(serviceSocket.getOutputStream());
baos = new ByteArrayOutputStream();
 
....
 
//code pour l'écriture
 
String toSend = "CONNECT";
baos.reset();
new DataOutputStream(baos).writeInt(toSend.length());
baos.write(toSend.getBytes("ISO-8859-1"));
dos.write(baos.toByteArray(), 0, baos.toByteArray().length);
dos.flush();

Merci, Fred.

[Didier 69]

La question que je me pose, c'est pourquoi tu te demandes si c'est mieux de faire compliqué que simple !
Si tu as un doute, le mieux est de faire un test rapide en itérant de nombreuses fois le traitement et de mesurer les perfs.
A mon avis la seconde solution + simple est la + efficace et tu peux même encore simplifier en remplaçant la seconde lecture par :

Code :

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

dis.read(tempBytes);

[ndp]

salut,
je vais te donner une autre piste de reflexion: imagine que ton flux de sortie te permette d'envoyer des donnes sur le reseau. En plus de ta "charge utile", tu envois des donnees de routage...

[rige47]

Salut Didier 69,
Ce n'est pas que je veux faire compliquer et je ne recherche pas spécialement une meilleur performence. Mais par exemple si j'envois un très long texte (la bible ) cela ne pose pas non plus de problemes ? C'est une déformation de programmation C ? En fait la boucle de lecture se trouve tel quel dans le cours Java de mon prof qui n'est pas un con alors je me pose vraiment la question de la pertinence de cette boucle.

Fred.

[rige47]

Salut ndp,
Tu veux dire que c'est les flux qui gerent tout automatiquement et que je ne dois pas me tracasser et regarder si toutes les données envoyées sont bien recues.

Fred.

[Didier 69]

Oups, autant pour moi, je n'avais pas bien vu où était ton problème.
La bonne solution est effectivement la première puisque tu ne vas pas forcément obtenir le max que tu demandes à lire : cf. javadoc DataInputStream. read :

An attempt is made to read as many as len bytes, but a smaller number may be read, possibly zero

Pour info, si tu veux systématiquement convertir tous tes byte[] en String avec CharSet spécifique, tu peux utiliser l'option -Dfile.encoding=charsetName ou faire un

System.setProperty("file.encoding", charsetName);

en début de programme.

[rige47]

Ok merci beaucoup pour ces explications .
Dis, tu ne saurais pas m'éclairer à propos du l'écriture ? Faut il aussi prevoir une boucle si on veux écrire un très gros texte ? Apparement pas d'apres ce que je comprends dans les javadocs.

Fred.

[Didier 69]

En fait tout dépend si l'écriture est bloquante ou non.
Et tu peux surement te baser également sur la signature de la méthode : si elle retourne le nb d'octets écrits, c'est que l'écriture peut ne pas se faire d'un seul coup, sinon c'est très probablement bloquant donc une écriture suffit.

[rige47]

DataOutputStream :

void write(byte[] b, int off, int len)

Donc normalement c'est bloquant ?

Je pense aussi au Timeout de ma socket, ca peux poser des problèmes (si par exemple une TimeoutException est lancée pendant l'écriture (je ne sais meme pas si c'est possible ...))?

[Didier 69]

Je pense donc que oui.
Et selon la javadoc, la seule exception que tu peux avoir est une IOException, donc tu n'as que ces cas à prévoir.

[rige47]

ok merci pour tout ces eclaircissements.

Fred.