Discussion :

[_skip]

Bonjour,

Je sais que ça paraît une question de débutant mais je n'avais jamais utilisé la STL directement pour manipuler des fichiers.
Après avoir passé un long moment sur la doc, je pensais utiliser cette technique :

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
std::ifstream  fs( _termsFilePath->c_str()  ) ;
std::string line;
 
if(!fs )
{
	//erreur
}
 
while( !fs.eof() )
{
	if(!std::getline(fs, line) )
	{
	      //erreur	
	}
 
        //faire qqch avec line
}

Ensuite je suis tombé sur la faq C++ de dvp où on conseille cette approche :

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
#include <string>
#include <fstream>
#include <iostream>
 
int main()
{
    // le constructeur de ifstream permet d'ouvrir un fichier en lecture
    std::ifstream fichier( "fichier.txt" );
 
    if ( fichier ) // ce test échoue si le fichier n'est pas ouvert
    {
        std::string ligne; // variable contenant chaque ligne lue
 
        // cette boucle s'arrête dès qu'une erreur de lecture survient
        while ( std::getline( fichier, ligne ) )
        {
            // afficher la ligne à l'écran
            std::cout << ligne << std::endl;
        }
    }
}

Je me demande si avec cette dernière, il n'y a pas un risque de confondre le cas ou la lecture s'arrête normalement à cause d'un eof() ou à cause d'une erreur?

[Arzar]

Bonjour,
Pour faire la distinction entre une fin de fichier classique et une erreur il me semble qu'il faut vérifier en sortie du while si le flag badbit a été mis à 1 ou non.

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
 
// le constructeur de ifstream permet d'ouvrir un fichier en lecture
std::ifstream fichier( "fichier.txt" );
 
if ( fichier )
{
   std::string ligne; // variable contenant chaque ligne lue
    while ( std::getline( fichier, ligne ) )
   {
      // afficher la ligne à l'écran
      std::cout << ligne << std::endl;
   }
   if(fichier.bad())
   {
      // Erreur pendant la lecture (qui n'est pas une simple fin du fichier)
   }
}

[Ekleog]

La solution de la faq a également l'avantage d'être plus jolie (enfin, si on retire le if(fs) { ... } par un if(!fs) { return; } ... pour gagner un niveau d'indentation).

[_skip]

Oui merci mais ce n'est pas un concours d'esthétique de code ce sont juste des exemples et la question est de savoir si on passe pas à côté d'un cas d'erreur. J'ai lu plein de blabla sur les forums à ce sujet mais il semble que personne sur le net n'en ait rien à faire des erreurs lors de la lecture.
A croire que pour tout le monde, si le fichier a été ouvert, c'est succès assuré...
Enfin si on ose pas se fier à cela, le bad bit mentionné par Arzar serait la bonne solution on dirait pour ne pas confondre la sortie de boucle en cas d'eof et la sortie de boucle silencieuse due à une erreur...

De mon côté j'ai opté pour un code à base d'exception :

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
 
	std::ifstream  fs ;
	std::string line;
 
	fs.exceptions ( std::ifstream::failbit | std::ifstream::badbit );
 
	try
	{
		fs.open("fichier.txt");
 
		while( std::getline(fs, line) )
		{
			std::cout << line << std::endl;
		}
	}
	catch(std::ifstream::failure& ex)
	{
		//lancer exception perso
	}

C'est la solution que j'utilise mais mes questions initiales demeurent. C'est pour ma culture générale disons.

[Trademark]

Je me demande si avec cette dernière, il n'y a pas un risque de confondre le cas ou la lecture s'arrête normalement à cause d'un eof() ou à cause d'une erreur?

Non car le :

Code :

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

while ( std::getline( fichier, ligne ) )

renvoie false dès qu'un flag est set (badbit, eofbit or failbit), ça a le même comportement que si tu faisais :

Code :

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

while ( std::getline( fichier, ligne ).good() )

Donc le code de la faq est très bien Et celui de Arzar est adapté à ton cas, c'est en tout cas moins lourd et plus performant que d'utiliser des exceptions.

[_skip]

Ca je l'ai compris mais il renvoie aussi faux si le bit eof est mis...
Donc tu pourrais avoir ta lecture qui s'arrête au milieu du fichier sans que tu saches jamais que quelque chose s'est mal passé, c'est le risque de confondre que je mentionne dans mon premier post.

C'est pour cela que je pensais à une solution qui teste EOF séparément...