Discussion :

[PocoYote]

Bonjour à tous.

Je débute en Java et je tente d'écrire un petit serveur. Les clients sont écrits en C++. Ils envoient des messages en tableau de char (= byte en Java) avec le caractère '\n' pour séparer les messages.

Voici mon code pour récupérer les messages des clients :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
String message;
BufferedReader reader = new BufferedReader(new InputStreamReader(socket.getInputStream()));
while ((message = reader.readLine()) != null) { 
	int pos = message.indexOf(';');
        if (pos == -1) continue;
        String cmd = message.substring(0, pos);
	if (cmd.equals("START")) {
		...
	}
	if (cmd.equals("STOP")) {
		...
	}
}

Ceci fonctionne plutôt bien. Mais seulement quand les messages des clients contiennent des caractères ASCII. Sinon la classe String de Java encode et transforme les messages reçus.

Y a t-il moyen d'empêcher la classe String de transformer ces messages ?
Y a t-il moyen d'avoir accès au tableau de byte reçu initialement ?
Ou faut-il tout réécrire sans utiliser la classe String (tableau de byte) :ouïlle: ?

Merci pour vos réponses.

[hugo123]

Pas sur d'avoir saisi ton souci. C'est un problème d'encodage ?

Quand tu lis ton flux d'entrée, tu peux préciser le charset. InputStreamReader prend un charset en second paramètre.

[le y@m's]

Alors soit tu veux récupérer et travailler avec des bytes, dans ce cas tu travailles directement avec l'InputStream, soit tu veux travailler avec des chaînes de caractères.
Dans ce cas là, il te faut spécifier l'encodage utilisé. En effet, si tu ne spécifies pas l'encodage, java utilisera celui par défaut du système (qui peut être différent de celui utilisé pour envoyer les données, d'où des problèmes de décodage).
Tu dois donc t'assurer que les clients et le serveur utilise le même encodage pour se transmettre des chaînes de caractères.
Pour spécifier l'encodage dans ton code java, tu peux utiliser le constructeur de l'InputStreamReader.
Par exemple si tu veux utiliser l'encodage UTF8

Code :

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

BufferedReader reader = new BufferedReader(new InputStreamReader(socket.getInputStream(), "UTF8"));

Plus d'infos : L'implémentation des entrées/sorties en java

[PocoYote]

Merci pour vos réponses.

@hugo123 : moi même je ne suis pas sûr de tout comprendre.

@le y@m's : il semble que j'ai réussi à faire quelque chose car effectivement les clients envoient leurs messages en UTF8. Mais il faut que je fasse encore quelques tests car je suis perdu dans les entrées / sorties. Le mal de tête guette.

[PocoYote]

En fait, le plus simple finalement serait d'avoir une classe proche de String ou StringBuffer (avec des fonctions bien utiles comme indexOf, substring,...) qui stockerait un tableau de bytes (et ne ferait pas d'encodage / décodage).

[Deaf]

Dès que tu as une notion de caractères, tu dois avoir une notion d'encodage pour utiliser des flux, sinon tu vas au devant de problèmes.

Et comme String est une chaîne de caractères, tu ne peux pas t'en passer, à moins de traiter directement (et uniquement) avec des byte[]...

[thelvin]

Bah oui, mais c'est pas faux : supposons que dans le cas présent il ne veuille pas voir de notion de caractère, et juste transmettre les octets tels qu'il les reçoit, partitionnés par l'octet de la valeur ASCII de \n (ou par le doublet \r\n).
START et STOP pourraient être vu comme des nombres magiques, et au pire ça pourrait être les seuls caractères de chaque partition.

Bon, dans le cas présent je vois pas pourquoi s'embêter avec ça (et en particulier je vois mal comment ça pourrait être plus simple,) mais avec certains protocoles ça pourrait arriver.

Quand on a besoin de faire des traitements sur des tableaux, que ce soit des byte[] ou autre chose, la classe Arrays aide bien.

[PocoYote]

Effectivement c'est bien le cas thelvin. En fait, je n'arrive pas à reproduire exactement le fonctionnement de l'ancien serveur C++. 90% des messages fonctionnent maintenant car je les traite en UTF8. C'est déjà très bien. Mais certains messages sont des données brutes (cmd = "DATA") que je dois stocker sans les modifier. Et là ça coince.

J'ai deux solutions : soit modifier les clients en C++ pour qu'ils envoient les messages DATA sur un autre port, mais à la base je ne voulais pas. Soit tout ré-écrire avec des tableaux de byte, mais cela va me prendre du temps et j'ai un peu peur pour les performances n'étant pas au top avec Java.

En tous cas, merci pour vos réponses, cela m'a fait avancer dans ma réflexion.

[Deaf]

Je n'avais pas tout à fait compris ton besoin, au temps pour moi.

Dans ce cas, tu peux t'orienter vers un DataInputStream qui te permettra de lire à la fois des tableaux de byte et des String en UTF8 (et même des int, long, etc.) selon ce que tu attends.

[PocoYote]

Ok. Merci à tous. J'ai finalement utilisé un autre port et DataInputStream pour les messages DATA. Assez impressionné par le langage et le framework au passage.

Une petite question subsidiaire (je n'ose pas créer une autre discussion pour ça). Il me semble que les nombreux exemples d'écriture de fichier en Java sont partiellement faux :

Code :

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

1
2
3
4
5
6
7
8
9
 
try { 
     BufferedWriter out = new BufferedWriter(new FileWriter("outfilename"));
     out.write("aString"); 
     out.close(); 
     } 
catch (IOException e) 
     { 
}

J'aurais tendance à faire ça pour libérer les ressources :

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
 
BufferedWriter out = null;
try { 
     out = new BufferedWriter(new FileWriter("outfilename"));
     out.write("aString"); 
     out.close(); 
     } 
catch (IOException e) 
     {
     try {
          if (out != null) out.close();
     }
     catch (IOException e) {
     }
}

[Deaf]

En effet, la fermeture peut ne pas avoir lieu dans le premier cas.

Ton écriture n'est pas fausse, mais pour libérer les flux, la syntaxe à utiliser est plutôt celle-ci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
try { 
     BufferedWriter out = new BufferedWriter(new FileWriter("outfilename"));
     try { 
          out.write("aString"); 
     } finally {
          out.close();
     }
} 
catch (IOException e) 
{ 
}

[hwoarang]

J'aurais tendance à faire ça pour libérer les ressources :

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
 
BufferedWriter out = null;
try { 
     out = new BufferedWriter(new FileWriter("outfilename"));
     out.write("aString"); 
     out.close(); 
     } 
catch (IOException e) 
     {
     try {
          if (out != null) out.close();
     }
     catch (IOException e) {
     }
}

Au passage, tu peux faire directement :

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
 
BufferedWriter out = null;
try { 
     out = new BufferedWriter(new FileWriter("outfilename"));
     out.write("aString"); 
     } 
catch (IOException e) 
     {
     System.out.println("Erreur");
     }
finally {
     try {
          if (out != null) out.close();
     }
     catch (IOException e) {
     System.out.println("Erreur");
     }
}

Rappelons qu'un bloc finally est toujours executé, qu'on ait une exception ou non ;-)

a+

[PocoYote]

Ok. Merci Deaf.