reception de données avec recv()

Version imprimable

Salut,

J'essaie de modifier mon code serveur/client pour pouvoir envoyer des fichiers entiers. Du coté, serveur, je dispose de la fonction qui me lit le fichier que je lui désigne puis me l'envoie avec la fonction send () :

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
44
45
46
47
48
49
50
51
52
53
 
char nom[50] = "c:\\fichier_send.txt";
int fin, current;
FILE *fichier_src = fopen (nom, "rb");
if (fichier_src == NULL)
{
   perror ("fopen() ");
   exit (0); //exit(EXIT_FAILURE)
}
 
fseek(fichier_src, 0, SEEK_END);
fin=ftell(fichier_src);
/* On se place a l'endroit qui va bien dans le binaire */
fseek (fichier_src, 0, SEEK_SET);
current = 0; /* Détermination de la position courante dans la fichier */
 
#define BUFSIZE 1024
int nb_lu;
char buffer[BUFSIZE];
memset (buffer, 0, BUFSIZE);
nb_lu = fread (buffer, sizeof (char), BUFSIZE, fichier_src);
int nb_send;
int inc = 1;
while ((nb_lu > 0) && (nb_lu <= fin - current))
{
   if ((nb_send = send (csock, buffer, nb_lu, 0)) == -1)
      {
         perror ("send() ");
         exit (-1);
      }
 
   printf ("nb_send(boucle) \'%d \' = %d\n", inc, nb_send);
   if (nb_send == -1)
      perror ("send () ");
   current = current + nb_lu;
   memset (buffer, 0, BUFSIZE);
   inc++;
   if (fin - current < BUFSIZE)
   {
      break;
   }
   else
   nb_lu = fread (buffer, sizeof (char), BUFSIZE, fichier_src);
}
 
memset (buffer, 0, BUFSIZE);
fread (buffer, sizeof (char), fin - current, fichier_src);
if ((nb_send = send (csock, buffer, fin - current, 0)) == -1);
{
   perror ("write() ");
}
printf ("nb_send = %d\n", nb_send);
fclose (fichier_src);

Mais mon problème se situ au niveau du client qui reçoit les données transmises par send(). Quand je lance le transfère d'un paragraphe de texte, la fonction recv() me le copie dans un fichier et répète la copie infiniment (en tout cas, ça n'a pas l'air de s'arrêter). Puis j'ai remarqué je mets un temps infini pour sortir de la boucle while. La taille du fichier en sortie est très lourd (40MB) pour un petit paragraphe. Est ce que vous pouvez m'indiquer comment améliorer ma fonction recv() ?
merci

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
 
char nom[50] = "fichierRecubis.txt";
        FILE *fichier_src = fopen (nom, "wb");
        if (fichier_src == NULL)
            {
                    perror ("fopen() ");
                        exit (0); //exit(EXIT_FAILURE);
                }
 
    int bytesrecu=0;
    int totalbytes=0;
 
    /* Si l'on reçoit des informations : on les affiche à l'écran */
        if(bytesrecu=recv(sock, buffer, 1024, 0) != SOCKET_ERROR)
    {
    totalbytes += bytesrecu;
    fwrite(buffer,1024, 1,fichier_src);
        /*printf("Recu : %s\n", buffer);*/
    }
 
    while(totalbytes<4096)
    {
 
        /* Si l'on reçoit des informations : on les affiche à l'écran */
        if(bytesrecu=recv(sock, buffer, 1024, 0) != SOCKET_ERROR)
    totalbytes += bytesrecu;
    fwrite(buffer,1024, 1,fichier_src);
        /*printf("Recu : %s\n", buffer);*/
 
    }
 
    fclose(fichier_src);
        }
        /* sinon, on affiche "Impossible de se connecter" */
        else
        {
            printf("Impossible de se connecter\n");
        }

PS : ne soyez pas choqué par mon code, je suis un étudiant en physique qui s'intéresse à l'informatique :)

18/06/2012, 16h59
mala92
salut,

je ne vais pas résoudre ton souci, mais ...
je n'aime pas ce genre de ligne

Code:

if(bytesrecu=recv(sock, buffer, 1024, 0) != SOCKET_ERROR)

