[RESOLU] vtable error

vtable error [RESOLU] - C++ - Programmation

Marsh Posté le 13-11-2010 à 16:56:26    

Salut,
 
j'ai une erreur (vous l'aurez deviné) en lien avec d'apres ce que j'ai lu, les destructeurs virtuelles.
 
pour resumer les codes:
j'ai une fonction:
- point
- graphical object qui est une classe abstraite qui include point
- differentes figures classe publique de graphical object (ex class cercle: public graphicalobject)
- une classe group qui contient 50 obj graph
 
avant que j'ajoute la classe group, tout marchait bien
depuis, j'ai une erreur vtable
 
voici les differents codes:
 
point.cpp

Code :
  1. #include "Point.hpp"
  3. Point Point::ORIGINE(0,0);
  5. Point::~Point() {}
  7. double Point::dist(Point& a, Point& b)
  8. {
  9.     return sqrt((a._x-b._x)*(a._x-b._x)+(a._y-b._y)*(a._y-b._y));
  10. }
  12. std::string Point::tostring()
  13. {
  14.   std::string chaine("un point:\n" );
  15.   std::ostringstream ossx;
  16.   std::ostringstream ossy;
  17.   ossx << _x;
  18.   ossy << _y;
  19.   chaine += "x = "+ossx.str()+"\n"+"y = "+ossy.str()+"\n";
  20.  
  21.   return chaine;
  22. }
  24. void Point::display()
  25. {
  26.   std::cout << this->tostring() << std::endl;
  27. }


 
point.hpp

Code :
  1. #include <iostream>
  2. #include <math.h>
  3. #include <sstream>
  5. class Point
  6. {
  7.     private:
  8.         int _x;
  9.         int _y;
  11.     public:
  12. static Point ORIGINE;
  13. Point():_x(0),_y(0) {};
  14.         Point(int inx, int iny): _x(inx),_y(iny) {};
  15.         ~Point();
  17.         int getx() const {return _x;}
  18.         int gety() const {return _y;}
  19. void setPoint(int inx, int iny) {_x=inx,_y=iny;}
  20. void setx(int inx) {_x=inx;}
  21. void sety(int iny) {_y=iny;}
  23.         static double dist(Point& a, Point& b);
  24. std::string tostring();
  25. void display();
  26. };


 
GraphicalObject.hpp

Code :
  1. #ifndef GraphicalObject_HPP
  2. #define GraphicalObject_HPP
  4. #include <iostream>
  5. #include "Point.hpp"
  7. typedef enum COLORS
  8. {
  9.     rouge,
  10.     vert,
  11.     bleu
  12. }COLORS;
  14. class GraphicalObject
  15. {
  16.     private:
  17.         static int _cpt_id; //compteur
  18.         int _id;
  19.         Point _origin;
  20.         int _width;
  21.         int _height;
  22.         COLORS _couleurs;
  23.        
  24.   public:
  25.     GraphicalObject();
  26.     GraphicalObject(Point inp, int inwidth, int inheight);
  27.     virtual ~GraphicalObject() {_cpt_id--;};
  28.    
  29.     void setorigin(Point inp) {_origin=inp;}
  30.     void setwidth(int inwidth) {_width=inwidth;}
  31.     void setheight(int inheight) {_height=inheight;}
  32.     void setcolor(COLORS incoul) {_couleurs=incoul;}
  33.    
  34.     int getid() {return _id;};
  35.     Point getorigin() {return _origin;};
  36.     int getwidth() {return _width;};
  37.     int getheight() {return _height;};
  38.     COLORS getcolors() {return _couleurs;};
  39.    
  40.     virtual std::string tostring() = 0;
  41.     virtual void display() = 0;
  42. };
  44. #endif


 
GraphicalObject.cpp

Code :
  1. #include "GraphicalObject.hpp"
  3. int GraphicalObject::_cpt_id = 0;
  5. GraphicalObject::GraphicalObject()
  6. {
  7.   _cpt_id +=1;
  8.  
  9.   _id = _cpt_id;
  10.   _origin=Point::ORIGINE;
  11.   _width = 0;
  12.   _height = 0;
  13.   _couleurs = vert;
  14. }
  16. GraphicalObject::GraphicalObject(Point inp, int inwidth, int inheight)
  17. {
  18.   _cpt_id +=1;
  19.  
  20.   _id = _cpt_id;
  21.   _origin = inp;
  22.   _width = inwidth;
  23.   _height = inheight;
  24.   _couleurs = vert;
  25. }


 
Cercle.hpp

Code :
  1. #ifndef Cercle_HPP
  2. #define Cercle_HPP
  4. #include "GraphicalObject.hpp"
  6. class Cercle: public GraphicalObject
  7. {
  8.   public:
  9.     void display();
  10.     std::string tostring();
  11. };
  13. #endif


 
Cercle.cpp

Code :
  1. #include "Cercle.hpp"
  3. std::string Cercle::tostring()
  4. {
  5.   std::string chaine("cercle:\n" );
  6.   std::ostringstream ossx;
  7.   std::ostringstream ossy;
  8.   std::ostringstream ossh;
  9.   std::ostringstream ossid;
  10.  
  11.   ossx << getorigin().getx();
  12.   ossy << getorigin().gety();
  13.   ossh << getheight();
  14.   ossid << getid();
  15.   chaine += "id: "+ossid.str()+"\n"+"x: "+ossx.str()+"\n"+"y: "+ossy.str()+"\n";
  16.   chaine += "rayon: "+ossh.str()+"\n";
  17.  
  18.   return chaine;
  19. }
  21. void Cercle::display()
  22. {
  23.   std::cout << this->tostring() << std::endl;
  24. }


 
Group.hpp

Code :
  1. #ifndef Group_HPP
  2. #define Group_HPP
  4. #include "GraphicalObject.hpp"
  6. class Group: public GraphicalObject
  7. {
  8.  
  9.     private:
  10.       GraphicalObject * _list[50];
  11.       int _compt;
  12.      
  13.     public:
  14.       Group();
  15.       virtual ~Group() {};
  16.      
  17.       bool attach(GraphicalObject *pOG);
  18.       void detach(GraphicalObject *pOG);
  19.      
  20.       void display();
  21.       std::string tostring();
  22.      
  23. };
  25. #endif


 
Group.cpp

Code :
  1. #include "Group.hpp"
  3. Group::Group()
  4. {
  5. for(int i=0;i<50;i++) _list[i]=0;
  6. _compt = 0;
  7. }
  9. bool Group::attach(GraphicalObject * pOG)
  10. {
  11.   int res =0;
  12.   if(_compt<50)
  13.   {
  14.     int i=0;
  15.     while(_list[i]!=0) i++;
  16.     _list[i] = pOG;
  17.     _compt++;
  18.    
  19.     Point temp(std::min(getorigin().getx(), pOG->getorigin().getx()),
  20.       std::min(getorigin().gety(), pOG->getorigin().gety()));
  21.     setorigin(temp);
  22.    
  23.     setwidth(std::max(getwidth(), getorigin().getx()+pOG->getwidth()));
  24.     setheight(std::max(getheight(), getorigin().gety()+pOG->getheight()));
  25.    
  26.     res=1;
  27.   }
  28.   return res;
  29. }
  31. void Group::detach(GraphicalObject * pOG)
  32. { }


 
Erreur

Code :
  1. g++ -Wall -ansi -pedantic -Wextra   Cercle.o GraphicalObject.o Group.o line.o main.o Point.o Rectangle.o -lm -o ./exe
  2. Group.o: In function `Group::Group()':
  3. Group.cpp:(.text+0x16): undefined reference to `vtable for Group'
  4. Group.o: In function `Group::Group()':
  5. Group.cpp:(.text+0x68): undefined reference to `vtable for Group'
  6. main.o: In function `Group::~Group()':
  7. main.cpp:(.text._ZN5GroupD1Ev[Group::~Group()]+0xb): undefined reference to `vtable for Group'
  8. collect2: ld returned 1 exit status
  9. make: *** [pMed] Erreur 1


 
voila, (et sinon, le code doit pas etre génial, je debute :D )
 
merci d'avance


Message édité par karlakir le 15-11-2010 à 21:08:36

---------------
Si vous avez rien à faire, cliquez ici:
Reply

Marsh Posté le 13-11-2010 à 16:56:26   

Reply

Marsh Posté le 13-11-2010 à 17:18:19    

Ton seul membre virtuel de Group (le destructeur) est inline et il y a des chaînes de développement qui n'aiment pas ça.


---------------
The truth is rarely pure and never simple (Oscar Wilde)
Reply

Marsh Posté le 13-11-2010 à 18:04:56    

ca marche!!
 
merci beaucoup!
 
donc plus de destructeur virtuel inline si j'ai bien compris?


---------------
Si vous avez rien à faire, cliquez ici:
Reply

Marsh Posté le 13-11-2010 à 18:23:07    

Le problème c'est de ne pas avoir de membre virtuel non inline.


---------------
The truth is rarely pure and never simple (Oscar Wilde)
Reply

Marsh Posté le 14-11-2010 à 14:22:55    

Je comprend pas,
 
je dois avoir au moins un membre virtuel non inline et puis je peux mettre tous les autre inline ?
 
j'ai vraiment du mal avec les notions d heritage et les methodes virtuelles (bref le concept objet :) )

Message cité 1 fois

---------------
Si vous avez rien à faire, cliquez ici:
Reply

Marsh Posté le 14-11-2010 à 17:26:31    

karlakir a écrit :

Je comprend pas,
 
je dois avoir au moins un membre virtuel non inline et puis je peux mettre tous les autre inline ?


 
Oui.
 

Citation :

j'ai vraiment du mal avec les notions d heritage et les methodes virtuelles (bref le concept objet :) )


 
Ceci n'est en rien conceptuel, c'est juste une limitation d'un certain nombre de compilateurs.  Je ne pense pas que la norme l'impose (j'ai jamais cherché).


---------------
The truth is rarely pure and never simple (Oscar Wilde)
Reply

Marsh Posté le 15-11-2010 à 21:07:48    

oki!!
 
merci beaucoup pour ton aide  
 
 
(et je te dis à bientot, ainsi qu'à toute la communauté, I Will Be Back!!! c'est sur :) )


---------------
Si vous avez rien à faire, cliquez ici:
Reply

Sujets relatifs:

Leave a Replay

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