dynamic_cast, segmentation fault, constructeur par copie

dynamic_cast, segmentation fault, constructeur par copie - C++ - Programmation

Marsh Posté le 02-04-2005 à 19:52:05    

Bonjour à tous !
Voilà je découvre le c++, et je cherche à modéliser quelque chose de simple du point de vue de la conception objet, mais c'est déjà assez subtil quand on débute (et qu'on a des "habitudes" liées au langage java)
 
En gros ce que je veux faire :
 
 

Code :
  1. int main(void) {
  3.     MachineAEtat machine;
  4.     Action changementdEtat;
  6.     machine.run(changementdEtat);
  8.     return 0;
  9. }
  15. class MachineAEtat {
  16.     public:
  17.         Etat* etatCourant;
  19.         MachineAEtat() {
  20.             etatCourant = new Depart();
  21.         }     
  23.         void run(const Action& action) {
  24.             *etatCourant = etatCourant->agir(action);
  25.         }     
  26. };
  29. class Etat {
  30.     public:
  31.         virtual void agir(const Action& ) = 0;
  32. };
  34. class Depart : public Etat {
  35.     public:
  36.         Etat& agir(const Action& action) {
  37.             if (action.getMessage() == "changer" ) {
  38.                 return *(new Arrivee());
  39.             }     
  40.             return *this;
  41.         }     
  43. };
  45. class Arrivee : public Etat {
  46.     public:
  47.         Etat& agir(const Action& action) {
  48.             if (action.getMessage() == "changer" ) {
  49.                 return *(new Depart());
  50.             }     
  51.             return *this;
  52.         }     
  54. };


 
 
 
Ce à quoi j'aspire :
 
- si je suis dans Depart, que je crée un Arrivee, je veux transmettre des informations, donc je passe par un constructeur par copie.
Etat peut faire : Etat(const Etat& )
Arrivee peut faire : Arrivee(const Etat& )
Et donc moi je renvoie un *(new Arrivee(*this)) depuis la classe Départ
 
 
 
 
En gros vous en pensez quoi niveau "intégrité des objets ?" Je me demande si je ne suis pas en train de caster comme un grouik dans tous les sens :/
 
 
 
Autre question ! J'ai trouvé ceci sur un post de fin2004 du forum, au sujet du cast du C qui est à proscrire :
 

Code :
  1. class Truc {
  2.     public:
  3.         virtual void prout() {
  4.             std::cout<<"prout\n";
  5.         }
  6. }; 
  9. class Machin {
  10.     public:
  11.         virtual void roger() {
  12.             std::cout<<"roger\n";
  13.         }
  14. }; 
  15.        
  16. class Bidule : public Truc, Machin {
  17.        
  18. };         
  19.        
  20. int main() {
  21.     Bidule * bidule = new Bidule(); // on fait un bidule
  22.     Truc   * tr = (Truc*) (bidule); // bidule etant aussi un Truc,
  23.                                     //on peut le caster, n'est ce pas ?
  24.     tr->prout();
  26.     Machin * mc = (Machin* ) (tr);    // d'accord, Truc n'est pas un Machin, mais finalement,
  27.                                       // le type reel de notre "Truc"
  28.                                       // c'est "Bidule" et "Bidule" est aussi un machin. Logique, non ?
  29.        
  30.     mc->roger();  //de facon tres interessante,
  31.                   //on ne verra pas "roger" a l'ecran, mais "prout".
  32.                   //avec RTTI et dynamic cast, ca affiche bien "roger"
  33.     return 0;
  34. }


 
 
Sauf que pour moi:
avec le cast du C : ca affiche en effet ce que disent les commentaires, donc pas le résultat attendu
avec le dynamic cast : ben moi ca crash  :heink:  :??:  
 
 
 
 
Voilà, si quelqu'un veut bien éclairer ma lanterne  :jap:


---------------
Il y a autant d'atomes d'oxygène dans une molécule d'eau que d'étoiles dans le système solaire.
Reply

Marsh Posté le 02-04-2005 à 19:52:05   

Reply

Marsh Posté le 02-04-2005 à 20:31:54    

active les RTTI dans ton compilateur (de ******) ?

Reply

Marsh Posté le 02-04-2005 à 20:52:03    

Taz a écrit :

active les RTTI dans ton compilateur (de ******) ?


 
Ben je compile avec g++, a priori il ne semble pas y avoir de trucs supplémentaires à activer.
 
Par exemple sur un truc comme ça ça marche bien :

Code :
  1. #include <typeinfo>
  2. #include <iostream>
  3. using namespace std;
  5. class A {
  6.     public:
  7.         virtual void do_it() { cout << "I'm an A" << endl; }
  8. };
  9. class B : public A {
  10.     public:
  11.         void do_it() { cout << "I'm a B" << endl; }
  12. };
  14. class C : public A {
  15.     public:
  16.         void do_it() { cout << "I'm a C" << endl; }
  17. };
  19. int main(void) {
  20.     A a;
  21.     B b;
  22.     C c;
  23.     A* obj;
  25.     cout << typeid(obj).name() << endl;
  26.     obj = &a;
  27.     cout << typeid(*obj).name() << endl;
  28.     obj = &b;
  29.     cout << typeid(*obj).name() << endl;
  30.     if (typeid(*obj) == typeid(b))
  31.         cout << "on pointe un B" << endl;
  32.     obj = &c;
  33.     cout << typeid(*obj).name() << endl;
  35.     return 0;
  36. }


Message édité par Xavier_OM le 02-04-2005 à 20:52:21

---------------
Il y a autant d'atomes d'oxygène dans une molécule d'eau que d'étoiles dans le système solaire.
Reply

Marsh Posté le 02-04-2005 à 21:14:37    

je vois pas de dynamic_cast là
 
et ton utilisation de typeid() est exactement l'usage qu'il ne faut pas en faire


Message édité par Taz le 02-04-2005 à 21:15:06
Reply

Marsh Posté le 02-04-2005 à 21:17:06    

et n'oublie pas le destructeur virtuel vu que tu veux pas du polymorphisme. Dans ton cas, je vois pas l'intérêt du dynamic_cast vu que le dispatching est fait par le mécanisme de virtual.

Reply

Marsh Posté le 02-04-2005 à 21:32:02    

Code :
  1. #include <typeinfo>
  2. #include <iostream>
  3. #include <vector>
  5. using namespace std;
  8. class Graine
  9. {
  10. public:
  11.   virtual ~Graine() { }
  12. };
  15. class BonGrain : public Graine
  16. {
  17. public:
  18.   virtual ~BonGrain() { }
  19. };
  22. class Ivraie : public Graine
  23. {
  24. public:
  25.   virtual ~Ivraie() { }
  26. };
  29. void separe(vector<Graine*> &recolte, vector<BonGrain> &grains, vector<Ivraie> &ivraies)
  30. {
  31.   for(vector<Graine*>::iterator i(recolte.begin()); i != recolte.end(); ++i)
  32.     {
  33.       if(BonGrain* bon_grain = dynamic_cast<BonGrain*>(*i))
  34. {
  35.   grains.push_back(*bon_grain);
  36. }
  37.       else if(Ivraie* ivraie = dynamic_cast<Ivraie*>(*i))
  38. {
  39.   ivraies.push_back(*ivraie);
  40. }
  41.       else
  42. {
  43.   cerr << "Je ne sais pas ce que c'est que cette cochonnerie !\n";
  44. }
  46.       delete *i;
  47.     }
  49.   recolte.clear();
  50. }
  53. int main()
  54. {
  55.   vector<Graine*> recolte;
  56.   vector<BonGrain> grains;
  57.   vector<Ivraie> ivraies;
  59.   recolte.push_back(new BonGrain);
  60.   recolte.push_back(new Ivraie);
  61.   recolte.push_back(new BonGrain);
  63.   separe(recolte, grains, ivraies);
  65.   cout << grains.size() << " bon grains et " << ivraies.size() << " ivraies\n";
  67.   return 0;
  68. }

un exemple complet de dynamic_cast.

Reply

Marsh Posté le 02-04-2005 à 22:01:42    

Merci pour l'exemple  :jap:  
 
(Pour le dynamic_cast c'est plus pour comprendre pourquoi le truc trouvé sur ce forum crash chez moi que pour mes états :D)


---------------
Il y a autant d'atomes d'oxygène dans une molécule d'eau que d'étoiles dans le système solaire.
Reply

Marsh Posté le 02-04-2005 à 22:15:10    

ton trucs avec tes états à l'air crade comme pas possible. compile en -Wall -std=c++98 -Weffc++ déjà, et ensuite, va falloir apprendre à gérer la mémoire, parce que là ça fuit et ça corromp à tout va

Reply

Marsh Posté le 02-04-2005 à 22:42:57    

Taz a écrit :

ton trucs avec tes états à l'air crade comme pas possible. compile en -Wall -std=c++98 -Weffc++ déjà, et ensuite, va falloir apprendre à gérer la mémoire, parce que là ça fuit et ça corromp à tout va


 
Sympa cette option -Weffc++  :ouch:  :jap:  
En effet je ne maitrise pas trop les destructeurs et les fuites mémoires.
 
Bon 0 erreurs 0 warnings pour "g++ -Wall -Wshadow -W -std=c++98"
4 pages de warnings avec -Weffc++  :cry:  
 
 
Bon ben j'y retourne
 


---------------
Il y a autant d'atomes d'oxygène dans une molécule d'eau que d'étoiles dans le système solaire.
Reply

Sujets relatifs:

Leave a Replay

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