Discussion :

[jesus144]

Bonjour,
J'essaye de lire dans un fichier texte. le fichier est lu mais à l'affichage du buffer je ne comprend pas ce qu'il se passe:

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
DWORD LoadBufferFromFile(LPCTSTR  datafilename, char ** buffer){
	HANDLE hFile; 
	DWORD dwSize;
	DWORD bLu;

	hFile = CreateFile(datafilename,GENERIC_READ,FILE_SHARE_READ,NULL,OPEN_EXISTING,FILE_ATTRIBUTE_NORMAL,NULL);
	dwSize = GetFileSize (hFile, NULL);
	
	*buffer = (char *) malloc(sizeof(char)* dwSize);	
	ReadFile(hFile,*buffer,dwSize,&bLu,NULL);
	if(bLu!=0){
		printf("%u read from %s\n", bLu, datafilename);
		printf("Loaded data:\n %s",buffer)
	}else{
		printf("Error reading %s",datafilename);
		printf("Error code : %lu \n",GetLastError());
	}

	CloseHandle(hFile);
	return dwSize;
}

ensuite lorsque je fait un printf("Loaded data:\n %s",buffer); je retrouve bien le contenu du fichier mais avec des caractères en plus:

Code :

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

1
2
3
4
//contenu du fichier "Hello Word!"
11 read from d:\tmp\toto.txt
Loaded data:
 hello word!²²²²½½½½½½½½■

Est ce que quelqu'un peu m'expliquer ce qu'il se passe ???
Merci

[jesus144]

je vien de voire que j'ai aussi un problème à un autre endroit avec le code suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
//move cursor to the begining of the section
	filePtr = SetFilePointer(hFile, sectionOffset, NULL, FILE_BEGIN); 

	//reading section data
	char * instruction = (char *)malloc(sizeof(char)* sectionSize);
	ReadFile(hFile,instruction,sectionSize,&bLu,NULL);
	if(bLu==0){
		printf("Couldn't read section\n");
		printf("Error code : %lu \n",GetLastError());
	}
	printf("Section [%u bytes]:\n %s \n",bLu,instruction);

qui me retourne la sortie suivante:

Code :

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

1
2
Section data [1227 bytes]:
 @P

Il y a bien 1227 octets a lire
la aussi, je comprend pas tout...
Merci encore

[souviron34]

on est sur le forum C et ça n'est pas du C ...

[Trap D]

Tu as un problème de taille de buffer, il faut ajouter 1 caractère de plus que la longueur du fichier, pour y mettre le zéro terminal des chaînes. Comme celà tu n'auras pas ce problème de caractères en plus à l'affichage.

[Médinoc]

on est sur le forum C et ça n'est pas du C ...

Je ne vois pas en quoi.
D'ici, ça ressemble plus à du C qu'à du C++.
Par contre, j'admets que ce n'est pas du C standard, ça utilise l'API Windows.

jesus144 : Au passage, ne caste pas le retour de malloc() : C'est inutile en C, et parfois nuisible.

[stephl]

jesus144 : Au passage, ne caste pas le retour de malloc() : C'est inutile en C, et parfois nuisible.

Nuisible? Comment cela?

[Médinoc]

Peut cacher un oubli d'inclusion de <stdlib.h> si le compilateur n'est pas convenablement réglé.

[stephl]

Peut cacher un oubli d'inclusion de <stdlib.h> si le compilateur n'est pas convenablement réglé.

Excusez-moi mais je ne vois toujours pas. Pourriez-vous être plus précis?

[jesus144]

Bonjour et merci de vos réponse.

Si je viens d'augmenter de 1 la taille des buffers, ça ne change pas l'affichage. Est ce qu'il faut que je mette le dernier caractère à %00 à la main?

Pour le deuxième affichage (la ou il n'affiche que le debut et pas la fin...) est ce que ça ne peut pas etre du a l'existance de %00 dans la chaine?

Par contre si je ne caste pas le malloc, j'ai l'erreur suivante à la compilation :

Code :

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

1
2
error C2440: '=' : cannot convert from 'void *' to 'char *'
        Conversion from 'void*' to pointer to non-'void' requires an explicit cast

J'utilise visual studio et donc cl comme compilateur...

Merci de votre aide!!!

[jesus144]

Est ce qu'il faut que je mette le dernier caractère à %00 à la main?

effectivement, pour Hello Word!, il faut que je rajoute manuellement le dernier caractère à %00 et c'est bon... Mais c'est pas bizzard de le faire a la main ça, non???

[stephl]

Vous utilisez des fonctions de l'API Windows pour lire le fichier texte. Or ces fonctions ne font pas la différence entre texte et binaire. Le '\0' n'est pas présent dans le fichier en lui-même et ce n'est pas ReadFile qui va le rajouter dans le buffer; cette fonction se contente de lire les octets du fichier.

[Médinoc]

Par contre si je ne caste pas le malloc, j'ai l'erreur suivante à la compilation :

Code :

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

1
2
error C2440: '=' : cannot convert from 'void *' to 'char *'
        Conversion from 'void*' to pointer to non-'void' requires an explicit cast

Ça veut dire que tu compiles en C++ alors que ton programme est en C.
Tu dois renommer ton fichier source en .c pour compiler correctement (ou bien, programmer en C++ et donc ne pas utiliser malloc()).

[Médinoc]

Excusez-moi mais je ne vois toujours pas. Pourriez-vous être plus précis?

Si <stdlib.h> n'est pas inclus, un compilateur C comme gcc ne fera pas d'erreur, tout au plus un warning et SEULEMENT S'IL EST ACTIVÉ (sachant que ce n'est pas activé par défaut).
gcc croira donc à une déclaration implicite de malloc() et supposera une fonction retournant un int.
Ainsi, on aura un warning "converting integer to pointer without a cast", SAUF si un petit malin a cru utile de caster le retour de malloc()...

[stephl]

OK, merci pour cette explication.

[jesus144]

Ca y est, je commence a y voire un peu plus clair... le problème est que je peux avoir des %00 dans mes chaines de caractères. Donc je ne dois pas employer %s comme format dans le printf. Il faudrai que je fasse un printf qui affiche en hexa par exemple les n premiers caratères de la chaine en les specifiant à la main en fonction du nombre de caractère lus...