je préfère:
Code:

1 2 bytesrecu=recv(sock, buffer, 1024, 0); if(bytesrecu != SOCKET_ERROR)
Pourquoi faire :

Code:

fwrite(buffer,1024, 1,fichier_src);

alors que recv te donne le nombre d'octets à copier :

Code:

fwrite(buffer, bytesrecu, 1,fichier_src);
18/06/2012, 17h05
fusionfroide

Ok merci pour vos précisions.

Sinon, est ce qu'il y a un moyen de vérifier si la fonction recv() reçoit effectivement tout les paquets qui lui sont transmit par send() ?
18/06/2012, 19h22
Neckara

Tu as déjà essayé de faire afficher a valeur de totalbytes à chaque tour de boucle afin de comprendre pourquoi tu as une boucle infinie?

Salut,

Alors j'ai modifié le code de réception des données de la façon suivante pour faire un test :
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 char nom[50] = "fichierRecu.txt"; FILE *fichier_src = fopen (nom, "wb"); if (fichier_src == NULL) { perror ("fopen() "); exit (0); //exit(EXIT_FAILURE); } int bytesrecu=0; int totalbytes=0; /* Si l'on reçoit des informations : on les affiche à l'écran */ if(bytesrecu=recv(sock, buffer, BUFZISE, 0) != SOCKET_ERROR) { totalbytes = totalbytes + bytesrecu; fwrite(buffer,1024, 1,fichier_src); printf("===========================\n"); printf("Recu : %d\n",totalbytes); printf("===========================\n"); } fclose(fichier_src); } /* sinon, on affiche "Impossible de se connecter" */ else { printf("Impossible de se connecter\n"); }
Alors ce code, me sauvegarde dans un fichier seulement les 1024 premiers octets du texte à transmettre (donc le début du texte est transmis). d'ailleurs le fichier crée pèse 1024 octets. Sinon, ce qui m'étonne c'est ce que me renvoie la fonction recv() : la variable totalbytes vaut 1 après une première exécution 8O . normalement recv() renvoie le nombre d'octets reçu qui dans ce cas là 1024 ... je ne comprends pas pourquoi.

19/06/2012, 14h39
mala92

On dirait que mes recommandations ne servent à rien. :aie:

Citation:

Envoyé par fusionfroide

la variable totalbytes vaut 1 après une première exécution 8O . normalement recv() renvoie le nombre d'octets reçu qui dans ce cas là 1024 ... je ne comprends pas pourquoi.

Qui te dit que bytesrecu est le nombre d'octets reçu ?
moi je dirais plutôt que c'est le résultat de recv(sock, buffer, BUFZISE, 0) != SOCKET_ERROR donc true, donc 1.:aie:

PS : c'est pour ça que je n'aime pas les opérations multiples sur 1 seule ligne et en plus sans parenthèse.
19/06/2012, 15h27
fusionfroide

Je me suis renseigné sur la fonction recv() et elle renvoie bien le nombre d'octets reçu. Je viens de faire un test avec un while(bytesrecu) en espérant qu'à la fin de réception la variable bytesrecu vaille 0 et je me suis retrouvé avec une boucle infinie et un fichier texte crée de 250 Mo ... décidément je ne comprends pas à quoi joue cette fonction recv() !
19/06/2012, 15h35
Neckara
Citation:

Envoyé par http://man.developpez.com/man2/recv.2.php

Ces fonctions renvoient le nombre d'octets reçus si elles réussissent, ou -1 si elles échouent, auquel cas errno contient le code d'erreur.

-1 est considéré comme true.

Ta boucle doit donc être :
Code:

1 2 3 while(nbOctetLu > 0) { }
19/06/2012, 15h51
fusionfroide

oui mais le problème c'est ce que dans mon cas la fonction recv() renvoie 1 même si je reçois dans le fichier 1024 octets (taille du buffer). logiquement elle dois me renvoyer 1024 non ?

Si je fais while(nombrelu>0) je vais encore entrer dans une boucle infinie puisque recv() renvoie tout le temps 1 dans mon cas ... :?
19/06/2012, 15h56
Neckara

Peux-tu nous remontrer ton code avec les modifications que tu as faites?

Voilà pour le code : je l'ai modifié pour qu'il m'affiche ce que renvoie recv() comme tu l'a conseillé dans un poste précédent. et effectivement recv() me renvoie la valeur "1" et non pas le nombre d'octets reçu. totalbytes vaut 1 ... bizarre non ?

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
 
 
char nom[50] = "fichierRecu.txt";
FILE *fichier_src = fopen (nom, "wb");
if (fichier_src == NULL)
{
   perror ("fopen() ");
   exit (0); //exit(EXIT_FAILURE);
}
 
int bytesrecu=0;
int totalbytes=0;
 
/* Si l'on reçoit des informations : on les affiche à l'écran */
if(bytesrecu=recv(sock, buffer, BUFZISE, 0) != SOCKET_ERROR)
{
    totalbytes = totalbytes + bytesrecu;
    fwrite(buffer,1024, 1,fichier_src);
    printf("===========================\n");
        printf("Recu : %d\n",totalbytes);
    printf("===========================\n");
 
}
 
fclose(fichier_src);
}
        /* sinon, on affiche "Impossible de se connecter" */
        else
        {
            printf("Impossible de se connecter\n");
        }

19/06/2012, 16h11
mala92

:koi:
Pas de boucle while, exactement le même code qu'au début, tu es sûre d'avoir posté le bon ? :aie:
19/06/2012, 16h16
Neckara
Soit tu n'as pas posté le bon code, soit tu n'as pas suivi la remarque de mala92 :
Code:

1 2 if( ( bytesrecu=recv(sock, buffer, BUFZISE, 0) ) != SOCKET_ERROR)
19/06/2012, 16h19
mala92
Citation:
Envoyé par Neckara

Soit tu n'as pas posté le bon code, soit tu n'as pas suivi la remarque de mala92 :

Code:

1 2 if( ( bytesrecu=recv(sock, buffer, BUFZISE, 0) ) != SOCKET_ERROR)
A mon avis ce n'est pas le bon code, mais je crois avoir trouvé le souci (avec le while de la question d'orgine), pour confirmer ce que je pense il me faut le nouveau code (avec le while).
19/06/2012, 16h19
fusionfroide

j'ai effectivement enlevé la boucle while pour voir pourquoi le code ne se termine pas. si le résultat renvoyé par recv() n'est pas le nombre d'octets lu, comment je vais pouvoir maitriser la boucle ? c'est pour ça que je fais un test juste avec une seule réception de données avec laquelle je reçois le premier paquet de données soit 1024 octets qui correspondent au début du teste que je veux envoyé et j'arrive à les stocker dans un autre fichier. Donc logiquement pour recevoir les autres paquets, il faut faire une boucle mais comment la maitriser si recv() ne renvoie pas le nombre d'octets lu comme prévu ... y a un truc louche dans cette histoire.

Code:

1 2 3 4 5 6 7 8 9 10 while(totalbytes<4096) { /* Si l'on reçoit des informations : on les affiche à l'écran */ if(bytesrecu=recv(sock, buffer, 1024, 0) != SOCKET_ERROR) totalbytes += bytesrecu; fwrite(buffer,1024, 1,fichier_src); /*printf("Recu : %s\n", buffer);*/ }
bytesrecu = 1 à ce que tu dis. donc pour que totalbytes = 1024 (condition du while), tu va tourner 1024 fois dans le while et donc écrire 1024 *1024, donc 10Mo.

Citation:

si le résultat renvoyé par recv() n'est pas le nombre d'octets lu, comment je vais pouvoir maitriser la boucle ?

:furieux: tu es une tête de mule c'est pas possible. Coupe ce putain de if en 2 !!!! (dsl mais j'ai l'impression qu'il faut s'énervé pour que tu comprennes :oops:)
Si tu fais
Code:

1 2 bytesrecu=recv(sock, buffer, 1024, 0); printf("bytesrecu = %d", bytesrecu);
ca dit quoi.

19/06/2012, 16h31
Neckara

Restons polis.

Il faut comprendre que l'opérateur != est prioritaire sur l'opérateur =.

Ainsi quand tu fais :

Code:

if(a = b != c)

