Fonctions de transmition et de reception de blocks de données

Marsh Posté le 31-03-2009 à 13:54:50

Salut, je voudrai savoir si les fonctions ci-dessous que j'ai défini pour la transition (resp réception) de données sont correctes et efficases. Sinon comment les améliorer ?

Remarque: dans la fonction d'envoi, je commence par envoyer (en mode texte) la taille de la donnée qu'on veux envoyer puis la donnée elle même, et dans la fonction de réception je reçois avant tout, la taille de la données que je dois recevoir, puis la donnée elle même.

Merci.

Code :

/******** Reception *************/
int RecvData (SOCKET sock, char *data, int sizeof_data)
{
int len_data, data_receved = 0, n;
char len_data_buffer[10];
/* Recevoir la taille de data */
if (n = recv (sock, len_data_buffer, sizeof len_data_buffer - 1, 0) == SOCKET_ERROR);
return SOCKET_ERROR;
len_data_buffer[n] = 0;
len_data = atoi (len_data_buffer);
while (data_receved < len_data)
{
n = recv (sock, data + data_receved, sizeof_data - data_receved, 0);
if (n >= 0) data_receved += n;
else return SOCKET_ERROR;
}
return data_receved;
}
/******** Envoi *************/
int SendData (SOCKET sock, char *data)
{
int len_data = strlen(data), data_sent = 0, n;
char len_data_str[10];
sprintf (len_data_str, "%d", len_data);
if ((n = send (sock, len_data_str, strlen(len_data_str), 0)) == SOCKET_ERROR)
return SOCKET_ERROR;
while (data_sent < len_data)
{
n = send (sock, data + data_sent, len_data - data_sent, 0);
if (n >= 0) data_sent += n;
else return SOCKET_ERROR;
}
return data_sent;
}

Message édité par tomap le 31-03-2009 à 14:11:34

Reply

Marsh Posté le 31-03-2009 à 13:54:50

Reply

Marsh Posté le 31-03-2009 à 21:45:21

- Rien ne te garantit que tu recevras len_data_buffer en une fois. Il faudrait donc également boucler pour recevoir la totalité des données.
- Plutot que atoi(), préfères sscanf.
- Plutot que sprintf(), préfères snprintf pour éviter les débordements de buffer. Imagine len_data = -2000000000 , pouf t'as deja 10 caractères.

Message cité 2 fois

Reply

Marsh Posté le 31-03-2009 à 23:14:50

xilebo a écrit :

- Rien ne te garantit que tu recevras len_data_buffer en une fois. Il faudrait donc également boucler pour recevoir la totalité des données.

Et comment est ce que je vais savoir quand est ce qu'il faut s'arreter de recevoir dans len_data_buffer ? Quoi qu'il en soit, je pense a envoyer la taille de la donnée sous format binaire (un long de 4 octet) plutot qu'au format text comme je l'ai fait.

Dans SendData, j'envoi la taille comme suite:

Code :

long n, data_sent = 0, len_data_nl = htonl(strlen(data))
while( data_sent < 4 )
{
n = send (sock, ((char*)&len_data_nl) + data_sent, sizeof(len_data_nl) - data_sent, 0);
if ( n > 0 ) data_sent += n; else return SOCKET_ERROR;
}
...

Et dans RecvData, je reçois la taille comme suite:

Code :

long n, data_receved = 0, len_data_nl;
while( data_receved < 4 )
{
n = recv (sock, ((char*)&len_data_nl) + data_receved, sizeof(len_data_nl) - data_receved, 0);
if ( n > 0 ) data_receved += n; else return SOCKET_ERROR;
}
long len_data = ntohl(len_data_nl);
...

J'aimerai savoir si c'est une bonne idée

Reply

Marsh Posté le 01-04-2009 à 00:12:44

xilebo a écrit :

- Plutot que atoi(), préfères sscanf.

strtol() plutôt non ?

Message cité 1 fois

Reply

Marsh Posté le 01-04-2009 à 16:04:16

sligor a écrit :

strtol() plutôt non ?

Oui en effet. Mais personne n'as de commentaires sur ma méthode si dessus ?

Message édité par tomap le 01-04-2009 à 16:04:23

Reply

Fonctions de transmition et de reception de blocks de données

Sujets relatifs:

Leave a Replay