Discussion :

[Blo0d4x3]

Bonjour,

Petit problème pour lire une image png, Le code suivant marche bien quand je veux lire un bête fichier txt, mais pas une imag png, Dans mon buffer szBuf, je me retrouve qu'avec les 4 premiers octets du fichier, dans le fichier lu, le cinquième est un 00. Je pense que c'est lui qui fait bugger mon code.

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
 
fileHandle = CreateFile(ImagePath, GENERIC_READ, FILE_SHARE_READ, NULL, OPEN_EXISTING, 0, NULL);
 
	if (fileHandle == INVALID_HANDLE_VALUE)
	{
		printf("Error CreateFile: %d\n", GetLastError());
	}
 
	DWORD dwFileSize = GetFileSize(fileHandle, NULL);
 
	if (dwFileSize <= 0)
	{
		printf("Error filesize\n");
	}
	else
	{
		printf("Filesize: %d\n", dwFileSize);
 
		char *szBuf = new char[dwFileSize + 1];
		DWORD dwBytes = 0;
 
		if (!ReadFile(fileHandle, szBuf, dwFileSize, &dwBytes, NULL))
		{
			printf("Readfile failed: %d\n", GetLastError());
		}
		else
		{
			printf("sizeof read: %d, nbtoread: %d\n", sizeof(szBuf) / sizeof(szBuf[0]), dwFileSize);
 
			printf("file: %s\n", szBuf);
			string filecontent = string(szBuf);
			printf("filecontent %s\n", filecontent);

Si vous avez une solution.

Merci

[Bousk]

Salut,

pourquoi penses-tu avoir un pb ?
Parce que printf("file: %s\n", szBuf); ou printf("filecontent %s\n", filecontent); affichent que qqs caractères uniquement ?
Si oui, c'est normal, c'est un fichier binaire, donc des \0 y'en a un peu partout qui trainent. Vouloir afficher un fichier binaire dans la console, et comme du texte, c'est

[Médinoc]

Attention, je ne vois rien dans ton code qui permette de s'assurer que szBuf[dwFileSize] est bien un caractère nul.

[Blo0d4x3]

@bouzk printé un binaire c'est WTF je suis d'accord, c'était juste un test pour "voir" combien de caractères chelou j'aurai. et forcément il s'arrête à 4. Le sizeof read me renvoit 4...
@Médinoc Le cinquième est un null

Comment je peux m'assurer que le fichier est complet dans szBuff? Dans le debugger de VC je ne peux pas voir le contenu de la variable. :\

EDIT:

J'ai trouvé d'ou venait mon problème... après avoir lu l'image je l'encode en base64

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
string Base64Encode(const char *s)
{
	DWORD dwSize = 0;
	char szData[1024] = { 0 };
 
	CryptBinaryToStringA((BYTE*)s, strlen(s), CRYPT_STRING_BASE64, nullptr, &dwSize);
	if (dwSize < 1024) CryptBinaryToStringA((BYTE*)s, strlen(s), CRYPT_STRING_BASE64, szData, &dwSize);
 
	return szData;
}

Et ca doit être dans cette fontion que ca déconne avec l'histoire du null byte.

[Blo0d4x3]

Je cerne mieux le problème, à la lecture, j'ia pas de problème, le fichier est bien complet, c'est juste que convertis le szBuff en string, donc forcement ...

Code :

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

1
2
 
datas = datas + "&s=" + string(szBuf);

Une bonne solution serait de convertir szBuff en quelle type avant de l'avoir en string?

[white_tentacle]

Tout dépend ce que tu veux en faire derrière. Mais std::string ne peut pas contenir de \0. Si tu veux un conteneur qui contienne des \0, std::vector<char> peut être une solution.

Dans tous les cas, la signature de ta méthode base64encode n’est pas adaptée :*elle prend un char const*, donc une chaîne terminée par un \0. Or, tu as des \0 au milieu de ta valeur, que tu souhaites encoder en base64 (ça ne pose pas de problème particulier). Il faut donc que tu lui rajoutes un paramètre taille et modifie le corps en conséquence.

Sinon, la remarque de Médinoc est toujours valide : ta chaîne n’est pas terminée par un \0. Sois-en conscient quand tu la manipules (ça n’est pas forcément un problème, mais ça demande de l’attention, c’est vite fait d’appeler une fonction qui s’attend à ce que la chaîne soit terminée par un \0, et là, c’est le drame).

[Bousk]

Je cerne mieux le problème, à la lecture, j'ia pas de problème, le fichier est bien complet, c'est juste que convertis le szBuff en string, donc forcement ...

Code :

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

1
2
 
datas = datas + "&s=" + string(szBuf);

Une bonne solution serait de convertir szBuff en quelle type avant de l'avoir en string?

std::string peut tout à fait convenir, à condition d'utiliser le bon constructeur. C'est identique à l'utilisation d'un vector<char> en pratique.