a sera égal que résultat de b != c, c'est à dire que a sera égal à :
- 1 si b != c
- 0 si b == c

Tu as alors deux solutions :
- mettre des parenthèses pour qu'il fasse d'abord a = b puis ensuite b != c : (a = b) != c
- mettre ton code en deux lignes : a = b; puis if(b != c)

EDIT : je ne fait que présenter les deux solutions possibles, il va sans dire que la deuxième solution est bien meilleurs que la première car c'est beaucoup plus lisible et ce sans aucune perte de performance.

Y a du nouveau !

Donc voilà ma boucle while modifiée mais qui ne finit pas, mais j'arrive cette fois ci à recevoir l'ensemble du texte du fichier envoyé. le problème c'est que le dernier paragraphe est recopié infiniment dans le fichier.

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
 
 
char nom[50] = "fichierRecu.txt";
        FILE *fichier_src = fopen (nom, "wb");
        if (fichier_src == NULL)
            {
                    perror ("fopen() ");
                        exit (0); //exit(EXIT_FAILURE);
                }
 
    int bytesrecu=0;
    int totalbytes=0;
 
    /* Si l'on reçoit des informations : on les affiche à l'écran */
        if(bytesrecu=recv(sock, buffer, BUFZISE, 0) != SOCKET_ERROR)
    {
    totalbytes = totalbytes + bytesrecu;
    fwrite(buffer,BUFZISE, 1,fichier_src);
    printf("===========================\n");
        printf("test : %d\n",totalbytes);
    printf("===========================\n");
 
    }
 
    while(totalbytes<BUFZISE)
    {
 
        /* Si l'on reçoit des informations : on les affiche à l'écran */
        if(totalbytes=recv(sock, buffer, 1024, 0) != SOCKET_ERROR)
    totalbytes = totalbytes + bytesrecu;
    fwrite(buffer,1024, 1,fichier_src);
        printf("test : %d\n", totalbytes);
 
    }
 
    fclose(fichier_src);
        }
        /* sinon, on affiche "Impossible de se connecter" */
        else
        {
            printf("Impossible de se connecter\n");
        }

désolé, tout ce que je fais me permet d'apprendre la programmation, je ne suis pas brillant dans le domaine je le sais.

19/06/2012, 16h40
mala92

Citation:

Envoyé par Neckara

Tu as alors deux solutions :
- mettre des parenthèses pour qu'il fasse d'abord a = b puis ensuite b != c : (a = b) != c
- mettre ton code en deux lignes : a = b; puis if(b != c)EDIT : je ne fait que présenter les deux solutions possibles, il va sans dire que la deuxième solution est bien meilleurs que la première car c'est beaucoup plus lisible et ce sans aucune perte de performance.

Surtout pour un débutant. je n'ai jamais compris l'intérêt de condenser un code sur une ligne.

Citation:
Envoyé par fusionfroide
Y a du nouveau !

Donc voilà ma boucle while modifiée mais qui ne finit pas, mais j'arrive cette fois ci à recevoir l'ensemble du texte du fichier envoyé. le problème c'est que le dernier paragraphe est recopié infiniment dans le fichier.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 .... while(totalbytes<BUFZISE) { /* Si l'on reçoit des informations : on les affiche à l'écran */ if(totalbytes=recv(sock, buffer, 1024, 0) != SOCKET_ERROR) totalbytes = totalbytes + bytesrecu; fwrite(buffer,1024, 1,fichier_src); printf("test : %d\n", totalbytes); } ....
désolé, tout ce que je fais me permet d'apprendre la programmation, je ne suis pas brillant dans le domaine je le sais.
En plus de condenser le code sur une ligne, tu as un gros souci avec les accolades, à mon avis.

perso, un code qui marche devrait ressembler à :
Code:

1 2 3 4 5 6 7 8 9 10 11 while(totalbytes<BUFZISE) { bytesrecu=recv(sock, buffer, 1024, 0) /* Si l'on reçoit des informations : on les affiche à l'écran */ if(bytesrecu!= SOCKET_ERROR) { totalbytes = totalbytes + bytesrecu; fwrite(buffer,1024, 1,fichier_src); } printf("test : %d\n", totalbytes); }

