Partage

héritage

Sujet résolu

8 août 2007 à 21:46:11

salut

alors je commence les notions d'héritage car j'aimerais créer un rpg et je ne différencie pas quelque fonction comme celle du déplacement, etc cependant j'ai un petit problème

#include <iostream>
#include <SDL/SDL.h>
#include <SDL/SDL_image.h>
#include <SDL/SDL_ttf.h>
#include "Classes.h"
#include "constante.h"
//--------------------------classe Pnj----------------------------------------------------//
Pnj::Pnj(SDL_Rect depart)
{
_image[0] = new SDL_Surface;
_image[1] = new SDL_Surface;
_image[2] = new SDL_Surface;
_image[3] = new SDL_Surface;
_position.x = depart.x;
_position.y = depart.y;
}
Pnj::~Pnj()
{
delete[] _image;
}
//---------------------------------classe allies------------------------------------------//
Allies::Allies(SDL_Rect depart)
{
_image[0] = new SDL_Surface;
_image[1] = new SDL_Surface;
_image[2] = new SDL_Surface;
_image[3] = new SDL_Surface;
_position.x = depart.x;
_position.y = depart.y;
}
Allies::~Allies()
{
delete[] _image;
}

lorsque je compile ça avec les autres feuille de codes qui vont avec il me signal une erreur

