Problème template c++

Problème template c++ - C++ - Programmation

Marsh Posté le 05-11-2011 à 21:28:20    

Bonjour à tous et à toutes,
 
Je rencontre un problème que je ne parviens pas à résoudre avec les templates.
 
En fait, j'ai une classe template Vecteur (qui agit comme un vecteur...). Elle agit normalement lorsque le type est un entier, par contre, dés qu'il s'agit d'une classe, on dirait que la fonction membre d'affichage des éléments oublie le type (alors que si je l'affiche élément par élément tout se passe bien).
 
J'utilise g++ v3.4.3 sur Solaris 10.
 
Voici le code de la classe Vecteur (header et cxx)  

Code :
  1. #ifndef VECTEUR_H
  2. #define VECTEUR_H
  4. #include <stdio.h>
  5. #include <stdlib.h>
  6. #include <iostream.h>
  7. #include <string.h>
  8. #include "Structure.h"
  10. #include "ExceptionBase.h"
  12. template <class type>
  13. class Vecteur : public Structure<type>
  14. {
  15. private:
  16.  type* tab;
  17.  int* VecPresence;
  18. public:
  19.  Vecteur();
  20.  Vecteur(int TmpNbrElements);
  21.  type& getElement(int TmpIndice);
  22.  void setElement(int TmpIndice,const type& TmpObj);
  23.  type retireElement(int TmpIndice);
  24.  bool estOccupe(int TmpIndice);
  25.  bool indiceValide(int TmpIndice);
  26.  void affiche();
  27.  Vecteur<type> operator+(const type& TmpObj);
  29.  int PremierLibre();
  30.  ~Vecteur();
  32. };
  36. #endif


 

Code :
  1. #include "Vecteur.h"
  3. template <class type>
  4. Vecteur<type>::Vecteur()
  5. {
  6. Structure<type>::setNbrElements(0);
  7. }
  9. template <class type>
  10. Vecteur<type>::Vecteur(int TmpNbrElements)
  11. {
  12. int i=0;
  13. Structure<type>::setNbrElements(TmpNbrElements);
  14. tab = new type[Structure<type>::NbrElements];
  15. VecPresence = new int[Structure<type>::NbrElements];
  16. while(i < Structure<type>::NbrElements)
  17. {
  18.  *(VecPresence +i) = 0;
  19.  i++;
  20. }
  21. }
  25. template <class type>
  26. type& Vecteur<type>::getElement(int TmpIndice)
  27. {
  28. if(indiceValide(TmpIndice) && estOccupe(TmpIndice)) {
  29.            
  30.             return *(tab+TmpIndice);
  31.            }
  32. else throw ExceptionBase("Case non occupee ou indice invalide" );
  33. }
  37. template <class type>
  38. void Vecteur<type>::setElement(int TmpIndice,const type& TmpObj)
  39. {
  40. if(indiceValide(TmpIndice))
  41. {
  42.     *(tab + TmpIndice) = TmpObj;
  44.  *(VecPresence + TmpIndice) = 1;
  46. }
  47. else throw ExceptionBase("Indice invalide" );
  48. }
  50. template <class type>
  51. type Vecteur<type>::retireElement(int TmpIndice)
  52. {
  53. if(indiceValide(TmpIndice) && estOccupe(TmpIndice))
  54. {
  55.  type TmpObj;
  56.  TmpObj = *(tab + (TmpIndice));
  57.  *(VecPresence + (TmpIndice))=0;
  58.  return TmpObj;
  59. }
  60. else throw ExceptionBase("Indice invalide ou case inoccupee" );
  61. }
  63. template <class type>
  64. bool Vecteur<type>::estOccupe(int TmpIndice)
  65. {
  66. bool ok = indiceValide(TmpIndice);
  67. if(ok)
  68. {
  69.  if(*(VecPresence + (TmpIndice)) == 1) return true;
  70.  else return false;
  71. }
  72. else throw ExceptionBase("Indice invalide" );
  73. }
  75. template <class type>
  76. bool Vecteur<type>::indiceValide(int TmpIndice)
  77. {
  78. if((TmpIndice <= Structure<type>::NbrElements-1) && TmpIndice >= 0) return true;
  79. else return false;
  80. }
  82. template <class type>
  83. void Vecteur<type>::affiche()  //le problème viendrait apparement de cette méthode
  84. {
  85. int Indice=0;
  86. while(Indice<(Structure<type>::NbrElements))
  87. {     
  88.  cout<<Indice<<". "<<endl;
  89.  cout<<tab[Indice]<<endl;    //Toutes les classes utilisées implémentent une surcharge de << qui fonctionne
  91.  cout<<"------------------------------------------"<<endl;
  93.  Indice++;
  94. }
  95. }
  97. template <class type>
  98. Vecteur<type> Vecteur<type>::operator+(const type& TmpObj)
  99. {
  100. int TmpIndice= 0;
  101. TmpIndice = PremierLibre();
  102. setElement(TmpIndice, TmpObj);
  103. return *this;
  104. }
  107. template <class type>
  108. int Vecteur<type>::PremierLibre()
  109. {
  111. int TmpIndice = 0;
  112. while((TmpIndice < Structure<type>::NbrElements) && (*(VecPresence + TmpIndice) != 0))
  113. {
  114.  TmpIndice++;
  115. }
  116. if (TmpIndice >= Structure<type>::NbrElements) throw ExceptionBase("Vecteur plein" );
  117. else return TmpIndice;
  118. }
  120. template <class type>
  121. Vecteur<type>::~Vecteur()
  122. {
  123. if(tab)
  124. {
  125.  delete []tab;
  126.  delete []VecPresence;
  127. }
  128. }
  130. template class Vecteur<int>;
  131. #include "Carte.h"
  132. template class Vecteur<Carte>;
  133. #include "Collectionneur.h"
  134. template class Vecteur<Collectionneur>;
  135. #include "ConcepteurAlbum.h"
  136. template class Vecteur<ConcepteurAlbum>;


 
