Probleme avec send ?

Version imprimable

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:

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 !!

02/04/2011, 09h21
ram-0000

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

1/

Code:

recv(client, addr_file, 20, 0);

remplace le par

Code:

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:

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:

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

me parait plus "sage" :P
02/04/2011, 10h46
Invité

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.
02/04/2011, 10h52
ram-0000

Citation:

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.

Citation:

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.

Merci pour tes réponses, voila mon nouveau code :) Je vais tester sa de pc a pc et je te dis sa :)

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
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");
 
}

02/04/2011, 11h20
Invité

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 ...
02/04/2011, 15h37
souviron34

normal :D

une image couvrant de 0 à 255, il faut utiliser des "unsigned char" et non des char..

:D

De plus, comme on te l'a dit plus haut, utilise la fonction stat ou fstat por connaître la taille du fichier plutôt que fseek, tell, rewind...
03/04/2011, 20h37
Invité

Merci beaucoup =) les unsigned je les utilise jammais mais tu viens de résoudre mon problème :) Je vais retravailler tout mon code !!

Au passage si quelqu'un avait une voie vers laquelle me tourner car je suis un débutant en console et apres quelques tutos j'ai attaqué les sockets en protocole TCP (et j'ai une idée pour l'UDP), mais la je mets au point un programme avec lequel je peux faire pas mal de choses sur mon autre ordi en réseau local. Mais a part ça pour me faire les dents, je ne sais pas trop vers quoi me tourner =/ Les fenettres ? Dans ce cas je ne sais pas trop quoi utiliser ...