Lecture fichier texte ligne par ligne

Bonjour,
Je cherche à lire un fichier texte ligne par ligne pour récupérer les valeurs contenues dans ce fichier et effectuer des calculs avec.
Le problème n'est pas la lecture du fichier en soi mais la difficulté de gérer les cas où le fichier n'aurait pas le format attendu (typiquement nombres de valeurs par ligne différent ou caractères parasites dans le fichier).

Si on a un fichier de cette forme:
Code:

1 2 3 4 3.14 0 1.51 2.28 1 0.3 2.25 1.14 3.2 5.5 6.2 0.1 4.1 0 2.3 3.2
Pas de problème, on a le bon nombre de valeurs par ligne.

Par contre:
Code:

1 2 3.14 0 1.51 // manque une valeur à cette ligne 1 0.3 2.25 1.14
Ou
Code:

1 2 3.14 0 1.51 2.28 1 0.3 2.25 1.14 3.2 5.5 // deux valeurs en trop sur cette ligne
Ou plus vicieux:
Code:

1 2 3 4 3.14 0 1.51 2.28 1 0.3 2.25 1.14 3.2 5.5 6.2 0.1 4.1 0 2.3 3.2 // j'ai ajouté un espace à la fin de cette ligne
Je lis le fichier de cette manière:
Code:

1 2 3 4 5 6 7 8 9 10 while(!(m_file.getStream().eof())) { // je lis une ligne du fichier et récupère les valeurs initializeParameters(); // test des valeur lues if(hasValideParameters()) { // j'effectue mes calculs avec ces valeurs } }
Ce code marche si j'ai le bon nombre de valeurs par ligne et si la dernière ligne ne se finit pas par un espace (sinon un tour de boucle supplémentaire avec des valeurs aléatoires).
Je souhaiterais donc vérifier de manière simple que le fichier d'entrée soit bien "formaté".

Je ne sais pas si c'est suffisemment clair.

Merci de votre aide.

Bonjour,

je pense qu'il faudrait plutôt modifier les fonctions que tu présentes pour gérer ces "cas".
A priori avec une utilisation correcte des flux et opérateurs >> le mauvais formatage décrit n'est pas problématique.

Bonjour,

Personnellement, lorsque j'ai un fichier à lire ligne par ligne, j'utilise la fonction std::getline().
Code:

1 2 3 4 5 ifstream fichier("..."); string ligne; while (getline(fichier,ligne)) { traitement de la ligne ici }

OK, je vois ce que tu veux dire, j'ai besoin de rediriger les valeurs de la manière suivante

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
27
28
29
30
31
32
33
34
35
36
37
38
39
  std::fstream m_file;
  std::string sFileIn;
  std::string strCurrentLine;
  bool bFileIn = false;
 
  int param1;
  double param2;
  double param3;
  double param4;
 
  do
  {
	  std::cout <<"\n    Name of the entry file : ";
	  std::getline(std::cin, sFileIn);
	  m_file.open(sFileIn.c_str(),std::ios::in);
 
	  if(!m_file.is_open())
	  {
		  std::cout <<"\t\t ---> File  "<<sFileIn<<" doesn't exist !";
	  }
	  else
	  {
		  bFileIn = true;
	  }
  }while(bFileIn == false);
 
  while(std::getline(m_file, strCurrentLine))
  {
    std::stringstream stream(strCurrentLine);
 
    stream>>param1;
    stream>>param2;
    stream>>param3;
    stream>>param4;
   /* ... */
 
  }
  std::cout <<"\n  Close file."<<"\n\n";
  m_file.close();

Est-ce une bonne solution?

Citation:

Envoyé par LinuxUser

Est-ce une bonne solution?

Je pense que oui, fait l'essai, mais remplace std::stringstream par std::istringstream (ligne 29).

[EDIT]
Et std::fstream par std::ifstream à la ligne 1.
Dans la mesure du possible, évite les flux bidirectionnels si tu n'en a pas besoin.
[/EDIT]

OK, j'ai corrigé, ça à l'air de marcher impec.
Par contre, faut eviter les flux bidirectionnelles si on en a pas besoin, uniquement pour une raison de clareté (pour indiquer qu'on lu ou écrit seulement) ou y a t ils d'autres raisons plus pragmatiques (optimisation, mémoire, temps execution, ...)?

Citation:

Envoyé par LinuxUser

Par contre, faut eviter les flux bidirectionnelles si on en a pas besoin, uniquement pour une raison de clareté (pour indiquer qu'on lu ou écrit seulement) ou y a t ils d'autres raisons plus pragmatiques (optimisation, mémoire, temps execution, ...)?

Raison de clareté, oui sûrement.
Mais il peut aussi y avoir un problème de droit d'accès, si tu n'as que les droits de lecture sur un fichier, tu ne pourras pas l'ouvrir avec std::stream.
J'ai rencontré des problèmes avec une liaison série sous linux, ça ne marchait pas avec un flux bidirectionnel mais ça marchait bien avec un flux et entré et un autre en sortie. Je n'ai jamais compris pourquoi mais depuis je me méfie.

OK, c'est bon à savoir.

Merci beaucoup pour votre aide, problème résolu.