ifstream, d�rivation ou encapsulation ?

Salut � tous et bonne ann�e :D !

Voila, j'ai un petit probl�me, petit car je pourrai passer par des moyens d�tourn�. Je vous explique, j'ai un fichier configuration � charger, simple fichier texte listant un certain nombre de donn�es dans un certain ordre. J'aimerais y ajouter des commentaires, pr�c�d� d'un '#' par exemple. Jusque l�, il n'y a pas de soucis.

Oui, mais voila, je voudrai que �a soit compl�tement transparent � l'utilisation, je souhaiterai pouvoir faire des "file>>value" qui ignorerai automatiquement les commentaires. Pour cela, deux solutions me sont venus � l'esprit, l'encapsulation tout d'abord ou la d�rivation ensuite.

la d�rivation par exemple :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

class CFileConfig : public std::ifstream { public: CFileConfig(const char * filename):std::ifstream(filename){}; ~CFileConfig(); };   /* plus loin */ CFileConfig file("file.txt"); int var1,var2; file>>var1>>var2;

---

Mon probl�me est donc surd�finir l'op�rateur >> afin qu'il garde la capacit� d'�crire dans n'importe quel type de donn�e tout en ignorant les lignes commencant par un #...

Merci d'avance pour vos pistes :)

Edit: Oula, je viens de me rendre compte que ce message risque de ne pas �tre au bon endroit� :oops:

Tu dois composer. En effet, un ifstream est un basic_ifstream<char> et basic_ifstream d�rive de basic_istream. Dans ces deux classes, aucune m�thode n'est virtuelle en particulier les op�rateurs >> (sauf le destructeur de basic_istream).
Tu ne pourras donc pas profiter des m�canismes de virtualit�.
Tu dois donc composer et non d�river.

J'avais constat� effectivement, mais qu'entend tu par "composer", je r��cris tout les op�rateurs >> ?

Sinon, j'avais peut �tre pens� �, � l'ouverture du fichier, lire et recr�er un flux sans les commentaires et donc finalement ouvrir ce dernier, une voie � explorer? Je vais essayer...

Donc voila, en d�finitive je suis pass� par un fichier temporaire

FileConfig.h

Code:

---

1 2 3 4 5 6 7 8 9 10 11

#define TEMP_EXT ".temp" class CFileConfig : public std::ifstream { public: CFileConfig(const char * filename); ~CFileConfig(); private: char * m_tempFileName;   };

---

FileConfig.cpp

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

CFileConfig::CFileConfig(const char * filename):std::ifstream(filename) { /* save temporary file name */ m_tempFileName = new char[strlen(filename) + strlen(TEMP_EXT)]; strcpy(m_tempFileName,STR.sprintf_rs("%s"TEMP_EXT"",filename));   std::ofstream tempFile(m_tempFileName); ASSERT(tempFile);   /* copy the file in new file */ std::string line; while(std::getline(*this, line)) { if(line.length()>0) if(line[0]!='#') tempFile<<line<<std::endl; }   /* close previous file, open temparary file */ this->close(); this->clear(); tempFile.close(); this->open(m_tempFileName); }   CFileConfig::~CFileConfig() { this->close(); this->clear(); /* delete temporary file */ std::remove(m_tempFileName); }

---

Petite note : mon "STR.sprintf_rs" est en fait un sprintf sur des chaines de caract�res temporaires.

Qu'en pensez vous ? C'est vrai que �a fait une ecriture de fichier, mais c'est une op�ration r�alis� uniquement au chargement... D'ailleurs, je n'ai pas trouv� un type ofstream qui n'ecrirai pas dans un fichier mais seulement en memoire...

J'ai retravaill� ton code ainsi pour composer avec un ifstream. Vu ce que tu en fait, il est inutile de le d�river (en plus des raisons que j'ai dit hier).
Par contre, je ne suis pas pourquoi tu rouvres ton fichier en lecture seul � la fin. Tu ne peux plus rien �crire dedans... J'aurais plut�t compos� avec un ofstream. A vrai dire, je ne vois pas ce que tu veux faire, pourquoi tu parles d'un ofstream qui n'�crirait qu'en m�moire...

Autrement,

• �viter les char * en C++, alors qu'on dispose des strings, �a �vite d'utililiser les strcpy, de se planter dans les tailles de chaines et d'�crire un STR.sprintf_rs inutile
• �viter les macros en C++, pr�f�rer d�clarer des constantes
• pas de macro ASSERT sur une erreur qui peut toujours appara�tre � l'ex�cution, en effet, une fois que ton assert sera d�sactiv� (par NDEBUG), tu ne pourras plus d�tecter une erreur d'ouverture de fichier

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 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70

#include <cassert> #include <fstream> #include <iostream> #include <stdexcept> #include <string>   const std::string TEMP_EXT = ".temp";   class invalid_file : public std::runtime_error { public: explicit invalid_file (const std::string & args); };   invalid_file::invalid_file (const std::string & args) : std::runtime_error (args) {}   class CFileConfig { public: CFileConfig(const std::string &); ~CFileConfig(); private: std::ifstream m_configFile; std::string m_tempFilename; };   CFileConfig::CFileConfig(const std::string & filename) : m_configFile (filename.c_str()), m_tempFilename (filename + TEMP_EXT) {   if ( ! m_configFile ) throw invalid_file ("Can't open " + filename);   std::ofstream tempFile(m_tempFilename.c_str());   if ( ! tempFile ) { m_configFile.close (); throw invalid_file ("Can't open " + m_tempFilename); }   /* copy the file in new file */ std::string line; while(std::getline(m_configFile, line)) { if(line.length() > 0) { if(line[0] != '#') { tempFile << line << std::endl; } } }   /* close previous file, open temporary file */ m_configFile.close (); tempFile.close (); m_configFile.open (m_tempFilename.c_str());   }   CFileConfig::~CFileConfig() { m_configFile.close(); /* delete temporary file */ std::remove(m_tempFilename.c_str()); }   int main () { try { CFileConfig config ("if.txt"); } catch (std::exception & ex) { std::cerr << ex.what () << std::endl; } return 0; }

---

Merci pour cette correction, � vrai dire, � propos de l'utilisation des char * pr�alou�, je les utilise car je tourne sur un syst�me tr�s limit� (une esp�ce de console portable) o� les std::string sont assez lent.

De la m�me mani�re, les exceptions me paraissent plus s�re mais encore une fois, pour un soucis d'optimisation, je me suis r�duit aux asserts (d'autant plus que c'est impos� par mon cahier des charges)...

Sinon, j'ouvre en lecture seule car je n'ai justement pas besoin de r��crire dans le fichier. Enfin, je parlais d' "ofstream en memoire" en esperant trouver un �quivalent de ofstream qui n'�crirai pas sur le disque dur, je ne sais pas vraiment si cela existe...

En tout cas, merci beaucoup pour ces indications! �a me lib�re du flou dans lequel je navigue parfois en utilisant std...

EDIT : Deux petites erreures dans mon code d'origine qui est loin d'�tre une r�f�rence d'ailleurs :lol: :

Code:

---

1 2 3

m_tempFileName = new char[strlen(filename) + strlen(TEMP_EXT) + 1]; /* in the destuctor */ delete [] m_tempFileName;

---