La classe template Structure n'est qu'une classe générique, je ne pense pas qu'il soit possible que le problème vienne de là.
Voici le résultat (l'endroit ou ça casse).
 
Personne
Nom : Coptere                                         //test d'une des classe, ici, c'est déjà un problème puisqu'apparement il affiche
Prenom : Eli                                          //le type comme si il s'agissait d'une des classes mère de ma classe concernee
Date de naissance : 25/03/1990
Nationalite : francais
------------------------------------------
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX   //Cette ligne est dans le main()
Collect                                              //affichage qui sert à tracer la surcharge du << dans chaque classe de la  
Personne                                             //Hièrarchie
Nom : Sonite
Prenom : Sam
Date de naissance : 11/01/1992
Nationalite : belge
Author
Login : sonitesa
Nombre de Collections : 6
On ajoute Sonite en utilisant l'operateur +
Author
0.
Personne
Nom :                                //S'arrête ici
 
 
Voilà, ici (mais c'est pareil avec les autres classes) il s'agit d'une hièrarchie ;
 
class Personne (Classe mère)
class AuthorisedPerson : public Personne (Classe abstraite)
class Collectionneur : public AuthorisedPerson
class ConcepteurAlbum : public AuthorisedPerson
 
Je suis désolé de ne pas pouvoir rendre le problème plus général avec un code plus court. Je pense qu'il doit s'agir d'un cas assez particulier.
 
Voilà, en espèrant que vous pourrez m'aider. Si besoin, je peux ajouter les valeurs que peuvent prendre le type (le code des différentes classes).
J'ajoute qu'il s'agit d'un cas d'école et que je suis obligé de travailler comme ça.
 
Je vous remercie d'avance.
 
Bonne soirée.
 
Christallis

Reply

Marsh Posté le 05-11-2011 à 21:28:20   

Reply

Marsh Posté le 05-11-2011 à 23:52:57    

donne ton main.
 
Un template doit être défini aussi dans le .h et non pas dans un .cpp
 
Edit : à la lecture de ton code, j'ai comme l'impression que tu espères que l'opérateur << pourra bénéficier du polymorphisme. Ce n'est pas le cas. Donne aussi quelques exemples de classes que tu utilises, comme Carte, Collectionneur, ConcepteurAlbum et les classes dérivées s'il y en a,et montre nous, dans ton main donc, les appels que tu fais.


Message édité par theshockwave le 05-11-2011 à 23:55:40

---------------
last.fm
Reply

Marsh Posté le 06-11-2011 à 10:56:48    

Bonjour,
 
Tout d'abord, merci pour ta réponse.
 
Oui, en effet normalement le code d'un template doit être dans le header mais j'ai essayé de tout mettre dans le header (et aussi (même si c'est pareil) d'inclure le cxx à la fin du header) et ça ne change absolument rien à l'exécution.
 
Voici le main (enfin, la partie concernée qui est faite dans un fonction void).
 

Code :
  1. void main()
  2. {
  3.   cout << "-----  Test du template Vecteur avec des objets de la classe Collectionneur ------" << endl;
  4.   cout << "Creation  d'un vecteur de 5 cases..." << endl;
  5.   Vecteur<Collectionneur> *p = new Vecteur<Collectionneur>(5);
  6.   p->affiche();               
  7.   cout << endl;
  8.  
  9.   Collectionneur c1("Tombale","Pierre","07/10/1985","belge","tombalpi",3);
  10.   Collectionneur c2("Coptere","Eli","25/03/1990","francais","copterel",4);
  12.   cout << "L'element d'indice 2 devient Tombale" << endl;
  13.   if (p->indiceValide(2)) p->setElement(2,c1);
  14.   p->affiche();                                 
  15.   cout << endl;
  16.  
  17.   cout << "L'element d'indice 4 devient Coptere" << endl;
  18.   if (p->indiceValide(4)) p->setElement(4,c2);
  19.   p->affiche();                                 
  20.   cout << endl;
  22.   cout << "On recupere l'element d'indice 2 (sans le retirer !!!)" << endl;
  23.   if (p->indiceValide(2) && p->estOccupe(2))
  24.   {
  26.     Collectionneur c=p->getElement(2);   
  28.    
  29.     cout << "Element recupere = " << c << endl;
  30.   }
  31.   else cout << "Case non occupee ou indice invalide !" << endl;
  32.   cout << endl;
  34.   cout << "On recupere l'element d'indice 3 (sans le retirer !!!)" << endl;
  35.   if (p->indiceValide(3) && p->estOccupe(3))   // exception
  36.   {
  37.     Collectionneur c = p->getElement(3);
  39.     cout << "Element recupere = " << c << endl;
  40.   }
  41.   else cout << "Case non occupee ou indice invalide !" << endl;
  42.   cout << endl;
  44.   p->affiche();
  45.   cout << endl;
  47.   cout << "On retire l'element d'indice 2" << endl;
  48.   if (p->indiceValide(2) && p->estOccupe(2))   // exception
  49.   {
  50.     Collectionneur c = p->retireElement(2);
  51.     cout << "Element recupere = " << c << endl;
  52.   }
  53.   else cout << "Case non occupee ou indice invalide !" << endl;
  54.   cout << endl;
  56.   p->affiche(); 
  57.   cout << endl;
  59.   cout << "On ajoute Issier en utilisant l'operateur +" << endl;
  60.   // operator+ : Ajout dans la premiere case libre du vecteur
  61.   (*p) + Collectionneur("Issier","Pol","21/06/1996","belge","issierpo",1);
  62.   p->affiche();   
  63.                                  
  64.   cout << "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX" << endl;
  65.   Collectionneur c("Sonite","Sam","11/01/1992","belge","sonitesa",6);
  66. c.affiche();
  67.   cout << "On ajoute Sonite en utilisant l'operateur +" << endl;
  68.   (*p) + c;
  71.   p->affiche();                                       
  72.   cout << endl;
  74.   delete p;
  75. }


 
En fait, il s'agit d'une hiérarchie de classe et chaque classe fille redéfinis brièvement l'opérateur << (en appelant celui de la classe mère).
Voici la hiérarchie dont fait partie la classe Collectionneur ;
 
Header de la classe Personne ;  
 

Code :
  1. #ifndef PERSONNE_H
  2. #define PERSONNE_H
  3. #include <stdio.h>
  4. #include <stdlib.h>
  5. #include <iostream.h>
  8. class Personne
  9. {
  10. protected:
  11.  char * Nom;
  12.  char * Prenom;
  13.  char * DateNaissance;
  14.  char * Nationalite;
  15.  char * id;
  17. public:
  18. //Constructeurs
  20.  Personne();
  21.  Personne(const char *TmpNom,const char *TmpPrenom,const char *TmpDateNaissance, const char *TmpNationalite);
  22.  Personne(const Personne& Pers);
  25. //Surcharge d'operateurs
  27.  friend istream& operator>>(istream& s, Personne& Pers);
  28.  friend ostream& operator<<(ostream& s, const Personne& Pers);
  30.  virtual void affiche();
  31.  virtual char* getIdentification();
  33. //Setters
  34.  void setNom(const char* TmpNom);
  35.  void setPrenom(const char* TmpPrenom);
  36.  void setDateNaissance(const char* TmpDateNaissance);
  37.  void setNationalite(const char* TmpNationalite);
  39. //Getters
  40.  char* getNom() const;
  41.  char* getPrenom() const;
  42.  char* getDateNaissance() const;
  43.  char* getNationalite() const;
  44. //Destructeur
  45.  virtual ~Personne();
  46. };
  48. #endif

 
 
Corps ;
 

Code :
  1. #include "Personne.h"
  3. //Constructeurs
  5. Personne::Personne()
  6. {
  7. Nom=NULL;
  8. Prenom=NULL;
  9. DateNaissance=NULL; //Il faudra sans doute remplacer ce champ par une classe Cdate.
  10. Nationalite=NULL;
  11. id=NULL;
  12. setNom("--" );
  13. setPrenom("--" );
  14. setDateNaissance("--/--/--" );
  15. setNationalite("--" );
  17. }
  19. Personne::Personne(const char* TmpNom,const char* TmpPrenom,const char* TmpDateNaissance,const char* TmpNationalite)
  20. {
  21. Nom=NULL;
  22. Prenom=NULL;
  23. DateNaissance=NULL;
  24. Nationalite=NULL;
  25. id=NULL;
  26. setNom(TmpNom);
  27. setPrenom(TmpPrenom);
  28. setDateNaissance(TmpDateNaissance);
  29. setNationalite(TmpNationalite);
  31. }
  32. Personne::Personne(const Personne& Pers)
  33. { Nom=NULL;
  34. Prenom=NULL;
  35. DateNaissance=NULL;
  36. Nationalite=NULL;
  37. id=NULL;
  38. setNom(Pers.getNom());
  39. setPrenom(Pers.getPrenom());
  40. setDateNaissance(Pers.getDateNaissance());
  41. setNationalite(Pers.getNationalite());
  42. }
  43. //Surcharge d'operateurs
  45. istream& operator>>(istream& s, Personne& Pers)
  46. {
  47. char Buffer[100];
  48. Pers.id=NULL;
  49. cout<<"Nom : ";
  50. s>>Buffer;
  51. Pers.setNom(Buffer);
  52. cout<<"Prenom : ";
  53. s>>Buffer;
  54. Pers.setPrenom(Buffer);
  55. cout<<"Date de naissance : ";
  56. s>>Buffer;
  57. Pers.setDateNaissance(Buffer);
  58. cout<<"Nationalite : ";
  59. s>>Buffer;
  60. Pers.setNationalite(Buffer);
  63. return s;
  64. }
  66. ostream& operator<<(ostream& s, const Personne& Pers)
  67. {
  68. cout << "OUT Personne" << endl;
  69. s<<"Nom : "<<Pers.getNom()<<endl;
  70. s<<"Prenom : "<<Pers.getPrenom()<<endl;
  71. s<<"Date de naissance : "<<Pers.getDateNaissance()<<endl;
  72. s<<"Nationalite : "<<Pers.getNationalite()<<endl;
  74. return s;
  75. }
  77. void Personne::affiche()
  78. {cout<<"Personne"<<endl;
  79. cout<<"Nom : "<<getNom()<<endl;
  80. cout<<"Prenom : "<<getPrenom()<<endl;
  81. cout<<"Date de naissance : "<<getDateNaissance()<<endl;
  82. cout<<"Nationalite : "<<getNationalite()<<endl;
  83. }
  85. char* Personne::getIdentification()
  86. {
  87. if(Nom)
  88. {
  89.  int Nblettre = strlen(getNom());
  90.  id = new char[Nblettre+1];
  91.  strcpy(id, getNom());
  92.  return id;
  93. }
  94. else
  95. {
  96.  id = new char[3];
  97.  strcpy(id,"--" );
  98.  return id;
  99. }
  100. }
  101. //Setters
  103. void Personne::setNom(const char* TmpNom)
  104. {
  105. if(TmpNom)
  106. {
  107.  int Nblettre;
  108.  Nblettre = strlen(TmpNom);
  109.  if(Nom) delete Nom;
  110.  Nom = new char[Nblettre+1];
  111.  strcpy(Nom, TmpNom);
  112. }
  113. else
  114.  return;
  115. }
  117. void Personne::setPrenom(const char* TmpPrenom)
  118. {
  119. if(TmpPrenom)
  120. {
  121.  int Nblettre;
  122.  Nblettre = strlen(TmpPrenom);
  123.  if(Prenom) delete Prenom;
  124.  Prenom = new char[Nblettre+1];
  125.  strcpy(Prenom, TmpPrenom);
  126. }
  127. else
  128.  return;
  129. }
  131. void Personne::setDateNaissance(const char* TmpDateNaissance)
  132. {
  133. if(TmpDateNaissance)
  134. {
  135.  int Nblettre;
  136.  Nblettre = strlen(TmpDateNaissance);
  137.  if(DateNaissance) delete DateNaissance;
  138.  DateNaissance = new char[Nblettre+1];
  139.  strcpy(DateNaissance, TmpDateNaissance);
  140. }
  141. else
  142.  return;
  143. }
  145. void Personne::setNationalite(const char* TmpNationalite)
  146. {
  147. if(TmpNationalite)
  148. {
  149.  int Nblettre;
  150.  Nblettre = strlen(TmpNationalite);
  151.  if(Nationalite) delete Nationalite;
  152.  Nationalite = new char[Nblettre+1];
  153.  strcpy(Nationalite, TmpNationalite);
  154. }
  155. else
  156.  return;
  157. }
  160. //Getters
  162. char* Personne::getNom() const
  163. {
  164. return Nom;
  165. }
  167. char* Personne::getPrenom() const
  168. {
  169. return Prenom;
  170. }
  172. char* Personne::getDateNaissance() const
  173. {
  174. return DateNaissance;
  175. }
  177. char* Personne::getNationalite() const
  178. {
  179. return Nationalite;
  180. }
  181. //Destructeur
  183. Personne::~Personne()
  184. {
  185. if(Nom) delete Nom;
  186. if(Prenom) delete Prenom;
  187. if(DateNaissance) delete DateNaissance;
  188. if(Nationalite) delete Nationalite;
  189. if(id) delete id;
  190. }


 
Beaucoup de lignes superflues ont été ajoutée uniquement dans le but de tester/tracer. J'ai fais énormément de test avant de demander de l'aide.
 
Header de la classe AuthorisedPerson (Classe abstraite dérivée de Personne).
 

Code :
  1. #ifndef AUTHORIZEDPERSON_H
  2. #define AUTHORIZEDPERSON_H
  3. #include <iostream.h>
  4. #include <stdio.h>
  5. #include <stdlib.h>
  6. #include "Personne.h"
  8. class AuthorisedPerson : public Personne
  9. {
  10. protected:
  11.  char * Login;
  13. public:
  15.  AuthorisedPerson();
  17.  AuthorisedPerson(const char* TmpNom, const char* TmpPrenom, const char* TmpDateNaissance, const char* TmpNationalite, const char* TmpLogin);
  19.  AuthorisedPerson(const AuthorisedPerson& AuthPers);
  21.  friend istream& operator>>(istream& s, AuthorisedPerson& AuthPers);
  22.  friend ostream& operator<<(ostream& s, const AuthorisedPerson& AuthPers);
  25.  virtual char* getIdentification() = 0;
  27.  void setLogin(const char* TmpLogin);
  29.  char* getLogin() const;
  31.  virtual ~AuthorisedPerson();
  33. };
  35. #endif


 
Cxx ;  
 

Code :
  1. #include "AuthorisedPerson.h"
  3. AuthorisedPerson::AuthorisedPerson() : Personne()
  4. {
  5. //Appel du constructeur par défaut de la classe Personne
  7. Login = NULL;
  8. setLogin("--" );
  9. }
  11. AuthorisedPerson::AuthorisedPerson(const char* TmpNom, const char* TmpPrenom, const char* TmpDateNaissance, const char* TmpNationalite, const char* TmpLogin) : Personne(TmpNom, TmpPrenom, TmpDateNaissance, TmpNationalite)
  12. {
  13. //Appel du constructeur d'initialisation de la classe Personne
  15. Login = NULL;
  16. setLogin(TmpLogin);
  17. }
  19. AuthorisedPerson::AuthorisedPerson(const AuthorisedPerson& AuthPers) : Personne((AuthorisedPerson& )AuthPers)
  20. {
  21. Login = NULL;
  22. setLogin(AuthPers.getLogin());
  23. }
  25. istream& operator>>(istream& s, AuthorisedPerson& AuthPers)
  26. {
  27. char TmpLogin[100];
  28. s>>(Personne& )AuthPers;
  29. cout<<"Login : ";
  30. s>>TmpLogin;
  31. AuthPers.setLogin(TmpLogin);
  33. return s;
  35. }
  36. ostream& operator<<(ostream& s, const AuthorisedPerson& AuthPers)
  37. {
  38. cout << "OUT author" << endl;
  39. s<<(Personne& )AuthPers;
  40. s<<"Login : "<<AuthPers.getLogin()<<endl;
  42. return s;
  43. }
  46. void AuthorisedPerson::setLogin(const char* TmpLogin)
  47. {
  48. if(TmpLogin)
  49. { 
  50.  int Nblettre;
  51.  Nblettre = strlen(TmpLogin);
  52.  if(Login) delete Login;
  53.  Login = new char[Nblettre+1];
  54.  strcpy(Login, TmpLogin);
  56. }
  57. else
  58.  return;
  59. }
  61. char* AuthorisedPerson::getLogin() const
  62. {cout<<"Author"<<endl;
  63. return Login;
  64. }
  66. AuthorisedPerson::~AuthorisedPerson()
  67. {
  69. if(Login) delete Login;
  70. }


 
Header de la classe Collectionneur (suivit du corps) qui dérive de AuthorisedPerson
 

Code :
  1. #ifndef COLLECTIONNEUR_H
  2. #define COLLECTIONNEUR_H
  3. #include <stdio.h>
  4. #include <stdlib.h>
  5. #include <iostream.h>
  7. #include "AuthorisedPerson.h"
  9. class Collectionneur : public AuthorisedPerson
  10. {
  11. private:
  12.  int NbCollections;
  13. public:
  15. //Constructeurs
  17.  Collectionneur();
  18.  Collectionneur(const char* TmpNom, const char* TmpPrenom, const char* TmpDateNaissance, const char* TmpNationalite, const char* TmpLogin, const int TmpNbCollections);
  20.  Collectionneur(const Collectionneur& TmpCol);
  21. //Surcharges d'operateurs
  23. Collectionneur& operator=(const Collectionneur& TmpCol);
  24.  friend istream& operator>>(istream& s, Collectionneur& Col);
  25.  friend ostream& operator<<(ostream& s, const Collectionneur& Col);
  27. //Setter(s)
  28.  void setNbCollections(const int TmpNbCollections);
  29. //Getter(s)
  30.  int getNbCollections() const;
  32.  void affiche();
  33.  char* getIdentification();
  35.  //~Collectionneur();
  36. };
  38. #endif


 
Collectionneur.cxx
 

Code :
  1. #include "Collectionneur.h"
  3. Collectionneur::Collectionneur() : AuthorisedPerson()
  4. {
  5. setNbCollections(0);
  6. }
  8. Collectionneur::Collectionneur(const char* TmpNom, const char* TmpPrenom, const char* TmpDateNaissance, const char* TmpNationalite, const char* TmpLogin, const int TmpNbCollections) : AuthorisedPerson(TmpNom, TmpPrenom, TmpDateNaissance, TmpNationalite, TmpLogin)
  9. {
  10. setNbCollections(TmpNbCollections);
  11. }
  13. Collectionneur::Collectionneur(const Collectionneur& TmpCol) : AuthorisedPerson((AuthorisedPerson& )TmpCol)
  14. {
  15. setNbCollections(TmpCol.getNbCollections());
  16. }
  18. Collectionneur& Collectionneur::operator=(const Collectionneur& TmpCol)
  19. {
  20. /*Nom=NULL;
  21. Prenom=NULL;
  22. DateNaissance=NULL;
  23. Nationalite=NULL;
  24. id=NULL;
  25. Login=NULL;*/
  26. setNom(TmpCol.getNom());
  27. setPrenom(TmpCol.getPrenom());
  28. setDateNaissance(TmpCol.getDateNaissance());
  29. setNationalite(TmpCol.getNationalite());
  30. setLogin(TmpCol.getLogin());
  31. setNbCollections(TmpCol.getNbCollections());
  33. return *this;
  34. }
  36. istream& operator>>(istream& s, Collectionneur& Col)
  37. {
  38. cout << "IN" << endl;
  39. int TmpCol=0;
  41. s>>(AuthorisedPerson& )Col;
  43. cout<<"Nombre de Collections : ";
  44. s>>TmpCol;
  45. Col.setNbCollections(TmpCol);
  47. return s;
  48. }
  49. ostream& operator<<(ostream& s, const Collectionneur& Col)
  50. {
  51. cout << "OUT collect" << endl;
  52. s<<(AuthorisedPerson& )Col;
  53. s<<"Nom : " << Col.getNom() << endl;
  54. s<<"Login : " << Col.getLogin() << endl;
  55. s<<"Nombre de collections : "<<Col.getNbCollections()<<endl;
  57. return s;
  58. }
  61. void Collectionneur::setNbCollections(const int TmpNbCollections)
  62. {
  63. if(TmpNbCollections >= 0) NbCollections = TmpNbCollections;
  64. else return;
  66. }
  68. int Collectionneur::getNbCollections() const
  69. {
  70. return NbCollections;
  71. }
  73. void Collectionneur::affiche()
  74. {cout<<"Collect"<<endl;
  75. Personne::affiche();
  76. cout<<"Login : "<<getLogin()<<endl;
  77. cout<<"Nombre de Collections : "<<getNbCollections()<<endl;
  78. }
  80. char* Collectionneur::getIdentification()
  81. {
  82. if(Nom)
  83. {
  84.  int Nblettre = strlen(getNom());
  86.  id = new char[Nblettre+10];
  88.  sprintf(id, "COLLECTIONNEUR:%s#%d", getNom(),getNbCollections());
  91.  return id;
  92. }
  93. else
  94. {
  95.  id = new char[22];
  96.  strcpy(id,"COLLECTIONNEUR:--#00" );
  97.  return id;
  98. }
  99. }


 
Voilà, je rappel qu'il s'agit d'un cas d'école.
 
Merci de ton aide.


Message édité par christallis le 06-11-2011 à 11:01:16
Reply

Marsh Posté le 06-11-2011 à 15:58:40    

y'aurait plein de choses à redire. Il manque la déclaration de ton template Structure pour être exhaustif.
Cela dit, je pense que le premier problème auquel tu te butes, c'est qu'il manque la déclaration de tes opérateurs.
 
Ton prof attend sans doute que tu mettes en avant le polymorphisme. Du coup, tu n'as besoin que d'un seul opérateur qui prend en entrée une Personne (par référence constante comme tu le fais très bien). Déclare une fonction d'affichage qui va être virtuelle au niveau de personne, et surcharge-là à chaque étage de ton héritage. Retire tes autres opérateurs, et surtout, déclare les opérateurs dans tes .h (l'implémentation reste dans le .cpp)
 
Ensuite, attention à tes sprintf, ils sont complètement faux. Par exemple :

Code :
  1. int Nblettre = strlen(getNom());
  2.   id = new char[Nblettre+10];
  3.   sprintf(id, "COLLECTIONNEUR:%s#%d", getNom(),getNbCollections());


là, tu demandes un buffer pour écrire une chaine. Tu sais que cette chaine va être de la taille de ce que tu mets en dur dans ton formatage (au moins 16 caractères) + la taille du nom (c'est bien, tu en as tenu compte) + la taille en caractères du nombre de collections (log à base 10 de ce nombre) + 1 pour le zéro terminal. Avec le code que tu as écrit, c'est sur que tu débordes. Ca peut faire tout et nimporte quoi dans ton programme, mais surtout le faire crasher.
 
Normalement, tu ne devrais pas avoir de "magic numbers", comme ton tableau de 22 caractères dans la même fonction ou ton "+10" pour le buffer.
 
Après, j'ai pas tout relu, loin de là. Il faudrait que tu apprennes à te servir de ton debugger, pour ce genre de chose, ca t'aidera grandement pour le développement de tes programmes suivants. J'imagine que vous avez eu un bref cours sur gdb si on vous fait programmer sous solaris. Il va être temps de s'en servir (sinon, tu trouveras de la doc sur le net ...)


---------------
last.fm
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed