[POO] Problème de redéfinition de méthodes

Version imprimable

Au collège, ils nous ont enseigner seulement du Java, donc lorsque je veux faire quelque chose d'un peu plus poussé, je me retrouve avec ce problème...

J'ai 2 classes toute simples.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 class a { public : int ID; void f() { ... } a() { ID = 0; } }; class b: public a { public : int x; b() { ID = 1; } void f() { ... } };
Ensuite, si je déclare un object a et un object b, aucun problèmes. Si je déclare un pointeur a et qu'il pointe vers un object b, je dois simplement changer f en virtuel, aucun problèmes.

Mon problème, c'est que je n'ai aucune idée de combien il va y avoir d'objects a ou b, donc je les store dans un tableau d'object a. Ensuite, quand vient le temps d'appeler la méthode f, j'appelle toujours celle de a. J'ai essayé de caster l'objet sortant du tableau en b lorsque ID = 1, mais cela me donne une erreur "a cannot be cast to b". Ceci marche en java, je m'en suis servis plusieurs fois au travail, mais il y a probablement une petite nuance C++ que j'oublie, et à cause de celle-ci, il m'est impossible de pouvoir accéder à la valeur x de b aussi-tot qu'il est inséré dans un tableau de a.

J'ai essayé aussi avec un vecteur, même résultat.

12/12/2008, 09h31
3DArchi

Bonjour,
1/ Tu trouveras ici des informations sur les cast ;
2/ Tu trouveras ici des informations sur les membres virtuels ;
3/ C'est une erreur que de mettre un ID dans ta classe pour identifier son type. Cela est souvent inutile et relève de l'erreur de conception.
4/ Comme indiqué dans la FAQ sur le cast, si tu as une collection de a et qu'à un moment tu dois downcaster vers un b, c'est que tu as probablement un problème de conception.
12/12/2008, 09h44
JolyLoic

Ne pas oublier que pour que les fonctions virtuelles marchent pour des objets dans un conteneur, il faut en C++ passer par des pointeurs.
12/12/2008, 19h27
ShadowOfEden

Citation:

Envoyé par 3DArchi

3/ C'est une erreur que de mettre un ID dans ta classe pour identifier son type. Cela est souvent inutile et relève de l'erreur de conception.
4/ Comme indiqué dans la FAQ sur le cast, si tu as une collection de a et qu'à un moment tu dois downcaster vers un b, c'est que tu as probablement un problème de conception.

Le ID je peut faire sans, je l'ai ajouté beaucoup plus tard après que mes essais de méthodes virtuel ne marchait pas, lorsque j'ai tenté de casté.

Pour le downcast, je pense bien que je n'ai pas le choix tant que je ne réussis pas a faire marché ma méthode virtuelle, mais si tu as une autre suggestion, j'aimerais bien la savoir.

Je travaille présentement sur un engine graphique en DirectX. Dans mes jeux précédent, j'utilisais un GraphicObject qui contenait tout ce qu'il pouvait contenir. Il contenait des variables reliées à l'animation et à plusieurs d'autres effets même s'il ne se servait jamais. Ca fonctionne, mais ca alourdi le rendu avec un paquet de conditions qui sont souvent innutile.

Ce que j'essais de faire c'est de créé plusieurs sous-GraphicObject, tel que AnimatedGraphicObject. De cette façon, les variables pour la gestion d'animation ne se retrouve pas dans un GlowingGraphicObject ou dans background inanimé.

Comme j'ai dit plus haut, ma fonction virtuelle fonctionne... jusqu'à temps que j'utilise un conteneur. Dans les FAQs, il n'y a que ce qui fonctionne déjà pour moi et rien sur mon problème.

Citation:

Envoyé par JolyLoic

Ne pas oublier que pour que les fonctions virtuelles marchent pour des objets dans un conteneur, il faut en C++ passer par des pointeurs.

J'ai essayé de plusieurs façons, toujours sans résultats positif.
12/12/2008, 19h42
Matthieu Brucher

Tu les stockes dans un tableau de a, donc tous les b deviennent des a. Il te faut un tableau de pointeurs vers a.
12/12/2008, 21h18
ShadowOfEden

Citation:

Envoyé par Matthieu Brucher

Tu les stockes dans un tableau de a, donc tous les b deviennent des a. Il te faut un tableau de pointeurs vers a.

C'est ce que je dit depuis le début. Comme j'ai dit, j'ai essayé de plusieurs façons (c'est à dire toute celles qui me venaient en tête) et aucunes n'a fonctionné. Mais un ami qui étudié dans un collège ou ils enseignaient le C++ a répondu à ma question, donc le problème est résolue!
12/12/2008, 23h44
Matthieu Brucher

Citation:

Envoyé par ShadowOfEden

C'est ce que je dit depuis le début.

Alors je ne vois pas où était ton problème. Sauf si tu n'as pas compris ce qui se passait.

Salut,

Bien que tu sembles avoir trouvé une réponse ailleurs, il faut savoir que, si tu dois gérer une collection d'objets polymorphes, le mieux reste toujours d'utiliser le conteneur fournit par la STL qui correspond le mieux à tes besoins en utilisant ce diagramme récapitulatif et de stocker des pointeurs vers l'objet de base.

Ainsi, le mieux est peut etre un code proche de

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
int main()
{
    /* déclaration d'une collection d'objets dérivés de A */
    std::vector<A*> tab;
    /* ajout d'éléments dérivés de A (B, C et D) ... voir de A le cas
     * échéans
     */
    tab.push_back(new A/*( parametres éventuels) */);
    tab.push_back(new B/*( parametres éventuels) */);
    tab.push_back(new C/*( parametres éventuels) */);
    tab.push_back(new D/*( parametres éventuels) */);
    /* appel d'un comportement polymorpe sur tous les éléments
     */
    for(size_t i=0;i<tab.size();++i)
        tab[i]->fonction();
    /* nous aurions pu utiliser des itérateur (constants ou non)
     */
    for(std::vector<A*>::const_iterator it= tab.begin();it!=tab.end();++it)
        (*it)->fonction();
    /* ne pas oublier de libérer correctement la mémoire
     */
    for(size_t i=0;i<tab.size();++i)
        delete tab[i];
    return 0;
}

15/12/2008, 02h10
ShadowOfEden

Citation:

Envoyé par Matthieu Brucher

Alors je ne vois pas où était ton problème. Sauf si tu n'as pas compris ce qui se passait.

Je savais ou était mon problème, je ne savais seulement pas comment faire ce que je voulais faire (un conteneur de pointeur pour que le B ne deviennent pas un simple A)

Citation:

Envoyé par koala01

Salut,

Bien que tu sembles avoir trouvé une réponse ailleurs, il faut savoir que, si tu dois gérer une collection d'objets polymorphes, le mieux reste toujours d'utiliser le conteneur fournit par la STL qui correspond le mieux à tes besoins en utilisant ce diagramme récapitulatif et de stocker des pointeurs vers l'objet de base.

Ainsi, le mieux est peut etre un code proche de

Merci beaucoup même si j'ai déjà ma réponse car c'est exactement ce que je cherchait et c'est très détaillé.

Pour le type de conteneur, mes réponses dans le diagramme sont toujours non. Pour l'instant je travaille avec un tableau car je travail sur l'engin graphique et le conteneur appartient au jeu donc je peut utilisé n'importe quel conteneur car ca me sert seulement de test.

Comme j'ai dit plus haut, mon conteneur contient tout les objects à être afficher. Leur ordre n'a pas donc pas d'importances. Dans mon ancien engin graphique, j'avais plusieurs tableau pour chaques types. Pour les projectiles, par exemple, lorsqu'elle n'est plus affiché (qu'elle sortent de l'écran ou qu'elle touche une cible) un bool est changé pour false. Un compteur gardait le compte de quel projectile était le prochain et celui-ci était réinitialisé lorsque nécessaire. Je n'ai jamais eu le problème, mais il y a toujours la possibilité qu'il y ait trop de projectiles et qu'un nouveau reinitialise un projectile déja actif. De plus, 60 fois par secondes, je devais parcourir un tableau pour lire un bool même s'il n'y avait aucun projectiles à l'écran.

Donc je crois que le vecteur serait le meilleur choix pour moi, car je pourrait avoir le nombre de projectiles nécessaire et effacé un en plein millieu sans avoir a déplacé les objets suivant contrairement à dans un tableau. Si tu as d'autres suggestions ou quelque choses auquel je n'ai pas pensé, merci de m'en faire part.