Discussion :

[Marty69]

Bonjour à toutes et à tous.

Je suis confronté à une petit soucis sur openFrameworks 0.8.4 avec XCode 6.0.1 sous OSX Mavericks

Dans mon header ROI.h :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
#include <ofBaseTypes.h>
#include <ofRectangle.h>
 
class ROI : public ofBaseDraws, public ofRectangle {
 
    public :
 
    void draw(float _x, float _y) const;
    void draw(float _x, float _y, float _w, float _h) const;
    float getWidth() const;
    float getHeight() const;
};

La déclaration dans mon kinectTracker.h :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#include <ofxKinect.h>
#include <ofxOpenCv.h>
#include "ROI.h"
 
class kinectTracker {
 
public:
    kinectTracker();
    ~kinectTracker();
 
    ROI roi; // l'erreur apparait ici !
 
private:
 
    ofxKinect kinect;
 
};

Cela me donne l'erreur :

Field type 'ROI' is an abstract class

Je ne comprend pas cette erreur !

Si quelqu'un peut m'éclairer c'est top...

Merci à vous.

[bacelar]

ROI est abstraite car elle n'est pas instanciable.
Pourquoi n'est-elle pas instanciable ?
Car des méthodes de ses classes mères ou ancêtres ne sont pas définies et qu'elles doivent l'être avant qu'une classe fille ou descendante de ces classes mères soient instanciables.
Le compilateur devrait vous indiquer dans son message d'erreur les méthodes que la classe fille doit définir pour qu'elle soit instanciables.

[Invité]

Bonsoir,

Pense à donner (et lire) le message d'erreur en entier.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
kinectTracker.h:11:9: error: cannot declare field 'kinectTracker::roi' to be of abstract type 'ROI'
     ROI roi; // l'erreur apparait ici !
         ^
ROI.h:4:7: note:   because the following virtual functions are pure within 'ROI':
  class ROI : public ofBaseDraws {
        ^
ofBaseTypes.h:47:18: note:   virtual void ofBaseDraws::draw(float, float)
      virtual void draw(float x, float y)=0;
                   ^
ofBaseTypes.h:68:18: note:   virtual void ofBaseDraws::draw(float, float, float, float)
      virtual void draw(float x, float y, float w, float h)=0;
                   ^
ofBaseTypes.h:59:19: note:   virtual float ofBaseDraws::getHeight()
      virtual float getHeight()=0;
                    ^
ofBaseTypes.h:60:19: note:   virtual float ofBaseDraws::getWidth()
      virtual float getWidth()=0;
                    ^

D'après la documentation, la classe ofBaseDraws est une classe abstraite car elle possède quatre méthodes virtuelles pures :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
virtual void 	draw (float x, float y)=0;
virtual void 	draw (float x, float y, float w, float h)=0;
virtual float 	getHeight ()=0;
virtual float 	getWidth ()=0;

Or dans ta classe ROI, tu définis quatre nouvelles méthodes de signature différente (le const à la fin est en trop)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
void draw(float _x, float _y) const;
void draw(float _x, float _y, float _w, float _h) const;
float getWidth() const;
float getHeight() const;

Ton compilateur n'ayant pas les méthodes originales redéfinies considère que ta classe ROI est aussi abstraite, et ne peut donc être instanciée.

Si tu compiles en C++11, pense à utiliser le mot-clé override :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
class ROI : public ofBaseDraws, public ofRectangle {
 
    public :
 
    void draw(float _x, float _y) const override;
    void draw(float _x, float _y, float _w, float _h) const override;
    float getWidth() const override;
    float getHeight() const override;
};

qui aurait pu te prévenir que tu ne redéfinissais pas des méthodes existantes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
ROI.h:8:10: error: 'void ROI::draw(float, float) const' marked override, but does not override
     void draw(float _x, float _y) const override;
          ^
ROI.h:9:10: error: 'void ROI::draw(float, float, float, float) const' marked override, but does not override
     void draw(float _x, float _y, float _w, float _h) const override;
          ^
ROI.h:10:11: error: 'float ROI::getWidth() const' marked override, but does not override
     float getWidth() const override;
           ^
ROI.h:11:11: error: 'float ROI::getHeight() const' marked override, but does not override
     float getHeight() const override;
           ^

[Marty69]

Bonsoir et merci beaucoup pour ta suggestion.

Il me semble pourtant avoir ce qu'il faut notament pour ofBaseDraws : les méthodes virtuelles pures semblent pourtant ok dans ROI

cf. détails de ofBaseDraws : http://ofxfenster.undef.ch/doc/classofBaseDraws.html

[Marty69]

Oups ! j'avais pas vu la réponse !
On a du répondre en même temps.
Merci pour tous ces détails.

Tout fonctionne maintenant.

Merci beaucoup pour votre aide et vos précieux conseils.

Bonne soirée.