Discussion :

[Angrim]

Bonjour à tous,
voila mon soucis...
J'ai crée une classe (TArchiObj) contenant une fonction virtuelle pure (void PaintShape() = 0
Dans les classes (TRectObj, TCircleObj,...) héritant de celle ci j'ai donc redéfini la fonction PaintShape... seulement voila... quand j'utilise cette fonction par le biais de la classe TArchiObj, j'ai une erreur "Pure virtual function called"...
J'aimerais savoir si je peux éviter ce genre d'erreur... et surtout comment...

Voici le code appellant pour plus de clarté...

Code :

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

1
2
3
4
5
6
7
8
9
10
 
    TArchiObj* archiTemp = 0;
    for (int i=0; i<m_listObj->Count; i++)
    {
        archiTemp = ((TArchiObj*)(*m_listObj)[i]);
        if (!archiTemp)
            Application->MessageBox("Erreur liste","",0);
        else
            archiTemp->PaintObj();
    }

Merçi pour votre aide
NB : m_listObj est une TList contenant des éléments héritant de TArchiObj)

[Médinoc]

Généralement, on a cette erreur en appelant une fonction virtuelle pure dans un constructeur ou destructeur...

Au fait, quel est le type de m_listObj et de ses éléments ?

[Angrim]

m_listObj est une TList et contient donc des void*...
Dans mon cas, elle contient les TArchiObj* ou plus exactement des TRectObj* et des TCircleObj*.
J'appelle cette fonction sur un OnPaint d'un TPaintBox pour pouvoir redessiner ces objets...

[juzam]

Essaye peut être de remplacer ton cast normal par un dynamic_cast :

Code :

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

 archiTemp = ((TArchiObj*)(*m_listObj)[i]);

->

Code :

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

 archiTemp = dynamic_cast<TArchiObj*>((*m_listObj)[i]);

tu peux aussi afficher les types dynamiques de tes objets avec typeid(archiTemp).name() pour vérifier que les bonnes classes ont été instanciées.

[Angrim]

le dynami_cast ne fonctionne pas...
Une erreur de type "Type void is not a defined class with virtual functions."..

Quand au type des données, ils paraissent bons...

[juzam]

Je ne connais pas la classe TListe, mais s'(il s'agit d'un template, tu pourrais peut être typer des TArchiObj* plutot que des void* non?

Le problème arrive sur tous les objets de la liste?

[juzam]

et dans ton premier message, tu parles d'une méthode paintShape, mais dans le code tu appelles paintObj(). Juste une faute de frappe?

sinon, je confirme après avoir fait le test, on ne peut pas "dynamic caster" de void* :

Code :

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

error: cannot dynamic_cast `toto' (of type `void*') to type `class Foo*' (source is not a pointer to class)

[Angrim]

Effectivement faute de frappe désolé ...
Sinon le type TList permet de stocker tout type de pointeur... Et c'est une liste complète avec toutes les fonctions qui conviennent... (Add, Remove, Sort,...)
Je ne sais pas trop pourquoi cela ne fonctionne pas...

[juzam]

le type TList permet de stocker tout type de pointeur...

certes, c'est mieux.
Mais comment déclares tu cette liste?
tu dis "c'est une liste"?
"c'est une liste de void*"?

C'est un template, comme le sont les conteneurs STL, ou un ensemble à la Java, cad une liste d'objets d'un type prédéfini (Object en l'occurence)?

Si tu peux typer les éléments de la liste, autant utiliser directement ta classe de départ (TArchiObj).

[Angrim]

Non c'est une liste Dynamique...
Déclaration :

Code :

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

1
2
TList* myList;
myList = new TList();

Pour ajouter un élément :

Code :

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

myList->Add(Element);

TArchiObj represente un seul élément de la liste