Tienes un lío importante en tu código, es confuso, difícil de seguir y complicado de razonar. Posiblemente sea por la inexperiencia que comentas; de ser así: no te preocupes, el tiempo lo solucionará.
Problema(s).
Para empezar, tu código, tal y como lo has compartido es imposible que compile; el archivo Personaje.h
no tiene declaración de clase o estructura. Supongo que querías escribir esto:
class caractere
{
private:
int p_vie, attaque;
string n_caractere;
public:
caractere(int vie, string nom);
int attaqueentre_personnages(caractere Personnage);
};
Seguidamente, tu nomenclatura es muy confusa, en general ayuda a entender mejor el código si hay concordancia entre el nombre del archivo y su contenido, tú no sigues dicha concordancia:
- El archivo
Personaje.h
contiene la clase caractere
.
- El archivo
Monstruo.h
contiene la clase monstre
.
Siguiendo con la nomenclatura, ni siquiera se en qué idioma estas escribiendo, si fuese catalán Personaje debería ser Personatje y Dragon sería Drac, lo de Caractere no se de dónde lo sacas... es muy confuso, además de que las variables miembro tienen nombres que parecen sacados de triturar juntos los diccionarios de varios idiomas. Tampoco es habitual referirse a las clases como Madres e Hijas si no como Base y Derivadas.
Para acabar, no acabas de entender cómo funciona la herencia de objetos en C++, como analogía intenta imaginar bloques de lego:
Cada objeto es un bloque único con características únicas, cuando derivas de un bloque es como construir una figura juntando los bloques de los que ya dispones:
Una de las ventajas de la herencia es que te evita repetir código y esfuerzos. Eso es lo que te ha pasado derivando dragon
de monstre
, echemos un vistazo:
Herencia de caractere
→ dragon
→ monstre
.
Vamos a echar un vistazo a tu árbol de herencia:
class caractere
{
private:
int p_vie, attaque;
string n_caractere;
public:
caractere(int vie, string nom);
int attaqueentre_personnages(caractere Personnage);
};
Su pieza de lego tendría este aspecto:
^^^^^^^^^^^ caractere ^^^^^^^^^^^
+-------+---------+-------------+
| p_vie | attaque | n_caractere |
+-------+---------+-------------+
De caractere
deriva monstre
:
class monstre: public caractere
{
protected:
int monstrevi;
string monstreno;
public:
monstre(int monstrev,string monstren);
};
Su pieza de lego tendría este aspecto:
|^^^^^^^^^^ caractere ^^^^^^^^^^|^^^^^^^ monstre ^^^^^^^|
+-------+---------+-------------+-----------+-----------+
| p_vie | attaque | n_caractere | monstrevi | monstreno |
+-------+---------+-------------+-----------+-----------+
De monstre
deriva dragon
:
class dragon: public monstre
{
private:
int dragonvie;
public:
dragon();
};
Su pieza de lego tendría este aspecto:
|^^^^^^^^^^ caractere ^^^^^^^^^^|^^^^^^^ monstre ^^^^^^^|^ dragon ^|
+-------+---------+-------------+-----------+-----------+-----------+
| p_vie | attaque | n_caractere | monstrevi | monstreno | dragonvie |
+-------+---------+-------------+-----------+-----------+-----------+
Así que un dragon
tiene como puntos de vida1:
p_vie
al ser caractere
.
monstrevi
al ser monstre
.
dragonvie
al ser dragon
.
Por ello, cuando haces un ataque ¿qué vida le restas? ¿la de caractere
, la de monstre
o la de dragon
?
Propuesta.
Cambia tus clases para no repetir código ni datos:
class caractere
{
int p_vie, attaque;
string n_caractere;
friend caractere &operator >(caractere &, caractere &);
public:
caractere(int vie, string nom);
};
class monstre: public caractere
{
// La vida la obtiene de 'caractere'
// El nombre lo obtiene de 'caractere'
public:
// Llamamos al constructor Base en la Derivada
monstre(int monstrev, string monstren) : caractere(monstrev, monstren){};
};
class dragon: public monstre
{
// La vida la obtiene de 'monstre' que la obtiene de 'caractere'
public:
// Llamamos al constructor Base en la Derivada
dragon() : monstre(1000000, "PufF, el drac màgic") {}
};
Para realizar el ataque entre personajes, te aconsejo usar una función externa en forma de operador, por ejemplo el operador >
para representar que A > B
significa A
ataca B
:
caractere &operator >(caractere &A, caractere &B)
{
B.p_vie -= A.attaque;
return B;
}
Puedes ver un ejemplo de mi propuesta en Wandbox.
Adicionalmente como nombre tendría:
n_caractere
al ser caractere
.
monstreno
al ser monstre
.
Degats
?