erreur : no matching function for call to `Pnj::Pnj()'

je crois qu'il me dis qu'il n'y a pas de fonction qui appelle le constructeur de la classe mère mais je suis pas sûr de bien comprendre

si quelqu'un pouvait m'aider

merci d'avance icare

P.S: le fichier .h

#ifndef DEF_PNJ
#define DEF_PNJ
class Pnj
{
protected:
SDL_Surface *_image[4];
SDL_Rect _position;
public:
Pnj(SDL_Rect depart);
~Pnj();
};
class Allies:public Pnj
{
protected:
public:
Allies(SDL_Rect depart);
~Allies();
};
#endif

atoboldom

8 août 2007 à 21:53:14

Bonsoir.

Il faut que tu définisse ton costructeur par défaut.
Il me semble que cela est due à l'héritage(à vérifier).

Mais pas besoin de spécifier quoique ce sois dans le constructeur par défaut.

Sinon essaie en faisant ceci :

Allies::Allies(SDL_Rect depart) : Pnj::Pnj(depart)
{
_image[0] = new SDL_Surface;
_image[1] = new SDL_Surface;
_image[2] = new SDL_Surface;
_image[3] = new SDL_Surface;
_position.x = depart.x;
_position.y = depart.y;
}

lmghs

8 août 2007 à 22:00:34

PS: tes libérations ne sont pas bonnes.

Freedom

8 août 2007 à 22:02:31

Le problème vient de l'héritage, quand tu va créer un objet de la classe fille il va appeller le constructeur de la classe mère :

#include <iostream>
#include <SDL/SDL.h>
#include <SDL/SDL_image.h>
#include <SDL/SDL_ttf.h>
#include "Classes.h"
#include "constante.h"
//--------------------------classe Pnj----------------------------------------------------//
Pnj::Pnj(SDL_Rect depart)
{
_image[0] = new SDL_Surface;
_image[1] = new SDL_Surface;
_image[2] = new SDL_Surface;
_image[3] = new SDL_Surface;
_position.x = depart.x;
_position.y = depart.y;
}
Pnj::~Pnj()
{
delete[] _image;
}
//---------------------------------classe allies------------------------------------------//
Allies::Allies(SDL_Rect depart) : Pnj(depart)
{
_image[0] = new SDL_Surface;
_image[1] = new SDL_Surface;
_image[2] = new SDL_Surface;
_image[3] = new SDL_Surface;
_position.x = depart.x;
_position.y = depart.y;
}
Allies::~Allies()
{
delete[] _image;
}

#ifndef DEF_PNJ
#define DEF_PNJ
class Pnj
{
protected:
SDL_Surface *_image[4];
SDL_Rect _position;
public:
Pnj(SDL_Rect depart);
virtual ~Pnj();
};
class Allies:public Pnj
{
protected:
public:
Allies(SDL_Rect depart);
~Allies();
};
#endif

FaQ : Fr | En 1 2 | C++11 | Template || Blog : Deloget | C++|Boost--Dev | C++Next | GotW || Installer Boost

iNaKoll

8 août 2007 à 22:03:33

Je ne suis pas certain mais à mon avis le problème vient du fait que tu définis un constructeur prenant des paramètres pour une classe dérivée et que celui-ci ne fait pas appelle explicitement au constructeur de la classe de base.

Edit: grillé :colere2:

Inkamath on GitHub - Interpréteur d'expressions mathématiques. Reprise du développement en cours.

atoboldom

8 août 2007 à 22:07:28

Les deux grillé en plus

Pour les libérations il faut faire :

for(int i = 0; i < nbImages; i++)
delete _images[i];

Mais bon c'est plus simple d'utiliser un std::list ou un std::map et un iterateur pour la libération

icare14

8 août 2007 à 22:12:12

vous ètes sûr pour les libérations ? parce que dans le tutorial de mattéo il m'a semblé avoir vu que pour libérer un tableau delete[] suffisait

une autre question si y a un constructeur par defaut y a aussi un destructeur par defaut ou les fonctions qui marchent dans le même style ?

Freedom

8 août 2007 à 22:25:08

Pour les libération, quand tu alloue avec new tu libère avec delete, et quand tu utilise new[] tu libère avec delete[], il ne faut pas allouer avec un et libèrer avec l'autre comme tu as fait ic.

Il n'y as qu'un seul et unique destructeur par class : ~nom_de_la_class();

FaQ : Fr | En 1 2 | C++11 | Template || Blog : Deloget | C++|Boost--Dev | C++Next | GotW || Installer Boost

lmghs

8 août 2007 à 22:27:36

Sauf que là, ce n'est pas un tableau dynamique d'objets, c'est un tableau statique de pointeurs pointant chacun vers un objet alloué dynamiquement.
C'est un peu beaucoup différent.

Il peut y avoir 150 constructeurs différents, mais il n'y a qu'un seul destructeur.
Dans tous les cas, le chainage est obligatoire. Implicite pour le destructeur. Implicite vers le constructeur par défaut si rien n'est précisé ; explicite si précisé.

[Kansuu]

8 août 2007 à 22:31:02

Citation : atoboldom

Les deux grillé en plus
Pour les libérations il faut faire :
for(int i = 0; i < nbImages; i++)

delete _images[i];

Mais bon c'est plus simple d'utiliser un std::list ou un std::map et un iterateur pour la libération

delete[] est la bonne manière a faire...

quoi que ce sont des SDL_Surface, alors peut-être vaudrait mieux faire SDL_FreeSurface.

icare14

8 août 2007 à 22:49:21

Citation : lmghs

Il peut y avoir 150 constructeurs différents, mais il n'y a qu'un seul destructeur.Dans tous les cas, le chainage est obligatoire. Implicite pour le destructeur. Implicite vers le constructeur par défaut si rien n'est précisé ; explicite si précisé.

pardonne mon inculture mais le tutorial que j'ai trouvé parle pas de chainage

c'est quoi en fait ?

ceci dis je pensais si je fais une fonction deplacer dans ma classe mere et que j'en refais une dans la classe fille je dois aussi faire ce truc ?

void deplacer(int orientation) : Pnj(orientation)

(exemple vite fais

merci encore pour vos réponse

[Kansuu]

8 août 2007 à 22:54:54

Citation : icare14

Citation : lmghs

Il peut y avoir 150 constructeurs différents, mais il n'y a qu'un seul destructeur.Dans tous les cas, le chainage est obligatoire. Implicite pour le destructeur. Implicite vers le constructeur par défaut si rien n'est précisé ; explicite si précisé.

pardonne mon inculture mais le tutorial que j'ai trouvé parle pas de chainage c'est quoi en fait ?

ceci dis je pensais si je fais une fonction deplacer dans ma classe mere et que j'en refais une dans la classe fille je dois aussi faire ce truc ?

void deplacer(int orientation) : Pnj(orientation)

(exemple vite fais

merci encore pour vos réponse

Non, tu ne doit appeler que le constructeur de ta classe mère que lorsque le constructeur de ta classe fille est appeler. Ton objet est déjà créer pourquoi réappeler le constructeur?

Freedom

8 août 2007 à 22:58:31

Non si tu fait sa sa veut dire que ta fonction déplacer va appeller le constructeur de la classe mère avec comme paramètre (orientation) ce n'est pas l'objectif il me semble par contre tu doit déclarer la fonction de la classe mère qui à le même nom virtual, comme je l'ai fait plus au pour ton destructeur. Ce mot permet au langage de savoir quel fonction appelle selon que l'objet est de la classe mère ou de la classe fille.

Comme tes 2 fonction ont le même nom et qu'une classe hérite de l'autre elle va se retrouver avec 2 fonction de même nom, et de même paramètre si j'ai bien compris, donc même signature, le programme ne saura pas qu'elle est la bonne fonction à appellé, si le mot clé virtuelle est présent, si l'objet est de classe mère c'est la fonction de la classe mère qui est appellé et pareil pour la classe fille. D'ou l'utilité de déclarer la fonction de la classe mère virtuel.

Fait le avec les destructeurs de tes classes mère aussi.

FaQ : Fr | En 1 2 | C++11 | Template || Blog : Deloget | C++|Boost--Dev | C++Next | GotW || Installer Boost

iNaKoll

8 août 2007 à 23:12:58

Citation : icare14

Citation : lmghs

Il peut y avoir 150 constructeurs différents, mais il n'y a qu'un seul destructeur.Dans tous les cas, le chainage est obligatoire. Implicite pour le destructeur. Implicite vers le constructeur par défaut si rien n'est précisé ; explicite si précisé.

pardonne mon inculture mais le tutorial que j'ai trouvé parle pas de chainage c'est quoi en fait ?

Je crois que ce que lmghs veut dire par la c'est que : si un constructeur d'une classe dérivée prend des paramètres, on doit IMPERATIVEMENT EXPLICITER le constructeur de la classe de base que l'on veut utiliser. Dans les autres cas, ce chainage des constructeurs est fait implicitement par le compilateur avec les constructeurs par défaut. Arrêtez-moi si je me trompe, c'est nouveau pour moi aussi :-°

Inkamath on GitHub - Interpréteur d'expressions mathématiques. Reprise du développement en cours.

lmghs

8 août 2007 à 23:48:14

Le chainage, c'est le fait qu'un constructeur (/resp. destructeur) appelle _un_ constructeur (/resp. _le_ destructeur) de la classe parente.

Dans le cas des constructeur, par défaut, c'est le constructeut par défaut (des classes parentes) qui est appellé car rien n'est explicité. En explicitant, on choisit son constructeur. C'est tout ce que je voulais dire.

(Il est évident qu'avec une classe mère sans constructeur par défaut (comme ici), il faut obligatoirement expliciter le constructeur parent que l'on veut)

(je vous passe les cas particuliers des héritages multiples, virtuels ou non)

Et, NON, ce n'est pas delete[] qui doit être appelé ici!

[Kansuu]

8 août 2007 à 23:54:15

Citation : lmghs

Et, NON, ce n'est pas delete[] qui doit être appelé ici!

Ici non, mais c'est quand même delete[] qui faut utiliser et non la méthode de atoboldom.

lmghs

9 août 2007 à 0:52:56

Non. La méthode d'atoboldom est celle qui doit être employée avec les tableaux statiques de pointeurs -- ce qui ce trouve être ce qu'utilise l'OP. Il ne faut pas la rejeter parce qu'ailleurs on a juste des tableaux de variables. Ce sont deux choses différentes.

[Kansuu]

9 août 2007 à 2:15:02

Citation : lmghs

Non. La méthode d'atoboldom est celle qui doit être employée avec les tableaux statiques de pointeurs -- ce qui ce trouve être ce qu'utilise l'OP. Il ne faut pas la rejeter parce qu'ailleurs on a juste des tableaux de variables. Ce sont deux choses différentes.

Je voit ce que tu veux dire.

atoboldom

9 août 2007 à 3:12:40

Mais en même temps ma méthode est fausse dans le cas présent ?
Parceque c'est SDL_FreeSurface qui'il faut appeler non ?

[Kansuu]

9 août 2007 à 3:30:50

Citation : atoboldom

Mais en même temps ma méthode est fausse dans le cas présent ?
Parceque c'est SDL_FreeSurface qui'il faut appeler non ?

Si si, mais dans le cas contraire si ça n'aurais pas été des SDL_surface, ta méthode aurait été la bonne.

icare14

9 août 2007 à 10:36:23

si j'ai bien compris comme mon tableau est un tableau de pointeur non-variable (au niveau de la taille) je détruit chaque pointeur de ce tableau un par un

tandis que si c'était un simple tableau de valeur je pourrais utiliser delete[]

c'est bien ça ?

EDIT : une autre question : pour l'instant mon programme ne fait que déplacer un personnage à travers plusieurs niveau mais lorsque je fais ctrl + alt + delete je vois mon programme qui bouffe une mémoire pas possible (après deux minutes de déplacement jusqu'à 500 mo o_O

) je sais pas trop d'ou ça vient mais est ce que c'est possible que ça vienne du fait que j'ai créé une fonction qui blitte les image en les appellant puis en les détruisant

(ça serait plus clair avec le code

)

void blitterimage(SDL_Surface *ecran, int plateau[][NBR_CASE_HAUTEUR])
{
SDL_Surface *maison = NULL, *chateau = NULL, *arbre = NULL, *fond = NULL;
SDL_Rect posobjet;
fond = IMG_Load("image/environnement/fond_herbe.bmp");//image de fond
maison = IMG_Load("image/environnement/maison_chem.bmp");//maison avec cheminee
SDL_SetColorKey(maison, SDL_SRCCOLORKEY, SDL_MapRGB(maison->format, 255, 0, 0));//transparence
chateau = IMG_Load("image/environnement/chateau.bmp");
SDL_SetColorKey(chateau, SDL_SRCCOLORKEY, SDL_MapRGB(chateau->format, 0, 128, 0));//transparence
arbre = IMG_Load("image/environnement/arbre.bmp");
SDL_SetColorKey(arbre, SDL_SRCCOLORKEY, SDL_MapRGB(chateau->format, 0, 0, 255));
SDL_FillRect(ecran, NULL, SDL_MapRGB(ecran->format, 255, 255, 255)); /* On efface l'écran */
posobjet.x= 0;
posobjet.y = 0;
SDL_BlitSurface(fond, NULL, ecran, &posobjet);//on colle le fond
for(int i = 0; i < NBR_CASE_HAUTEUR; i++)//pour tout x
{
for(int j = 0; j < NBR_CASE_LARGEUR; j++)//pour tout Y
{
switch(plateau[j][i])//on compare le tableau
{
case VIDE:
break;
case MAISON:
posobjet.x = LARGEUR * j;
posobjet.y = HAUTEUR * i;
SDL_BlitSurface(maison, NULL, ecran, &posobjet);
break;
case CHATEAU:
posobjet.x = LARGEUR * j;
posobjet.y = HAUTEUR * i;
SDL_BlitSurface(chateau, NULL, ecran, &posobjet);
break;
case ARBRE:
posobjet.x = LARGEUR * j;
posobjet.y = HAUTEUR * i;
SDL_BlitSurface(arbre, NULL, ecran, &posobjet);
break;
}
}
}
SDL_FreeSurface(chateau);
SDL_FreeSurface(maison);
SDL_FreeSurface(arbre);
}

et voici comment j'utilise cette fonction :

...
blitterimage(ecran,plateau);//on préblitte toutes les images pour le décors
Prophete.deplacer(ecran, BAS, plateau_collision, plateau);
SDL_Flip(ecran);
while (continuer)//boucle evenementielle
{
SDL_WaitEvent(&event);
switch(event.type)
{
case SDL_QUIT:
continuer = 0;
break;
case SDL_KEYDOWN:
switch(event.key.keysym.sym)
{
case SDLK_UP: // Flèche haut
blitterimage(ecran, plateau);
Prophete.deplacer(ecran, HAUT, plateau_collision, plateau);
break;
case SDLK_DOWN: // Flèche bas
blitterimage(ecran,plateau);
Prophete.deplacer(ecran, BAS, plateau_collision, plateau);
break;
case SDLK_RIGHT: // Flèche droite
blitterimage(ecran,plateau);
Prophete.deplacer(ecran, DROITE, plateau_collision, plateau);
break;
case SDLK_LEFT: // Flèche gauche
blitterimage(ecran,plateau);
Prophete.deplacer(ecran, GAUCHE, plateau_collision, plateau);
break;
case SDLK_ESCAPE:
continuer = 0;
break;
}
break;
}
SDL_Flip(ecran); /* On met à jour l'affichage */
}
...

P.S : si vous voulez autre chose demandez moi

merci encore

iNaKoll

9 août 2007 à 11:26:36

Evite de faire des chargements d'images a chaque tour de ta boucle principale.. Il faut faire tous les chargements en une seul fois au début du programme...

Ensuite si tu as toujours un problème, vérifie que tu n'as pas de fuite de mémoire.

Inkamath on GitHub - Interpréteur d'expressions mathématiques. Reprise du développement en cours.

icare14

9 août 2007 à 12:30:47

merci inakoll ça marche

héritage

× Après avoir cliqué sur "Répondre" vous serez invité à vous connecter pour que votre message soit publié.

× Attention, ce sujet est très ancien. Le déterrer n'est pas forcément approprié. Nous te conseillons de créer un nouveau sujet pour poser ta question.

héritage

héritage

Supprimer

Modérer

Signaler le message aux modérateurs