19/06/2012, 16h47
fusionfroide

ok je vais modifier tout ça suivant vos recommandations pour qu'il devient lisible et surtout en finir avec cette boucle infinie. Je vous présenterais après la nouvelle version.

PS : dans ma boucle modifiée que j'ai posté en dernier, totalbytes vaut 2 ... :P ...

Citation:

Envoyé par mala92

Surtout pour un débutant. je n'ai jamais compris l'intérêt de condenser un code sur une ligne.

Bah, la fainéantise d'aller à la ligne :mouarf:

sinon :
Code:

1 2 3 4 5 6 7 8 if(bytesrecu!= SOCKET_ERROR) { totalbytes = totalbytes + bytesrecu; fwrite(buffer,1024, 1,fichier_src); } else break;
S'il y a une erreur, il faut sortir de la boucle.
Personnellement je ferais plutôt :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 while(...) { if(bytesrecu > 0) { totalbytes = totalbytes + bytesrecu; fwrite(buffer,1024, 1,fichier_src); } else break; } if(bytesrecu == SOCKET_ERROR) { //traitement de l'erreur }

19/06/2012, 16h56
mala92

@fusionfroide ton code d'origine n'est pas trop déconnant. c'est juste l'affectation (retour de recv) et la comparaison ( != SOCKET_ERROR) tu te fous dedans.

j'espère que tu auras appris une chose : ne jamais faire une double opération sur une même ligne (sans parenthèse). Même après plus de 10ans en C, je n'ose pas faire ce genre de chose (principalement, comme le dit Neckara, pour des questions de lisibilité).

Salut,

J'ai modifié le code et voici les points positifs que je constat :

- le fichier est envoyé et écrit dans un fichier entièrement.
- la fonction recv() renvoie le nombre de bytes lu dans mon cas 2503 octets
- la boucle s'arrête au 2503ème octet lu.

Le seul point négatif :

- présence de texte parasite à la fin du fichier. Je pense que ça vient du buffer qui contient les anciennes données transmises juste avant et vient me les rajouter à la fin du texte.

Donc la solution serait de vider le buffer à chaque tour de transmission de données et éviter que fwrite() n'ajoute du texte déjà écrit dans le fichier de récéption. Donc j'ai pensé à utiliser memset (buffer, 0, BUFZISE) juste après fwrite() mais le fichier devient impossible à ouvrir.

Quelle fonction permet de vider un buffer ? merci
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 44 char nom[50] = "fichierRecubis.txt"; FILE *fichier_src = fopen (nom, "wb"); if (fichier_src == NULL) { perror ("fopen() "); exit (0); //exit(EXIT_FAILURE); } int BUFZISE=1024; char *buffer = malloc(BUFZISE); // Allocation de la mémoire int bytesrecu=0; int totalbytes=0; /* Si l'on reçoit des informations : on les affiche à l'écran */ while(totalbytes<2503) { bytesrecu=recv(sock, buffer, BUFZISE, 0); /* Si l'on reçoit des informations : on les affiche à l'écran */ if(bytesrecu!= SOCKET_ERROR) { totalbytes = totalbytes + bytesrecu; fwrite(buffer,BUFZISE, 1,fichier_src); } printf("test : %d octets \n", totalbytes); } fclose(fichier_src); } /* sinon, on affiche "Impossible de se connecter" */ else { printf("Impossible de se connecter\n"); }

20/06/2012, 15h48
fusionfroide

Re salut,

je viens de trouver la solution à ce texte parasite qui s'ajoute à la fin du fichier. Dans fwrite, au lieux de d'écrire BUFZISE j'ai écris bytesrecu. comme ça le fwrite écrire dans mon fichier de réception des données juste les octets reçu !

solution :

Code:

fwrite(buffer,bytesrecu, 1,fichier_src);

Donc finalement pas besoin de réinitialiser le buffer :)

Merci beaucoup pour l'aide.

Maintenant que ça marche avec du texte, j'espère que je n'aurais aucune modification (ou peut être très peu) pour que je puisse transférer des fichiers binaire. après tout fread et fwrite ça marche aussi avec des fichiers binaire :zoubi: