// No uses el plural cuando te refieres a objetos.
class Pokemon
{
protected:
// Inicializa los miembros como vacíos
std::string nombre{};
int vida{};
int ataque{};
public:
// Usa un constructor para asignar valores internos.
Pokemon(std::string Nombre, int Vida, int Ataque) :
nombre{Nombre}, vida{Vida}, ataque{Ataque}
{}
// La función no modifica el objeto: debe ser 'const'.
// Puedes devolver una referencia constante al nombre, evitando copias de datos innecesarias.
// Ya sabemos que estas leyendo el nombre, el 'get' es redundante (y mezcla idiomas).
const std::string &Nombre() const
{
// No hay ningún otro 'nombre' aquí, el 'this->' es innecesario.
return nombre;
}
// La función no modifica el objeto: debe ser 'const'.
// Ya sabemos que estas leyendo la vida, el 'get' es redundante (y mezcla idiomas).
int Vida() const
{
// No hay otra 'vida' aquí, el 'this->' es innecesario.
return vida;
}
// La función no modifica el objeto: debe ser 'const'.
// Ya sabemos que estas leyendo el ataque, el 'get' es redundante (y mezcla idiomas).
int Ataque() const
{
// No hay otro 'ataque' aquí, el 'this->' es innecesario.
return vida;
}
virtual void recibirDanio(int) = 0;
// Si el cuerpo del destructor va a estar vacío, deja que el compilador se ocupe.
virtual ~Pokemons~Pokemon() = default;
// Evita los punteros, usa referencias.
virtual void atacar(Pokemons&Pokemon& objetivo) = 0;
};
// Si todos los miembros van a ser públicos, valora usar una struct
struct Pikachu: public Pokemon
{
// Llama al constructor de la clase base
Pikachu() : Pokemon{"Pikachu", 100, 50}
{}
void recibirDanio(int danio) {
// No hay otra 'vida' aquí, el 'this->' es innecesario.
vida -= danio/2;
}
// No pretendes sobrecargar más esta función, el 'virtual' es innecesario.
// Pretendes sobrecargar la función base, no olvides el 'override'
// Evita los punteros, usa referencias.
// No devuelves ningún valor, la función debería ser 'void'.
void atacar(Pokemons&Pokemon& target) override {
// No hay otro 'ataque' aquí, el 'this->' es innecesario.
target.recibirDanio(ataque);
}
};