Discussion :

[innosang]

Bonjour,

j'ai crée un serveur en c qui est censé simuler un système embarqué.

Des clients Java doivent pouvoir se connecter au serveur c. La connection fonctionne et le premier message envoyé du serveur c vers les clients java fonctionne. Cependant, les messages suivants n'arrivent pas. A mon avis il s'agit d'un problème de buffer. Voici comment je lis depuis mon client java :

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
BufferedReader plec = new BufferedReader(new InputStreamReader(socket.getInputStream()));
 
boolean continu = true ;
while(continu)
{			
	String buffer ;
 
	if ((buffer=plec.readLine()).equals("FIN")==true)
	continu = false ;	
 
	//CONNECTION MESSAGE
	if(buffer.equals("CON")==true)
	{	
		System.out.println("Connection avec le systeme embarque reussie") ;						
	}
	System.out.println(buffer) ;
 
 
}

Si vous vous pouviez m'éclairer ça serait cool.

Bonne journée

[innosang]

j'ai bien regardé du coté du serveur C ...
et apparamment tout est fait correctement. je fais un gets(buffer), je l'affiche sur la sortie standard, elle s'affiche correctement puis l'envoie s fait également correctement.

du coté Java ... je fais un read(chaine,0,10) et les messages reçu contiennent bien la bonne chaine mais elle est suivie de caractères chelou ...

[mehdiing]

salut,
tu veut dire quoi par "caractères chelou"?

[chtig]

Même question ...
Et puis ton code, on peut faire

Code :

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

1
2
3
4
 
if (((a==b)==true)==true) {
...
}

mais

Code :

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

1
2
3
4
 
if (a==b){
...
}

suffit ....

Donc ton code se résume à

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
 
boolean continu = true ;
while(continu)
{			
	String buffer ;
 
	continu = !((buffer=plec.readLine()).equals("FIN");	
 
	//CONNECTION MESSAGE
	if(buffer.equals("CON"))
	{	
		System.out.println("Connection avec le systeme embarque reussie") ;						
	}
	System.out.println(buffer) ;
 
 
}

[CyberChouan]

Tu as vérifié ton encodage? UTF-8, ISO-quelquechose, etc...?

Côté serveur, côté client, et lors du transfert

[mehdiing]

salut,
je crois que ton erreur vient de la taille du String buffer. tu doit l'initialiser à null ou à "".

[adiGuba]

Salut,

je crois que ton erreur vient de la taille du String buffer. tu doit l'initialiser à null ou à "".

Non c'est inutile puisqu'il est initialisé par le readLine(). De toute manière il est impossible d'avoir une variable "non-initialisé" en Java...


Par contre cela ressemble un peu à un buffer-overflow en C...
Comment tu envois les données du serveur C vers le client Java ???

a++

[innosang]

je fais comme celà du coté serveur :

Code :

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

1
2
3
4
5
6
7
8
9
10
while(1)
{
	char buf[50] ;
	gets(buf) ;
 
 
	if(sendto(csock, buf, sizeof(buf),0,(SOCKADDR *) &clientSin, &sinsize)==-1)
	printf("erreur send\n") ;
	printf("%s\n", buf) ;		
}

comment je peux verifier mon codage ?

[innosang]

par contre j'ai oublié de préciser que je bosse dans un thread ... ma classe étend la classe Thread

[adiGuba]

Salut,



sizeof(buf) renvoi la taille du tableau (soit 50) et non pas la taille de son contenu (3 pour "CON"). Bref tu écris 47 caractères en trop, et comme ton tableau n'a pas été mis à zéro tu obtiens des caractères aléatoires...

Code C :

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

if(sendto(csock, buf, strlen(buf),0,(SOCKADDR *) &clientSin, &sinsize)==-1)

a++

[innosang]

et bien j'ai encore un petit souci ...

lorsque j'envoi des chaine du serveur c vers le client java ... le client ne les reçoit pas complètement, il manque la dernière lettre

serveur.c :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
while(1)
{
	char buf[50] ;
	gets(buf) ;
	buf[strlen(buf)-1]='\n' ;
 
	if(sendto(csock, buf, strlen(buf),0,(SOCKADDR *) &clientSin, &sinsize)==-1)
	printf("erreur send\n") ;
	printf("%s", buf) ;
 
}

client :

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
 
boolean continu = true ;
while(continu)
{			
	String buffer ;
 
	continu = !((buffer=plec.readLine()).equals("FIN"));	
 
	//CONNECTION MESSAGE
	if(buffer.equals("CON"))
	{	
		System.out.println("Connection avec le systeme embarque reussie") ;						
	}
	System.out.println(buffer) ;
 
 
}

[innosang]

bien entendu je devrais faire buf[strlen(buf)]='\n' mais dans ce cas lorsque j'affiche la chaine du coté serveur la chaine s'affiche mais avec encore des caractères aléatoires ...

[adiGuba]

bien entendu je devrais faire buf[strlen(buf)]='\n' mais dans ce cas lorsque j'affiche la chaine du coté serveur la chaine s'affiche mais avec encore des caractères aléatoires ...

Ton buffer doit se terminer par un '\0' pour finir la chaine de caractères...

C'est une notion basique des chaines en C...

a++

[innosang]

oé biensur mais le gets le fait déjà ...

[innosang]

c'est bon tout marche à peu près ... j'ai utilisé fgets au lieu de gets