Probleme avec le polymorphisme et les templates :(

bonjour a tous,

excusez moi du dérangement mais voila,j ai un gros probleme avec le

polymorphisme et les templates en C++ :(

et je dois rendre un exercice a mon prof pour mercredi soir au plus tard

je vous explique en quelque mot ce que mon programme doit faire :

- tout d abord,on as une classe Billet (qui est abstraite,c est a dire qu on ne peut pas créer d objet de ce type).

- ensuite on a une classe EuroBillet (abstraite également) qui hérite de la classe Billet

- ensuite on a une multitude de classes telles que : EuroB5,EuroB10,EuroB20,etc .... qui héritent de la classe EuroBillet

- fiinalement,on a une classe totalement independante des autres, appellée : Portefeuille

dans cette classe Portefeuille,c est donc un pointeur sur tableau de pointeurs sur objet de cette maniere :

Billet ** _TabPort ;

- dans ce portefeuille donc,on doit etre capable d y placer des objets de differentes classes (EuroB5,EuroB10,etc...)

je dois donc utiliser les fonctions virtuelles et les templates.

les fonctions virtuelles j ai réussit a les faire.

mais les templates,j y comprends rien car j étais malade,le jour ou ils ont vu ca en classe :(

pour ne pas flooder ici , je vous mets le lien ou vous pouvez prendre mon code

ce serait sympa de m'aider ,et je vous en remercie tous d'avance ;)

http://perso.latribu.com/softstar/portefeuille.cpp

PS : on utilise Dev C++

En fait, on peut sans doute dire que tes billets sont des classes templates héritant de EuroBillet.
Pourquoi ?
Parce que ces billets ont une valeur générique, 5€, 10€, 20€ qui est connue à la compilation.
Donc outre ta classe EuroBillet, tu auras une classe

Code:

---

1 2 3 4 5 6 7 8 9

template<unsigned int Valeur> classe EuroB : public EuroBillet { public: EuroB() :EuroBillet(Valeur) {} }

---

En admettant que c'est l'objet parent qui gère la valeur du billet.

edit : après avoir regardé ton code, c'est en gros ça que tu vas faire, le constructeur fils appelle le constructeur père avec la bonne valeur de la monnaie et la vraie valeur - imontant -

merci,mais ca ne fonctionne pas :(

quand je compile,ca me mets ceci :

128 C:\PROGRAMMATION\C++\GI3_C++_NEW\MesProg\portefeuille\portefeuille.cpp

multiple types in one declaration

Bonjour,
je ne vais pas te faire un cours sur les templates, et je ne vais pas non plus faire le code.

Tu peux déjà regarder les templates, c'est dans la faq C++ de developpez. Sinon, tu peux très bien utiliser ton portefeuille sans template (ton prof t'oblige peut-être à les utiliser).
Le code ressemblerait à ceci avec la déclaration Billet ** _TabPort; :

Code:

---

1 2 3 4 5

_TabPort = new Billet [50]; _TabPort[0] = new EuroB5(); _TabPort[1] = new EuroB10(); ...

---

Donc ici, tu n'utilises pas de template. Je pense que votre prof voudrait peut-être un truc du style :

Code:

---

1 2 3 4 5

_TabPort = new Billet [50]; _TabPort[0] = new Billet<EuroB5>; _TabPort[1] = new Billet<EuroB10>; ...

---

Donc la par contre tu utilises les templates, je pense (voir d'autres avis) que tu dois juste rendre Billet template (je crois pas que EuroBillet ait besoin de l'être).

Euh, j'espere que j'ai répondu à une partie de la question car Miles rend template la valeur du billet.

Bonne journée.

b Oo

merci b O o , ca je suis arrivé a le faire,j'arrive a placer des billets de n importe quel type dans mon portefeuille

par contre,la ou ca coince , c est quand par exemple j utilise la fonction :

DelBillet(uint pos) ; // on retire un billet du portefeuille

car je dois renvoyer le billet par référence
(pour par exemple pouvoir l utiliser pour faire un achat , c'est symbolique bien sur)

et si je renvoye un objet de type Billet comme ceci :

Billet & DelBillet(uitn pos)

ben,le programme plante en me disant que je ne peut renvoyer un objet de type billet car Billet est une classe abstraite :(

donc je suppose que je doit utiliser les templates pour dire a ma fonction
DelBillet de renvoyer le bon type de billet

mais je vois pas comment faire :(

merci a tous de votre aide en tout cas ;)

Je viens de voir ton programme, en fait, ce que j'ai écris précèdemment remet en question tout ton programme.

Miles t'indique (je pense que ceci est le mieux), que lorsque tu définies ta classe EuroB template, cela t'évite de faire une classe pour chaque billet, par exemple :

Code:

---

1 2 3 4

EuroB<5> b5; //cree un billet de 5 euros EuroB<1à> b10; //cree un billet de 10 euros ...

---

Donc cela va t'enlever pas mal de classes.
Je te montre une classe template pour exemple :

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 34 35 36 37 38 39

template <class T = int> // = int permet de dire que par défaut T est int class TVecteur { public: TVecteur(int = 0); TVecteur(const TVecteur<T> &); void AddElement(const T & val); TVecteur<T> operator=(const TVecteur<T> &); virtual ~TVecteur(); private: T * vect; int taille; // taille actuelle du tableau int capacite; // taille max du tableau };   template <class T> TVecteur<T>::TVecteur(int _capacite):vect(0),taille(0), capacite(_capacite) { vect = new T [capacite]; }   template <class T> TVecteur<T>::TVecteur(const TVecteur<T> & v): vect(0), taille(0), capacite(0) { if (this != &v) Recopie(this, &v); }   template <class T> void TVecteur<T>::AddElement(const T & val) { ... } template <class T> TVecteur<T> TVecteur<T>::operator=(const TVecteur<T> & v) { ... }

---

Quand tu fais une classe template, la définition des méthodes doit se faire dans le .h

b Oo

ohhh la la la la , tu viens de m'embrouiller la boo :(

en fait,j ai pas le choix,le prof veut qu on utilise les classes que je t ai dites

donc ,créer une classe template pour eviter de creer des classes séparées,ca le fera pas :(

je rappelle mes classes obligatoires :

Billet , classe abstraite

EuroB , classe abstraite , qui hérite de Billet

EuroB5,EuroB10,EuroB20,EuroB50,EuroB100 , qui héritent toutes de EuroB

et Portefeuille qui est independant

comme je l'ai dit auparavant,mon probleme c est dans la classe Pörtefeuille

je crée mon portefeuille,j y mets les differents billets (EuroB5,EuroB10,etc..)

mais c est quand je veut par exemple en supprimer un (l enlever du portefeuille pour l utiliser par la suite (donc le renvoyer par reference))

que la ca plante car je vois pas comment lui dire dans la fonction DelItem(), que il doit renvoyer l'objet du type : EuroB5,
ou EuroB10,
ou EuroB20,
ou EuroB50,
ou EuroB100
voila , je sais pas comment etre plus clair :( parfois je m y perds un peu , snifff

merci en tout cas de votre aide,vous etes tous super ;)

Je pense que le problème vient du fait que, DelBillet supprime le billet du portfeuille (donc tu ne l'as plus en mémoire) et donc tu ne pas retourner une référence de celui-ci car tu l'as supprimé.
Par contre, si tu veux le recuperer dans le programme principal, tu ne peux faire :

Code:

---

1 2	Billet monBillet = monPorteFeuille.DelBillet();

---

par contre tu peux faire :

Code:

---

1 2	Billet * monBillet = monPorteFeuille.DelBillet();

---

Si DelBillet() renvoie un pointeur et si tu ne supprime pas le billet de la mémoire.

J'espère que cela répond à ta question.

b Oo

Edit: "C'est bon j'ai bien compris là, c'est juste que dans le 1er post tu ne fais pas mention du problème de DelBillet"

bon je te montre ce que j ai fait,et ca me plante a la figure :(

il compile sans probleme

dans mon main() , je fais ceci :

pf1.DelBillet(1);
pf1.Display();

donc je retire le billet a la position 1 du portefeuille (donc position 0 en c++)
puis j affiche le contenu du portefeuille
mais a l'execution , il m affiche bien ce que je veut
puis windows fait l'erreur systeme suivante :

l'instruction a "0x00409089" réference la memoire a "0x06fffffc" , la memoire ne peut pas etre "ecrite" , clicker sur ok pour terminer,clicker sur cancel pour debugguer le programme.

voici ma fonction Delitem(),si vous savez m'aider ;) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

Billet & DelBillet(uint pos){ if (!IsBillet(pos)) throw "Erreur"; total-=_TabPort[pos]->GetMontant(); Billet & temp=*_TabPort[pos]; delete _TabPort[pos]; _TabPort[pos]=0;   for(uint i=pos; i<_Contenance; i++) { if (i+1<_Contenance) _TabPort[i]=_TabPort[i+1]; else _TabPort[i]=0; } return temp; }

---

voila ;) je vous remercie tous d'avance ;)

plus personne , snif :( ?

Citation:

---

Envoyé par softstar

en fait,j ai pas le choix,le prof veut qu on utilise les classes que je t ai dites

donc ,créer une classe template pour eviter de creer des classes séparées,ca le fera pas :(

---

Bon, alors il faut les mettre où les templates ? Parce que là, il n'y a rien qui ne soit pas du polymorphisme "classique".

miles,possible que finalement je ne doivent pas utiliser les templates

par contre,cfr mon post un peu plus haut

j'ai un probleme avec ma fonction delbillet :(

tu sais y jeter un oeil svp ? je t'en remercie d avance ;)

Ben en fait tu déclares temp dans DelBillet, donc comme temp est une référence sur _TabPort[pos] que tu détruis juste après, temp ne fait plus référence à aucun objet.
Voila pourquoi cela plante.
Je pense que le code suivant fait ce que tu attends, mais la ca ne va compiler :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

Billet & DelBillet(uint pos) { if (!IsBillet(pos)) throw "Erreur"; total-=_TabPort[pos]->GetMontant(); Billet temp = *_TabPort[pos]); /* va te dire que Billet est une classe abstraite */ delete _TabPort[pos]; _TabPort[pos]=0; for(uint i=pos; i<_Contenance; i++) { if (i+1<_Contenance) _TabPort[i]=_TabPort[i+1]; else _TabPort[i]=0; } return temp; }

---

En plus, ici temp est un objet qui est détruit à la fin de la méthode, donc la référence sera encore invalide, mais cette fois dans le programme principal.

Et maintenant le code que tu veux faire et qui je pense fonctionne :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

Billet & DelBillet(uint pos) { if (!IsBillet(pos)) throw "Erreur"; total-=_TabPort[pos]->GetMontant(); Billet * temp = _TabPort[pos]; _TabPort[pos]=0; for(uint i=pos; i<_Contenance; i++) { if (i+1<_Contenance) _TabPort[i]=_TabPort[i+1]; else _TabPort[i]=0; } return *temp; }

---

La par contre, le problème c'est que pour le supprimer on va avoir du mal, (a moins que delete &nomdelref fonctionne, mais je ne suis pas sûr). Et en plus je ne sais pas s'il va pas te dire qu'on utilise Billet.

Ce que je ferais, moi :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

Billet * DelBillet(uint pos) { if (!IsBillet(pos)) throw "Erreur"; total-=_TabPort[pos]->GetMontant(); Billet * temp = _TabPort[pos]; _TabPort[pos]=0; for(uint i=pos; i<_Contenance; i++) { if (i+1<_Contenance) _TabPort[i]=_TabPort[i+1]; else _TabPort[i]=0; } return temp; }

---

Ici je pense qu'il n'y a pas de problème, il faut juste que tu n'oublis pas de faire un delete.

Je sais que toi tu veux :

Code:

---

1 2	Billet & DelBillet(uint pos)

---

Essaye le code 2, mais je ne suis pas convaincu que cela fonctionne.
Sinon, si tu ne veux absolument pas de pointeur, alors il ne faut pas rendre Billet abstraite. Et alors ta méthode sera la suivante :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

Billet DelBillet(uint pos) { if (!IsBillet(pos)) throw "Erreur"; total-=_TabPort[pos]->GetMontant(); Billet temp = _TabPort[pos]; delete _TabPort[pos]; _TabPort[pos]=0; for(uint i=pos; i<_Contenance; i++) { if (i+1<_Contenance) _TabPort[i]=_TabPort[i+1]; else _TabPort[i]=0; } return temp; }

---

J'espere avoir été clair.

A+

b Oo

Merciiiiiii Booo ;)

ton code 2 fonctionne nickel

merci encore a toi ;)

Tant mieux,
personnellement je pense qu'utiliser des références sur des objets alloués dynamiquement est source d'erreurs, donc fait attention à cette utilisation.

Sinon, met le sujet à résolu (tu vas dans outils de discussion en hau tde ton 1er post et ensuite mettre sujet à résolu).

Bonne journée.

A+

b Oo