Skip to main content
Commonmark migration
Origen Enlace

El error 0xC0000005 se corresponde a un uso no válido de memoria, en concreto a memoria no inicializada… error al que es muy propenso tu código pues usas punteros en crudo si necesidad alguna de hacerlo. Te propongo ciertos cambios: ###Propuesta.

Propuesta.

El error 0xC0000005 se corresponde a un uso no válido de memoria, en concreto a memoria no inicializada… error al que es muy propenso tu código pues usas punteros en crudo si necesidad alguna de hacerlo. Te propongo ciertos cambios: ###Propuesta.

El error 0xC0000005 se corresponde a un uso no válido de memoria, en concreto a memoria no inicializada… error al que es muy propenso tu código pues usas punteros en crudo si necesidad alguna de hacerlo. Te propongo ciertos cambios:

Propuesta.

se eliminaron 3 caracteres en el cuerpo
Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86
// 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);
    }
};
// 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() = default;
    // Evita los punteros, usa referencias.
    virtual void atacar(Pokemons& 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& target) override {
        // No hay otro 'ataque' aquí, el 'this->' es innecesario.
        target.recibirDanio(ataque);
    }
};
// 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 ~Pokemon() = default;
    // Evita los punteros, usa referencias.
    virtual void atacar(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(Pokemon& target) override {
        // No hay otro 'ataque' aquí, el 'this->' es innecesario.
        target.recibirDanio(ataque);
    }
};
se añadió 1 carácter en el cuerpo
Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86
// 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() = default;
    // Evita los punteros, usa referencias.
    virtual intvoid atacar(Pokemons& objetivo) = 0;
};
int main()
{
    // No necesitas memoria dinámica para crear la lista.
    // Puedes crear los Pokémon al mismo tiempo que la lista
    list<Pokemon*> listaPokemons
    {
        new Pikachu, new Charmander, new Squirtle
    };

    // Puedes listar con bucles lo que tienes en la lista, en lugar de uno a uno
    for (const auto &pokemon : listaPokemons)
    {
        // Un solo 'cout' es suficiente
        cout << pokemon->Nombre() << '\n'
             << pokemon->Vida() << '\n'
             << pokemon->Ataque() << '\n'
             // Si solo imprimes un carácter, no uses un literal de cadena.
             // un 'endl' por llamada es suficiente.
             << '\n' << endl;
    }

    // … hacer cosas    

    // Puedes borrar el contenido de la lista con otro sencillo bucle:
    for (const auto &pokemon : listaPokemons)
        delete *pokemon;pokemon;    

    return 0;
}
// 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() = default;
    // Evita los punteros, usa referencias.
    virtual int atacar(Pokemons& objetivo) = 0;
};
int main()
{
    // No necesitas memoria dinámica para crear la lista.
    // Puedes crear los Pokémon al mismo tiempo que la lista
    list<Pokemon*> listaPokemons
    {
        new Pikachu, new Charmander, new Squirtle
    };

    // Puedes listar con bucles lo que tienes en la lista, en lugar de uno a uno
    for (const auto &pokemon : listaPokemons)
    {
        // Un solo 'cout' es suficiente
        cout << pokemon->Nombre() << '\n'
             << pokemon->Vida() << '\n'
             << pokemon->Ataque() << '\n'
             // Si solo imprimes un carácter, no uses un literal de cadena.
             // un 'endl' por llamada es suficiente.
             << '\n' << endl;
    }

    // … hacer cosas    

    // Puedes borrar el contenido de la lista con otro sencillo bucle:
    for (const auto &pokemon : listaPokemons)
        delete *pokemon;    

    return 0;
}
// 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() = default;
    // Evita los punteros, usa referencias.
    virtual void atacar(Pokemons& objetivo) = 0;
};
int main()
{
    // No necesitas memoria dinámica para crear la lista.
    // Puedes crear los Pokémon al mismo tiempo que la lista
    list<Pokemon*> listaPokemons
    {
        new Pikachu, new Charmander, new Squirtle
    };

    // Puedes listar con bucles lo que tienes en la lista, en lugar de uno a uno
    for (const auto &pokemon : listaPokemons)
    {
        // Un solo 'cout' es suficiente
        cout << pokemon->Nombre() << '\n'
             << pokemon->Vida() << '\n'
             << pokemon->Ataque() << '\n'
             // Si solo imprimes un carácter, no uses un literal de cadena.
             // un 'endl' por llamada es suficiente.
             << '\n' << endl;
    }

    // … hacer cosas    

    // Puedes borrar el contenido de la lista con otro sencillo bucle:
    for (const auto &pokemon : listaPokemons)
        delete pokemon;    

    return 0;
}
Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86
Loading