Probleme avec send ?

Invité · 02/04/2011, 08h46

Salut a tous, j'ai un petit soucis lors de l'envoi d'un fichier octet par octet par un send, en effet le fichier est bien envoyé et arrive bien mais ma charge CPU monte a 80% et mon programme qui fait le send est figé pendant 15 secondes alors que le fichier est déja recu et écrit dans un fichier...

Ma fonction :

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
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
void envoi_fichier (SOCKET client)
{
    FILE* file = NULL;
    char addr_file[20] = "";
    recv(client, addr_file, 20, 0);     /*Le client envoit l'adresse du fichier a ouvrire */
    file = fopen(addr_file, "rb");
    char buff = 0;
    int taille = 0;
    printf("Ouverture de '%s'", addr_file);
 
        if(file != NULL)
        {
            fseek(file, 0, SEEK_END);
            taille = ftell(file);
            printf("\n%d octets a envoyer\n", taille);
            send(client, &taille, 10, 0);
    printf("\nOuverture du fichier\n");
    rewind(file);
    while (taille > 0)
    {
        /*T_BUFFER vaut 1 */
 
        if (fread(&buff, sizeof(char), T_BUFFER, file) == 0)
        {
            printf("\nErr fread()");
        }
        printf("\nCaractere lu : %c, taille : %d octets", buff, sizeof(buff));
        if (send(client, &buff, sizeof(buff), 0) == SOCKET_ERROR)
        {
            printf("\nErr send fichier\n");
        }
 
        taille -= T_BUFFER;
        }
 
        printf("Fichier envoyer\n");
        }
 
        else
        {
        printf("\nErr ouverture fichier\n");
        }
 
        fclose(file);
        printf("Fichier fermer\n");
 
}

Je ne suis pas sur d'avoir compris tout le principe de la mémoire Est ce que ma fonction envoit octet par octet ou char par char ? Et pourquoi cette surcharge CPU ? Mon programme marche très bien si j'envoit mon fichier par chaines de caractères mais apparement pour etre sur de tout bien recevoir il faut le faire caractère par caractère (ou octet par octet je ne sais pas trop). Merci de m'éclairer !!

**ram-0000** · 02/04/2011, 09h21

Quelques petites remarques (je ne sais pas si cela a à voir avec ton problème)

1/

Code :

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

recv(client, addr_file, 20, 0);

remplace le par

Code :

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

recv(client, addr_file, sizeof(addr_file), 0);

cela fait la même chose mais le jour où tu modifies la taille de ton buffer de réception, cela continuera à fonctionner

2/
Autre chose, 20 caractères, cela me semble peu pour recevoir un nom de fichier. Le premier utilisateur qui demande un fichier avec un nom plus long te fait exploser ton programme.
Pour moi, un bon choix, serait que le client envoie en premier la longueur du nom du fichier puis le nom du fichier. Toi tu reçois d'abord cette longueur, puis tu alloues un bloc de mémoire suffisant et enfin tu lise le nom du fichier dans ce bloc alloué (PS, ne pas oublier de libérer le bloc de mémoire quand tu n'en a plus besoin)

3/
plutôt que faire un fseek(END), ftell() et rewind() pour calculer la taille de ton fichier, utilise donc stat() qui permet de donner la taille du fichier (entre autres) sans l'ouvrir

4/
Plutôt que de lire ton fichier octet par octet et l'envoyer sur le réseau octet par octet, pour des raisons de performances, tu ferais mieux de le lire par bloc de 1024 octets (ou toute autre taille). La lecture dans le fichier et l'envoi sur le réseau sont consommateurs de ressources, autant diminuer leur nombre. L'envoi par bloc demande un peu de travail pour l'envoi du dernier bloc si la taille du fichier n'est pas un multiple de la taille du bloc (mais c'est tout de même assez facile)

5/
Je ne sais pas comment est fait le code de l'autre côté, mais ceci est très suspect avec bug potentiel

Code :

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

send(client, &taille, 10, 0);

Tu envoie la taille du fichier (sur un int de 4 octets a priori) sur 10 octets. Donc tu envoies 6 octets (10 - 4) de "n'importe quoi". Si à l'autre bout, c'est fait pareil, cela peut passer (même si c'est discutable) mais sinon, gare au crash

Code :

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

send(client, &taille, sizeof(taille), 0);

me parait plus "sage"

Invité · 02/04/2011, 10h46

Merci pour tes réponses j'ai quelque peux modifié mon code

En essayant de télécharger un fichier de mon autre ordi je me suis rendu compte que c'était le fait de faire plusieurs milliers de fread et send qui causait ce gros bug. Mais si j'envois mon fichier par vagues de 1024 octets il arrivera mal non ? J'avais essayé et d'un autre ordi, plus je recevais de caractère d'un coups, moins ma photo était bonne.

**ram-0000** · 02/04/2011, 10h52

Envoyé par romfox17

Mais si j'envois mon fichier par vagues de 1024 octets il arrivera mal non ?

Non non, c'est comme cela que fonctionnent des protocoles comme FTP qui sont dédiés aux transferts de fichiers.

Envoyé par romfox17

J'avais essayé et d'un autre ordi, plus je recevais de caractère d'un coups, moins ma photo était bonne.

A mon avis, il y avait un bug.

Invité · 02/04/2011, 11h10

Merci pour tes réponses, voila mon nouveau code

Je vais tester sa de pc a pc et je te dis sa

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
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
void envoi_fichier (SOCKET client)
{
    FILE* file = NULL;
    char addr_file[40] = "";
    recv(client, addr_file, 40, 0);     /*Le client envoit l'adresse du fichier a ouvrire */
    file = fopen(addr_file, "rb");
    char buff[T_BUFFER] = {0};
    int taille = 0;
    printf("Ouverture de '%s'", addr_file);
 
        if(file != NULL)
        {
            fseek(file, 0, SEEK_END);
            taille = ftell(file);
            printf("\n%d octets a envoyer\n", taille);
            send(client, &taille, sizeof(taille), 0);
    printf("\nOuverture du fichier\n");
    rewind(file);
    while (taille > T_BUFFER)
    {
        /*T_BUFFER vaut 1024 */
 
        if (fread(buff, sizeof(char), T_BUFFER, file) == 0)
        {
            printf("\nErr fread()");
        }
        printf("\nCaractere lu : %c, taille : %d octets", buff, sizeof(buff));
        if (send(client, buff, sizeof(buff), 0) == SOCKET_ERROR)
        {
            printf("\nErr send fichier\n");
        }
 
        taille -= T_BUFFER;
        }
        fread(buff, sizeof(char), taille, file);
        send(client, buff, taille, 0);
 
        printf("Fichier envoyer\n");
 
        }
 
        else
        {
        printf("\nErr ouverture fichier\n");
        }
 
        fclose(file);
        printf("Fichier fermer\n");
 
}

Invité · 02/04/2011, 11h20

Hé biens c'est beaucoups plus rapide :p Mais comme je disais j'ai des parties de l'image qui sont décolorées dans ce mode, je peux la voir mais elle n'est pas identique :/
En atendant je complète mon code sous d'autres aspects mais ce point me trouble ...

Probleme avec send ?

Réseau C

Vue hybride

Discussions similaires

Partager

Partager