Discussion :

[meufeu]

Bonjour,

J'ai un code C++ qui lit dans un fichier. Je sais exactement ce que je dois trouver et l'ordre, aussi, je parcours ligne par ligne, si je trouve bien "mon entier=", je lis la ligne dessous et j'affecte à mon entier la valeur lue ... etc ..
Cela fonctionne très bien mais le code est très redondant.

Etant débutant en C++, je n'ai pas réussi à le factoriser :

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
  int monentier;
  double mondouble;
  char monchar[100]
  fichier.open("file.in", ios::in);
  fichierRead.open("file.in.read");
  getline( fichier, ligne ) ;
  fichierRead<<ligne<<endl;
  cout << "#1 ligne >>"<< ligne <<"<<" <<endl; 
  if ( ligne == " monentier= " )
  {
  	fichier>>monentier;
	getline( fichier, ligne ) ;
	fichierRead<<monentier<<endl;
  } 
  //
  getline( fichier, ligne ) ;
  fichierRead<<ligne<<endl;
  cout << "#2 ligne >>"<< ligne <<"<<" <<endl;
  if ( ligne == " monchar= " )
  { 
	fichier>>monchar;
	getline( fichier, ligne ) ;
	fichierRead<<monchar<<endl;
  }
 
...

J'ai bien évidemment fait des recherches et trouvé quelques pistes du coté
des templates par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
template<typename T>
void readAndWrite(T& cible, fichier, fichierRead )
{
  string ligne;
  getline( fichier, ligne ) ;
  fichierRead<<ligne<<endl; 
  if ( ligne == pattern )
  {
  	fichier>>cible*;
	getline( fichier, ligne ) ;
	fichierRead<<cible*<<endl;
  } .
}

mais cela ne fonctionne pas, je pense que je ne suis pas très loin mais je n'arrive pas à faire fonctionner la chose.

Pourriez-vous m'aider s'il vous plait ?
Merci d'avance

[Laurent Gomila]

Il y a de l'idée

Ceci devrait fonctionner :

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
template<typename T>
void readAndWrite(std::ifstream& Entree, std::ofstream& Sortie, const std::string& Pattern)
{
  string ligne;
  getline( Entree, ligne ) ;
  Sortie<<ligne<<endl; 
  if ( ligne == Pattern )
  {
        T cible;
  	Entree>>cible;
	getline( Entree, ligne ) ;
	Sortie<<cible<<endl;
  }
}
 
readAndWrite<char>(fichier, fichierRead, " monchar= ");
readAndWrite<int>(fichier, fichierRead, " monentier= ");
...

[meufeu]

merci beaucoup pour la réponse ...

par contre, je voudrais pouvoir appeler cette méthode
en lui disant quelle est la cible ...

exemple (qui ne compile pas)

Code :

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

1
2
int mon_entier = 0;
mon_entier = readAndWrite<int>(fichier, fichierRead, " monentier= ");

merci encore !

[Laurent Gomila]

Ok, autant pour moi.

Code :

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

1
2
3
4
5
template<typename T>
void readAndWrite(T& cible, std::ifstream& Entree, std::ofstream& Sortie, const std::string& Pattern)
 
readAndWrite(mon_char, ...);
readAndWrite(mon_entier, ...);

Passer ton retour par référence t'évite de spécifier explicitement le type template.

[roulious]

Code :

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

1
2
template<typename T>
void readAndWrite(T& cible, std::ifstream& Entree, std::ofstream& Sortie, const std::string& Pattern);

Tant qu'on fait du code generique, autant utiliser std::istream et std::ostream plutôt que leur pendants travaillant sur des fichiers.

Et rajouter un traitement d'erreur (genre retourner un bool si tout s'est bien passé) peut être une bonne idée, afin de savoir si cible contient bien ce que l'on veut.

[meufeu]

d'abord merci !

oui ce serait bien de traiter les erreurs car justement, j'en ai ..
quand j'affiche mon ifstream il me met juste "0" ...

alors qu'il y a plein de choses dans mon fichier ...

je vous remercie
!

[Médinoc]

Laurent Gomila : Me tromperais-je ou ton code est-il incapable de lire ceci:

Code :

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

blabla = 1

En voyant les getline(), j'ai l'impression que le texte doit absolument être ainsi pour être lisible:

Code :

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

1
2
blabla =
1

[meufeu]

Tout à fait exact mais c'est bien comme cela que le fichier que je dois lire est présenté ...

Sinon pour mon problème, cela venait du fait que je lancais mon exécutable
comme cela :

./bin/exe

et le fichier étant dans bin, il faut que je me place dans bin puis que je lance l'exe ..

MERCI à tous !