Discussion :

[alex.buisson]

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 :

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

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 :

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

[JolyLoic]

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 :

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

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é.

[alex.buisson]

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 ?

[JolyLoic]

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.

[alex.buisson]

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

Code :

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

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.

[3DArchi]

Salut,

Code :

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

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 :

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

1
2
3
4
5
6
   int c = in.get();
   while(c!=std::ifstream::traits_type::eof())
   {
      crc32.process_byte(c);
      c = in.get();
   };