Discussion :

[LinuxUser]

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 :

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

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 :

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

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

Ou

Code :

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

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 :

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

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 :

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

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.

[Bousk]

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.

[jmv]

Bonjour,

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

Code :

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

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

[LinuxUser]

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

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
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?

[jmv]

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]

[LinuxUser]

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, ...)?

[jmv]

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.

[LinuxUser]

OK, c'est bon à savoir.

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