Enregistrer un tableau dans un fichier

Question

Bonjour ! Je développe une application en mode console en C++. Voici le thème : je veux gérer des clients qui réalisent plusieurs abonnements à un magazine. J'ai donc créé une classe C_client et une classe C_abonnement. Je sauvegarde mes données dans un fichier lorsque l'utilisateur saisit le sous menu 5 pour sortir de l'application. Or, la sauvegarde d'un client sans abonnements ne pose aucun problème. Seulement lorsque je lui ajoute un abonnement (chaque client possède un tableau d'abonnements), l'enregistrement des données quant à cet abonnement se trouvent erronées. Voici ma procédure d'enregistrement de fichiers : #include #include #include void enregFichier() { int i; // si le fichier est lisible... if ( is_readable( "abonnements.txt" ) ) { // ...alors on écrit dedans en effacant tout std::ofstream file( "abonnements.txt" ); for (i=0;i

luhtor · Answer

C'est quoi tabClient, dans le sens, tu le construit comment, ca contient quoi ?

car je crains que l'erreur vienne de la.

Poste les lignes de codes concernant tabClient.



J'ai l'impression que tu te m&#233;langes un peu pour &#233;crire fans les fichiers.

Ce qui m'&#233;tonne c'est que tu utilises deux librairies diff&#233;rentes en meme temps:

      std::ofstream file( "abonnements.txt" ); 
      for (i=0;i<MAXIMUM;i++) 
      { 

         fwrite(&tabClient[i], sizeof(tabClient[i]), 1, monFichier); 
      }; 



Ce serait pas plutot:



    
std::ofstream file("abonnements.txt");

    for (int i=0; i<MAXIMUM; i++)

       file << tabClient[i] << endl;



tu utilises fwrite qui vient le stdio.h il me semble. En tout cas, il vaut mieux utiliser la syntaxe: file << 

bien adapter au format texte.

cyberkate · Answer

Voici la d&#233;claration de tabClient, il contient tous mes clients de la classe C_client

const int MAXIMUM = 100;
C_client tabClient[MAXIMUM];

Avant j'utilisais la fonction suivante pour enregistrer dans un fichier :

monFichier = fopen("abonnements.txt", "w"); // ouverture du fichier en &#233;criture


 if (monFichier == NULL)
 {
  cout << "Erreur lors de l'ouverture du fichier Abonnements." << endl;
 }
 else
 {
  int i;


  for (i=0;i<MAXIMUM;i++)
  {
   tmp = fwrite(&tabClient[i], sizeof(tabClient[i]), 1, monFichier); // on enregistre ligne par ligne dans le fichier


   if (!tmp)
   {
    cout << "Erreur d'ecriture." << endl;
   };
   //else
   //{
   // cout << tabClient[i].getNumClient() << endl; 
   //};
  };
  fclose(monFichier);
 };

Cependant, d&#232;s que j'enregistrais un abonnement pour un client, mon fichier n'enregistrait aucune donn&#233;es.

La fa&#231;on de mon premier post peut elle enregistrer des donn&#233;es de type C_client (m&#234;me s'il y a des integer, etc dans la classe) ?

luhtor · Answer

Bon je commence a comprendre, mais comment veux tu que ca marche ca:

std::ofstream file( "abonnements.txt" ); 
      for (i=0;i<MAXIMUM;i++) 
      { 

         fwrite(&tabClient[i], sizeof(tabClient[i]), 1, monFichier); 
      }; 




&tabClient[i],
ca c'est l'adresse en m&#233;moire de ton client, mais pas l'objet lui meme.
DOnc au mieux ce que tu &#233;cris c'est l'adresse, qu'on se fou absolument.



Donc j'utilise ma version qui est plus lisible: ton code donnerait:





    
std::ofstream file("abonnements.txt");

    for (int i=0; i<MAXIMUM; i++)
   
   file << "N&#176;: " tablClient[i].num_client << " |
Nom : " << tabClient[i].nom << ...etc... << endl;



Il y a plus &#233;labor&#233; et plus lisible:

Tu ajoutes cette fonction en dessous de ta classe (c'est pas une fonction membre)



inline std::ostream & operator << (std::ostream & chaine, const C_client & client)

    {

    return chaine << 
"N&#176;: " client.num_client << " | Nom : " << client.nom; // tu ajoutes ce que tu veux apr&#232;s


    }



Puis apr&#232;s l'&#233;criture dans ton fichier devient simplement:





    
std::ofstream file("abonnements.txt");

    for (int i=0; i<MAXIMUM; i++)

       file << tabClient[i] << endl;



file.close();



++

steve_clamage · Answer

Le fwrite enregistre l'objet en binaire et non son adresse, en C++ on utiliserais plutot std::ostream::write.



cyberkate, pour faire propre il faudrait que ce soit tes classes qui
fournissent une methode de serialisation (= sauvegarde d'une instance
dans un fichier) car la fa&#231;on d'enregistr&#233; une instance d&#233;pends des
attributs (si pointeur sur tableau dynamique par exemple).



class C_client
{



public:



void enregistre( std::ostream & o ) const

{

o.write( reinterpret_cast<const char *>(&num_client), sizeof num_client );

// comme ca pour tout les membres

}



};




std::ofstream file("abonnements.txt", std::ios::binary);
for (int i=0; i<MAXIMUM; i++)

    
tabClient[i].enregistre(file);

cyberkate · Answer

Pour la méthode de steve_clamage, j'ai le problème suivant : tous mes champs qui ne sont pas de type chaine ne peuvent être convertis en chaine... Voici ce que j'ai mis comme code : void enregistre( std::ostream & o ) const { o.write( reinterpret_cast(&num_client), sizeof num_client ); o.write( reinterpret_cast(&nom), sizeof nom ); o.write( reinterpret_cast(&prenom), sizeof prenom ); o.write( reinterpret_cast(&adresse), sizeof adresse ); o.write( reinterpret_cast(&cp), sizeof cp ); o.write( reinterpret_cast(&ville), sizeof ville ); o.write( reinterpret_cast(&telephone), sizeof telephone ); o.write( reinterpret_cast(&nb_abonnements), sizeof nb_abonnements ); o.write( reinterpret_cast(&tabAbonnements), sizeof tabAbonnements ); }; Quant à la méthode de luhtor, j'ai essayé avec ce code dans la procédure enregFichier : int i; // si le fichier est lisible... if ( is_readable( "abonnements.txt" ) ) { std::ofstream file("abonnements.txt"); for (int i=0; i

luhtor · Answer

tabClient[i].getPrenom()



avec ca:



void getPrenom()
   {
    cout << prenom;
   };


oui en effet, ca ne peut pas marcher puisque ta fonction ne retourne rien.

Ta pas essayer ma deuxi&#232;me solution qui ne devrait pas poser de
probl&#232;me, mais il faudrait la d&#233;clarer friend car tu as des donn&#233;es
private.



Bref, ta fonciton getPrenom

J'utiliserais des string, ca donnerait:



std::string getPrenom()

{

return std::string(this->prenom);

}



Mais selon moi tu aurais du utiliser des string et non des char * pour tes donn&#233;es de ta classe.

cyberkate · Answer

Je pense que cela doit se voir que je d&#233;bute vraiment en C++ ! lol

J'arrive pas du tout &#224; faire ce que tu me dis alors que je fais du copier coller en essayant d'adapter le code. J'obtiens une bonne dizaine d'erreurs... 

J'ai du mal &#224; comprendre comment faire pour enregistrer donn&#233;e par donn&#233;e dans mon fichier afin de les relire lors du prochain lancement de l'application.

J'ai mis mes fichiers en ligne si &#231;a vous aide &#224; comprendre mon probl&#232;me : ils ne contienent aucune modification par rapport &#224; mon post initial.
Fichiers .cpp et .h

Je continue &#224; chercher en attendant votre aide, car je suis vraiment perdue et que je dois pr&#233;senter ce projet en examen...

Merci d'avance pour vos autres explications.

cyberkate · Answer

J'ai rajout&#233; ce code dans le public de ma classe C_client (si je le mets en dessous, il y a une erreur demandant de mettre la fonction friend &#224; l'int&#233;rieur d'une classe) :

friend inline std::ostream & operator << (std::ostream & chaine, const C_client & client)
  {
   return chaine << client.num_client << " " << client.nom << " " << client.prenom << " " << client.adresse << " " << client.cp << " " << client.ville << " " << client.telephone; 
  };

Ensuite, voici le contenu de ma fonction enregFichier :

if ( is_readable( "abonnements.txt" ) ) 
    { // ...alors on &#233;crit dedans en effacant tout
  std::ofstream file("abonnements.txt");
  for (int i=0; i<MAXIMUM; i++)
  {
   file << tabClient[i] << endl;
  };
  file.close();
 };

Voici le nouveau probl&#232;me : il n'y a pas "d'erreur" lors de l'&#233;criture mais lorsque je r&#233;ouvre l'application, les donn&#233;es sont affich&#233;es avec plein de fois 0040119F0 entre et apr&#232;s....

Voici le code qui charge les donn&#233;es dans mon tableau &#224; l'ouverture de l'application :
void chargeClient()
{
monFichier = fopen("abonnements.txt", "r+"); //ouverture du fichier en lecture


  if (monFichier == NULL)
  {
   cout << "Erreur d'ouverture du fichier Abonnements." << endl;
  }
  else
  {
   int i;
   for (i=0;i<MAXIMUM;i++)
   {
    tmp = fread(&clientEnCours, sizeof(clientEnCours), 1, monFichier); // on lit une ligne par une ligne


    if (!tmp) // si la ligne est vide
    {
     cout << "Erreur de lecture." << endl;
    }
    else
    {
     tabClient[i] = clientEnCours;
    };
   };


   fclose(monFichier); // fermeture du fichier
  };
};

D'o&#249; vient le probl&#232;me ?...

luhtor · Answer

Cette fonction la marche tr&#232;s bien, mais ca doit etre une erreur simple. Donne les erreurs que tu obtiens.

std::string getPrenom()

{

return std::string(this->prenom);

}



Et pr&#233;cises comment tu l'utilises ensuite.

C'est une obligation de ton projet d'utiliser des char * ou c'est toi qui choisit ?

cyberkate · Answer

Pour ce qui est des char *, je ne pense pas que ce soit une obligation, mais c'est qu'on a appris que comme &#231;a. Donc j'ai repris le "cours" et ce que j'avais &#224; peu pr&#232;s compris.

En laissant les fonctions de mon dernier post et en changeant les char * en std::string, je n'ai pas d'erreurs dans le d&#233;buggueur, mais les donn&#233;es dans le fichier ne sont pas correctes.

J'utilise cette fonction comme ceci : tabClient[i].getNom(); pour afficher le nom...

J'ai mes donn&#233;es et plein de 0040119F0     0040119F0      0040119F0...

Donc je ne comprends pas... peut-&#234;tre que je les lis mal... 
Merci de ton aide.

luhtor · Answer

:)

Poste ta fonction qui enregistre dans un fichier, car ce que tu as ce
sont des adresses m&#233;moires donc quelque part tu &#233;cris un pointeur et
nom un string.

cyberkate · Answer

Voici ma fonction qui enregistre dans un fichier :

void enregFichier()
{
 int i;


 // si le fichier est lisible...
 if ( is_readable( "abonnements.txt" ) ) 
    { // ...alors on &#233;crit dedans en effacant tout
  std::ofstream file("abonnements.txt");
  for (int i=0; i<MAXIMUM; i++)
  {
   file << tabClient[i] << endl;
  };
  file.close();
 };
};

Ce qui est bizarre, c'est que &#231;a enregistre mes donn&#233;es ainsi que les adresses m&#233;moires.

Il est &#224; noter que :

- je ne sais pas comment int&#233;grer mon tableau d'abonnements dans la fonction friend :
friend inline std::ostream & operator << (std::ostream & chaine, const C_client & client)
  {
   return chaine << client.num_client << " " << client.nom << " " << client.prenom << " " << client.adresse << " " << client.cp << " " << client.ville << " " << client.telephone; // comment int&#233;grer le tableau d'abonnements ?
  };

- je n'arrive pas &#224; lire les donn&#233;es (nom, prenom, adresse, etc) avec ma proc&#233;dure afficheInfo avec dedans :
tabClient[i].getNom();
etc.

luhtor · Answer

Tu fais bien cout << tabClien[i].getNom() << endl;



avec getNom() qui retourne un string ?


Pour tes abonnements:


friend inline std::ostream & operator << (std::ostream & chaine, const C_client & client)
  {
   chaine << client.num_client << " " << client.nom
<< " " << client.prenom << " " <<
client.adresse << " " << client.cp << " " <<
client.ville << " " << client.telephone; 



for (int i = 0; i<nombre d'abonnements; i++)

{

    chaine << client.abonnements[i].nom de l'abonnement << " " << d'autre truc << endl;

}

    return chaine;
  };

cyberkate · Answer

Si je fais cout << tabClient[i].getNom() << endl; dans ma fonction afficheInfo(), j'ai l'erreur suivante de compilation : error C2678: binary '<<' : no operator defined which takes a left-hand operand of type 'class ostream_withassign' (or there is no acceptable conversion) Et getNom() retourne un string : std::string getNom() { return std::string(this->nom); }; Et si je mets la fonction : friend inline std::ostream & operator << (std::ostream & chaine, const C_client & client) { chaine << client.num_client << " " << client.nom << " " << client.prenom << " " << client.adresse << " " << client.cp << " " << client.ville << " " << client.telephone; // tu ajoutes ce que tu veux après for (int i = 0; i< client.nb_abonnements; i++) { chaine << client.tabAbonnements[i].getNumAb() << " " << client.tabAbonnements[i].getLibelle() << endl; }; }; celà me fait 2 erreurs de même type : - error C2228: left of '.getNumAb' must have class/struct/union type - error C2228: left of '.getLibelle' must have class/struct/union type Je ne peux "accéder" aux données privées relatives aux abonnements que par l'intermédiaire de fonctions : int getNumAb() { return numAb; }; std::string getLibelle() { return std::string(this->libelle); };

luhtor · Answer

Ca devrait marcher. Ta bien mis #include #include #include using namespace std; en entete ?

cyberkate · Answer

J'ai bien mis les biblioth&#232;ques iostream, string, ftream en en-t&#234;te.

Par contre, pour ce qui est du using namespace std; , si je le mets en en-t&#234;te, j'ai 59 erreurs car, apparemment, le cout et le cin ne sont pas accept&#233;s :

error C2872: 'cout' : ambiguous symbol
error C2872: 'cin' : ambiguous symbol

Comment faire pour mettre le using namespace std; sans avoir les erreurs concernant le cout et le cin

PS : voici un lien avec les fichiers .cpp et .h, au cas o&#249;.

luhtor · Answer

Le prend pas mal, mais j'ai halluciné :) Faut surtout pas mettre iostream et iostream.h ainsi que string et string.h. donc en entete de main tu mest ca: #include #include #include #include #include using namespace std; #include "classes.h" et ca marche impec. enfin j'ai vu un bug quand j'ai essayer de créer un abonnement De plus dans classe.h, for (int i = 0; i< client.nb_abonnements; i++) { chaine << client.tabAbonnements[i]->getNumAb() << " " << client.tabAbonnements[i]->getLibelle() << endl; //chaine << client.abonnements[i].num_ab << " " << d'autre truc << endl; }; Ce sont des flèches qu'il faut mettre, puisque ce sont des pointeurs qui accèdent aux données, donc regarde ou j'ai mis les flèches juste avant.

cyberkate · Answer

Oh je ne le prends pas mal. Cela me semblait bizarre de mettre autant de biblioth&#232;ques avec les m&#234;mes noms, mais bon... lol

J'ai corrig&#233; ce que tu m'as dit. 
Je ne savais pas que c'&#233;tait des pointeurs qui acc&#233;daient aux donn&#233;es (d'ailleurs, on le voit comment ??)

Sinon j'ai test&#233; l'application : l'inscription dans le fichier se fait bien... par contre je n'arrive pas &#224; relire les informations...

Y-a-t-il quelquechose qui ne va pas dans ma fonction chargeClient ??
Je ne sais pas si je remplis bien mon tableau tabClient et si je le lis bien...

Peux tu m'aider sur ce point ??
Merci beaucoup

Je te re-poste ma fonction chargeClient() au cas o&#249;...

monFichier = fopen("abonnements.txt", "r+"); //ouverture du fichier en lecture


  if (monFichier == NULL)
  {
   cout << "Erreur d'ouverture du fichier Abonnements." << endl;
  }
  else
  {
   int i;
   for (i=0;i<MAXIMUM;i++)
   {
    tmp = fread(&clientEnCours, sizeof(clientEnCours), 1, monFichier); // on lit une ligne par une ligne


    if (!tmp) // si la ligne est vide
    {
     cout << "Erreur de lecture." << endl;
    }
    else
    {
     tabClient[i] = clientEnCours;
    };
   };


   fclose(monFichier); // fermeture du fichier
  };

luhtor · Answer

Est ce que tu peux reposter :) car la ca devient p&#233;nible d'acc&#233;der &#224; ce post ^^



De m&#233;moire pour ta question: c'est des pointeurs on le voit comment ?

Et bien c'est toi qui a d&#233;finit ca comme ca:

un truc du genre:

Abonnements *abonnement[MAX] <===== ca c'est un tableau de pointeur.
Donc il contient que des pointeurs et quand tu y fais r&#233;f&#233;rence, c'est
un pointeur.

cyberkate · Answer

Merci pour ton aide ! 
Je cr&#233;e un autre post pour le probl&#232;me de lecture des donn&#233;es.

@+

Enregistrer un tableau dans un fichier

20 réponses

Votre réponse

Discussions similaires