Problème réception chaine fonction recv()

Version imprimable

28/05/2010, 12h41
nostalgius

Problème réception chaine fonction recv()

bonjour tout le monde..
J'ai un p'tit problème avec la fonction recv(): cette dernière ne me récupère que le dernier caractère envoyé. j'ai développé 2 app, un serveur qui écoute sur un port, dès qu'un client se connecte, le serveur lui envoi un mot de bienvenue, mais là mon client ne récupère que la dernière lettre du mot ( le t du mot Salut), hors en utilisant le client telnet M$ , j'ai la chaine complète. d'où peut venir le problème ? :arf:
là pour l'éviter, je suis obligé d'envoyé ma chaine caractère par caractère, et faire de même pour la récupérer du coté client. là ça marche, mais coté performance c'est pas vraiment joli :calim2:
28/05/2010, 13h07
ram-0000

Citation:

Envoyé par nostalgius

... d'où peut venir le problème ? ...

Du client ou du serveur mais sans plus de code, on ne saura t'aider.

Montre nous le code du client (ou au moins l'envoi des caractères) et le code de serveur (ou au moins la réception des caractères)

Re,
comme je disais j'ai testé mon serveur avec le client telnet de micro****, et là je récupérai la totalité de la chaine envoyée par le serveur, ce qui me laisse pensé que le problème vient de ma fonction recoi_donnees(), voici le code des deux fonctions:

//Fonction envoi_donnees():

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
int envoi_donnees(SOCKET sur_socket)
{
    int retour = 1;
    char data[]="Salut";
    char t_donnees_a_envoyer[10] = {0};
    sprintf(t_donnees_a_envoyer, "%d", strlen(data));
    int i;
    for(i=0;i<strlen(t_donnees_a_envoyer);i++)
    {
        if(send(sur_socket, &t_donnees_a_envoyer[i], 1, 1)!= SOCKET_ERROR)
        {
            //printf("\n\tSucces envoi taille donnees (%d).", atoi(t_donnees_a_envoyer));
        }else{
            retour = 0;
            break;
        }
    }
    if(retour == 1){
        send(sur_socket, &t_donnees_a_envoyer[strlen(t_donnees_a_envoyer)+1], 1, 1); //j'ajoute le '\0'
            if (send(sur_socket, data, strlen(data) ,1) == SOCKET_ERROR)
                {
                    printf("\n\tErreur envoi donnees.");
                    retour = 0;
                }
            else printf("\n\tSucces envoi donnees (%s).", data);
    }else { printf("\n\tErreur envoi taille donnees."); }
    return retour;
}

Fonction recoi_donnees()

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
43
int  recoi_donnees(SOCKET sur_socket)
{
    int retour = 1;
    char t_donnees_a_recevoir[10];
    int i = -1;
    do
    {
        i++;
        if(recv(sur_socket, &t_donnees_a_recevoir[i], 1, 1) != SOCKET_ERROR)
        {
            printf("\n\tSucces reception donnees: %c", t_donnees_a_recevoir[i]);
        }else{
            retour = 0;
            break;
        }
    }while(t_donnees_a_recevoir[i]!='\0');
 
    if(retour == 1)
    {
        int taille=atoi(t_donnees_a_recevoir);
        int t=0, t_donnees_recu=0;
        char* data_recu = malloc(taille*sizeof(char));
        for(i=0;i<taille;i++) data_recu[0]='\0';
        while(t_donnees_recu<taille)
        {
            t=recv(sur_socket, data_recu+t_donnees_recu, 1, 1);
            if (t == SOCKET_ERROR)
            {
                printf("\n\tErreur reception donnees @ %d.", t_donnees_recu);
                retour = 0;
                break;
            }
            else{
                printf("\n\tSucces reception donnees (%d:%s).", t_donnees_recu, data_recu+t_donnees_recu);
                t_donnees_recu += t;
            }
        }
        data_recu[i]='\0';
        printf("\n\tdonnees recues (%s).", data_recu);
    }else { printf("\n\tErreur reception donnees."); }
 
    return retour;
}

Résultat avec telnet:
http://img190.imageshack.us/img190/1082/telneti.jpg
ça marche :)
Résultat avec mon client:
http://img171.imageshack.us/img171/7995/clientlq.jpg
:( je n'ai que le dernier caractère

28/05/2010, 17h03
ram-0000

Il va falloir travailler la présentation jeune padawan :D

1 pb dans la fonction recoi_donnees() : si tu recois 5 caractères, tu alloues 5 caractères et tu mets quand même un \0 en final (donc en dehors du bloc alloué.

Code:

char* data_recu = malloc((taille + 1) * sizeof(char));

Au fait, le retour d'un malloc(), cela se teste même si cela échoue rarement:P

Un autre pb ici (pas grave mais cela ne fait pas ce que tu penses :

Code:

for(i=0;i<taille;i++) data_recu[i]='\0';

Pourquoi fais tu systématiquement un send() avec 1 comme dernier paramètre. Le 4 eme parametre attend 0 ou bien un ou logique avec MSG_OOB, MSG_DONTROUTE, MSG_DONTWAIT et MSG_NOSIGNAL. Je pense que dans ton cas, 0 suffit largement (même remarque pour la fonction recv()).
28/05/2010, 17h11
nostalgius

Merci pour la réponse aussi rapide :D
je vais mettre le dernier paramètre de send() à 0, modifier la taille d'allocation et supprimer la ligne d'initialisation de toutes les cases de data_recu. Et voire si ça marche. merci encore :wink:

Dans la fonction envoi_donnees(), tu peux remplacer le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
int envoi_donnees(SOCKET sur_socket)
{
   int retour = 1;
   char data[]="Salut";
   char t_donnees_a_envoyer[10] = {0};
   sprintf(t_donnees_a_envoyer, "%d", strlen(data));
 
   int i;
   for(i=0;i<strlen(t_donnees_a_envoyer);i++)
   {
      if(send(sur_socket, &t_donnees_a_envoyer[i], 1, 1)!= SOCKET_ERROR)
      {
         //printf("\n\tSucces envoi taille donnees (%d).", atoi(t_donnees_a_envoyer));
      }
      else
      {
         retour = 0;
         break;
      }
   }
   if(retour == 1)
   {
      send(sur_socket, &t_donnees_a_envoyer[strlen(t_donnees_a_envoyer)+1], 1, 1); //j'ajoute le '\0'

par

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
int envoi_donnees(SOCKET sur_socket)
{
   int retour = 1;
   char data[]="Salut";
   char t_donnees_a_envoyer[10] = {0};
   sprintf(t_donnees_a_envoyer, "%d", strlen(data));
 
      if(send(sur_socket, t_donnees_a_envoyer, strlen(t_donnees_a_envoyer) + 1, 0)!= SOCKET_ERROR)
      {
         //printf("\n\tSucces envoi taille donnees (%d).", atoi(t_donnees_a_envoyer));
      }
      else
      {
         retour = 0;
         break;
      }
   }
   if(retour == 1)
   {

Tu y gagneras en concision et en clarté

28/05/2010, 17h29
nostalgius

http://img72.imageshack.us/img72/5049/clientw.jpg

Merci énormément :wow:
c'est résolu! mon client reçoit la chaine entière, tout venait du dernier paramètre
de send()/recv() :roll:.
28/05/2010, 17h35
ram-0000

Bon, maintenant que ton programme fonctionne, il est temps de le rendre plus joli (le code) et plus fiable aussi.

Je suis sûr que le programme ne fonctionne plus si au lieu d'envoyer "Salut", on envoit "Il fait beau et chaud".
28/05/2010, 17h36
nostalgius

je l'utilisai pour garantir l'envoi totale de la taille, mais maintenant j'en ai plus besoin, oui, je ferai mieux de supprimer toutes ces lignes, merci encore vous m'avez été d'une grande aide :ave: passez une bonne fin de journée.
28/05/2010, 17h50
nostalgius

oui, ça a donnée quelque chose du genre:
http://img16.imageshack.us/img16/4763/clientj.jpg
mais en initialisant le tableau:

Code:

for(i=0;i<taille+1;i++) data_recu[i]='\0';

ça pose plus de prb:
http://img227.imageshack.us/img227/1184/cliente.jpg
Merci encore :D