probleme dans la lecture d'un fichierRésolu

Question

bonjour,
voila je cherche a ouvrir un fichier bmp et mon code ne semble pas avoir acc&#233;s a tout le fichier je m'en remet a vous pour savoir si cela est normal merci d'avance.
 if (GetOpenFileName(&ofn)==TRUE)
       {
       HANDLE fo;
       DWORD lenbloc,s;
       char tampon[1146881],temp[1146881];
       int i;
       nbfichier++;//incremente le nombre total de fichier     
                   //on ouvre le fichier
       fo = CreateFile(ofn.lpstrFile, GENERIC_READ, 0,NULL,OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);
       if(fo == INVALID_HANDLE_VALUE)return 1;//on verifie que le fichier est ouvert
       lenbloc = 1146880 ;//recupere la taille du fichier
       tampon = (char) malloc(lenbloc); //alloue la taille du tampon
       ReadFile(fo, tampon, lenbloc, &s, NULL);//lit le fichier et place son contenu dans tampon
       fichier[1][lenbloc+1]=0;
       for(i=0;i<=lenbloc+1;i++) temp[i]= fichier[1][i];
       tampon[lenbloc+1]=0;//place le carractere de fin de chaine a la fin du tampon
       free(tampon);//reinitialise le tampon
       SendMessage(hwnd, WM_SETTEXT, 0, (LPARAM) ofn.lpstrFile);//affiche en haut de la fenetre le chemin et le nom du fichier ouvert
       CloseHandle(fo);//ferme le fichier
       }

BruNews · Accepted Answer

LPVOID pmem = 0;
DWORD size, d;
fo = CreateFile(ofn.lpstrFile,GENERIC_READ,0,0,OPEN_EXISTING,0,0);
if(fo == INVALID_HANDLE_VALUE) return 1; // ERREUR OUVERTURE
size = GetFileSize(fo, 0);
if(!size || GetLstError()) goto closeFL;
pmem = VirtualAlloc(0, size, MEM_COMMIT | MEM_RESERVE);
if(!pmem) goto closeFL;
d = 0; ReadFile(fo, pmem, size, &d, 0);
if(d != size) {
  VirtualFree(pmem, 0, MEM_RELEASE);
  pmem = 0;
}
closeFL:
CloseHandle(fo);


A ce point si pmem != 0 tu as les octets du fichier dans pmem sur une longueur size.
Pourquoi mettre un zero au bout de pmem ???

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

BruNews · Answer

ne pas d&#233;clarer d'aussi gros buffers locaux, faut alloc dynamique:
char tampon[1146881],temp[1146881];
ensuite on voit plus bas: tampon = (char) malloc(...) ??? faudrait savoir
et (char), serait pas (char*) plutot ?

tampon[lenbloc+1]=0; selon le malloc tu &#233;cris hors tableau, ecrasement de donn&#233;es.
surtout que suivi de: free(tampon); &#231;a ne me semble pas utile ce qui est plus haut.

et, etc...

Donne plutot le but &#224; atteindre, on verra ce qu'on peut faire.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

jack_ryan · Answer

en fait ce serai pour stoquer les donn&#233;es dans un tableau dans le but de les lires par la suite
a +

jack_ryan · Answer

en fait j'ai fait un m&#233;lange des codes que j'ai trouv&#233;, le zero je crois me souvenir que c'&#233;tait pour marquer la fin de mon tableau mais bon c'est vrai que c'est inutile je vais utiliser ton ton code merci beaucoup

jack_ryan · Answer

re moi  il ne reconnait pas GetLstError c'est dans quelle librairie?

BruNews · Answer

GetLastError()
pas &#233;vident de taper dans cet &#233;diteur, faut savoir aussi interpr&#233;ter les fautes de frappe...

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

jack_ryan · Answer

je suis trop nul merci
fabian

jack_ryan · Answer

VirtualAlloc(0, size, MEM_COMMIT | MEM_RESERVE); je crois qu'il manque un argument mais je ne voit pas lequel
fabian

BruNews · Answer

OUPS: PAGE_READWRITE en dernier, dur dur...

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

BruNews · Answer

et n'oublie pas de lib&#233;rer avec VirtualFree() quand fini avec le buffer pmem (SI != 0 en sortie).

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

jack_ryan · Answer

et bien merci ca a l'air de marcher pour cette partie du programme 
a bientot
fabian

jack_ryan · Answer

Bonjour je n'arrive pas &#224; r&#233;cup&#233;rer ce qu'il y a dans pmem et surtout je n'arrive pas a travailler avec les valeur du tableau. Si j'ai bien compris pmem est un tableau  de pointeur de type void comment faire pour r&#233;cup&#233;rer un tableau de type char dont je saurai me servir merci d'avance. Fabian

BruNews · Answer

NON pas un tableau de pointeurs mais un simple tableau.
Un simple cast comme d'habitude suffit:
char *c = (char*) pmem;
Si tu veux un 0 final, alloue size+1, VirtualAlloc aura d&#233;j&#224; initialis&#233; pour toi la zone m&#233;moire avec des 0.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

jack_ryan · Answer

merci pour les conseils je reviendrai peut etre pour d'autre questions a plus
fabian

Probleme dans la lecture d'un fichier

14 réponses

Votre réponse

Discussions similaires