Factoriser le code (via template<typename T> ?)

Version imprimable

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:

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:

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

Il y a de l'idée ;)

Ceci devrait fonctionner :

Code:

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= ");
...

31/07/2006, 17h36
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:

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

Ok, autant pour moi.
Code:

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.

31/07/2006, 18h02
roulious
Citation:
Envoyé par Laurent Gomila

Code:

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.
31/07/2006, 23h19
meufeu

Merci

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
!
01/08/2006, 10h08
Médinoc
Laurent Gomila : Me tromperais-je ou ton code est-il incapable de lire ceci:

Code:

blabla = 1

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

1 2 blabla = 1
01/08/2006, 11h58
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 !