Discussion :

[awawawa]

Bonjour tout le monde,

Avant tout je vous souhaite une bonne année.

Je suis confronté à une erreur que je n'arrive pas à comprendre.

J'ai trois classes qui répondent au schémas suivant : PowerMetter<-AdvancedCapteur <-Capteur. (je m'excuse pour le franglais)

J'ai une fonction

Code :

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

void Initialize (string)

qui permet d’initialiser mon périphérique. Elle est déclarée comme étant virtuelle dans les classes AdvancedCapteur et Capteur.

Lorsque je teste le code suivant :

Code :

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

1
2
3
4
5
 
        PowerMeter *a = new PowerMeter;
        a->Initialize(iniFile.toStdString());  // fait bien appel à powermetter::Initialize
        mSensorList.push_back(*a);
        mSensorList.at(mSensorList.size() - 1).Initialize(iniFile.toStdString()); // fait appel à AdvancedCapteur::Initialize

powermetter::Initialize est appelée une seule fois. Je m'attends à ce qu'elle le soit une seconde fois puisque AdvancedCapteur::Initialize est déclaré comme étant virtuelle.

(Pour info mSensorList est un std::vector d'objet de type AdvancedCapteur).

Avez vous une idée?

Pour info je vous laisse aussi la déclaration de mes classes :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class PowerMeter :
    public AdvancedCapteur
{
public:
    PowerMeter();
    ~PowerMeter();
    void Initialize(string IniFile);
 
//bla bla bla
}

Code :

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

1
2
3
4
5
6
7
8
 
class Capteur
{
public:
 
    Capteur();
~Capteur();  
    virtual void Initialize (string IniFile);

Code :

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

1
2
3
4
5
6
7
8
9
 
class AdvancedCapteur :
    public Capteur
{
public:
    AdvancedCapteur();
    ~AdvancedCapteur();
 
    virtual void Initialize(string IniFile) {};

Code :

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

1
2
3
4
5
 
void PowerMeter::Initialize(string iniFile)
{
    Capteur::Initialize(iniFile);
//...bla bla bla

Ca fait deux heures que je me relis et que je fais des tests rien n'y fait... SOS!

[Bousk]

D'après ma boule de crystal, mSensorList est déclarée comme std::vector<AdvancedCapteur> mSensorList; (ha ben oui tiens, tu le dis toi-même dans ton post), et donc comment espères-tu y stocker un PowerMetter (ce nom ) alors qu'il ne prend que des objets AdvancedCapteur ?
La base du polymorpshime c'est les pointeurs et références. Le typage statique et dynamique.

[awawawa]

Je suis un fou...

Effectivement, je ne sais pas ce qui m'a pris...

Le pire c'est que pour tester des hypothèses, j'ai écrit le code suivant :

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
 
int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);
 
    QVector<Base*> liste;
    liste.append(new Fille);
    liste.append(new Garcon);
 
    for (int i = 0; i < liste.size(); i++)
    {
        liste.at(i)->test();
        liste.at(i)->faisCa();
    }
    return a.exec();
}

qui lui marchait. J'ai passé une partie de la nuit à essayer de jouer aux 7 erreurs sans succès. Merci beaucoup Bousk.

[Bousk]

Pour éviter les fuites mémoires, utilise un vector de unique_ptr.