lire les bytes d'un fichier:ifstream vs FILE.

Bonjour,
J'avais à implementer dans un test-unitaire un calcul de CRC, et je suis tombé sur un os qui me semble assez bizarre...

En gros, j'utilise un ifstream et son operateur >> pour lire les bytes d'un fichier, mais il sort ma boucle, 14 bytes avant la fin du fichier!

Code:

---

1 2	for (unsigned char c ; out >> c ; m_crc32.process_byte(c), uiByteRead++) ;

---

Toutes les idée seront les bienvenues! C'est pas un probleme bloquant, mais ça me semblait etre une manière compact et sommes toutes assez élégante de faire et que cela ne marche pas ..... ?


Dans le code ci-dessous, j'utilise boost pour recuperer la taille de mon fichier.
Ensuite, la premier section entre {} est mon premier jet utilisant un ifstream. Et enfin, la seconde section entre {} est le code "à la C" qui lui ce comporte assez bien !

voici la sortie du code... il y a une erreur de 1 sur le compteur en mode C, mais c'est pas le probleme. Le probleme c'est que je ne m'explique pas pourquoi avec ifstream, j'ai un flag eof qui est levé trop tot (a noter que j'ai aussi essayer de faire un while(!out.eof()) et que le résultat est le même le flag est levé alors qu'il manque 14 iteration!)

File size = 1504
by ifstream in-loop 1490
by fopen/fread in-loop 1505

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

size_t filesize = boost::filesystem::file_size(outputFile); std::cout << "File size = " << filesize << std::endl;   { boost::crc_32_type m_crc32; unsigned int uiByteRead = 0; std::ifstream out(outputFile, std::ios::binary | std::ios::in); for (unsigned char c ; out >> c ; m_crc32.process_byte(c), uiByteRead++) ; std::cout << "by ifstream in-loop " << uiByteRead << std::endl; }   { boost::crc_32_type m_crc32; unsigned int uiByteRead = 0; FILE* pf = fopen(outputFile, "rb"); while(!feof(pf)) { unsigned char c = 0; fread(&c, 1, 1, pf); m_crc32.process_byte(c); uiByteRead++; } fclose(pf); std::cout << "by fopen/fread in-loop " << uiByteRead << std::endl; }

---

J'ai un peu de mal avec tes ifstream se nommant out...

Quand tu fais out >> c, tu interprètes le contenu du flux, sautant par exemple les espaces. Pour faire une lecture sans interprétation, utilise par exemple :

Code:

---

1 2 3

int c1 = out.get(); // En cas de problème, c1 vaut traits::eof(). char c2; out.get(c2); // Tester l'état de out pour savoir si ça c'est bien passé.

---

Merci de la réponse.

Je suis d'accord avec la remarque sur le 'out'... Mais comme tu l'auras remarqué c'est du quick&dirty pour tester et comprendre ce qui se passe!

Par contre, concernant la remarque sur le fait de 'sauter les espaces', je vais vérifier à l'occassion, mais via std::ios::binary, il ne devrait pas y avoir d'interpretation non ?

binary indique qu'en bas niveau, les caractères ne sont pas interprétés. C'est surtout applicable aux fins de ligne.

>> est un opérateur de flux formaté, par opposition à get.

Donc en bref et apres verification, les 2 ecritures ci-dessous sont équivalentes, et font bien ce que je souhaitais.

Code:

---

1 2 3

for (char c = 0; inputStream.get(c) ; crc32.process_byte(c), uiByteRead++) ; for (char c = 0; inputStream>>std::noskipws>>c ; crc32.process_byte(c), uiByteRead++) ;

---

Merci.

Salut,

Citation:

---

Envoyé par alex.buisson

Code:

---

for (char c = 0; inputStream>>std::noskipws>>c ; crc32.process_byte(c), uiByteRead++) ;

---

---

En maintenance, l'utilisation de l'opérateur >> me surprendra toujours pour un fichier binaire. Dans ce mode, le principe de moindre surprise me poussera plutôt à utiliser du get.

Deuxième remarque, faire la lecture dans la condition du for déjà bof, mais surtout faire le traitement dans la partie d'incrément, là ça passe pas.

Code:

---

1 2 3 4 5 6

int c = in.get(); while(c!=std::ifstream::traits_type::eof()) { crc32.process_byte(c); c = in.get(); };

---

Citation:

---

Envoyé par 3DArchi

Salut,


En maintenance, l'utilisation de l'opérateur >> me surprendra toujours pour un fichier binaire. Dans ce mode, le principe de moindre surprise me poussera plutôt à utiliser du get.

Deuxième remarque, faire la lecture dans la condition du for déjà bof, mais surtout faire le traitement dans la partie d'incrément, là ça passe pas.

Code:

---

1 2 3 4 5 6

int c = in.get(); while(c!=std::ifstream::traits_type::eof()) { crc32.process_byte(c); c = in.get(); };

---

---

Meme si en general je n'aime pas les conditions a effet de bord, il me semble que leur utilisation est idiomatique pour les IO. Donc j'ecrirais

Code:

---

1 2 3 4

while (in.get(c)) { crc32_process_byte(c); }

---

tout comme j'ecris while(getline(in, line)) quand je lis par ligne. D'autant plus que je n'aime pas la duplication dans ta maniere d'ecrire ni le controle de la boucle disperse. En fait, ce que tu as ecris est la definition de la boucle for, et celle ci serait preferable (mais la duplication me gene toujours, meme si elle est simple):

Code:

---

1 2 3 4

for (int c = in.get(); c!=std::ifstream::traits_type::eof(); c = in.get()) { crc32.process_byte(c); }

---