4

Estoy trabajando en un código en el cual por motivos de diseño tengo que hacer herencia múltiple, lo que me llevo al llamado problema del diamante, aunque en C++ se puede solucionar haciendo herencia virtual, tengo problemas con inicializar el atributo con el constructor parametrizado.

// ======== Jerarquia de clases ========

class Base{
    std::string dato_{};
public:
    Base() = default;
    Base(const std::string &dato)
        : dato_{dato}
    {}
    void print() const{
        std::cout<<dato_<<'\n';
    }
};

class Derivada_A : virtual public Base{
public:
    Derivada_A(const std::string &dato)
        : Base(dato)
    {}
};

class Derivada_B : virtual public Base{
public:
    Derivada_B(const std::string &dato)
        : Base(dato)
    {}
};

class Operador : public Derivada_A, public Derivada_B{
public:
    Operador(const std::string &dato)
        : Derivada_A(dato)
        , Derivada_B(dato)
    {}
};

// ======== Fallo al instanciar objeto con herencia multiple ========

int main(){
    Operador operador("Hola Mundo");

    operador.print(); //--> no imprime hola mundo en pantalla

    return 0;
}

2 respuestas 2

2

Si pones chivatos en las funciones, verás que no se está llamando Base::Base(const std::string &) en consecuencia Base::dato_ es una cadena vacía:

class Base{
    std::string dato_{};
public:
    Base() = default;
    Base(const std::string &dato)
        : dato_{dato}
    {
        std::cout << __PRETTY_FUNCTION__ << '\t' << dato << '\n';
    }
    void print() const{
        std::cout << '[' << dato_ << "]\n";
    }
};

class Derivada_A : virtual public Base{
public:
    Derivada_A(const std::string &dato)
        : Base(dato)
    {
        std::cout << __PRETTY_FUNCTION__ << '\t' << dato << '\n';
    }
};

class Derivada_B : virtual public Base{
public:
    Derivada_B(const std::string &dato)
        : Base(dato)
    {
        std::cout << __PRETTY_FUNCTION__ << '\t' << dato << '\n';
    }
};

Con esos cambios la salida del programa es:

Derivada_A::Derivada_A(const string&)   Hola Mundo
Derivada_B::Derivada_B(const string&)   Hola Mundo
[]

Para solucionar este problema, debes traer al ámbito de las clases derivadas el constructor de su base:

class Derivada_A : virtual public Base{
public:
    using Base::Base; // Trae el constructor de la base a este ambito
};

class Derivada_B : virtual public Base{
public:
    using Base::Base; // Trae el constructor de la base a este ambito
};

class Operador : public Derivada_A, public Derivada_B{
public:
    using Derivada_A::Derivada_A; // Trae el constructor de la primera base a este ambito
    using Derivada_B::Derivada_B; // Trae el constructor de la segunda base a este ambito
};

Con esos cambios la salida del programa pasa a ser:

Base::Base(const string&)   Hola Mundo
[Hola Mundo]
2
  • Problema resuelto, aunque sigo sin entender como es que using reemplaza las llamadas a los constructores, es una caracteristica de c++ que no conocia 👍
    – Eipo
    Commented el 29 ene. a las 15:25
  • La explicación la ha dado @Mateo en la otra respuesta. Commented el 30 ene. a las 8:44
2

En la herencia virtual, el constructor de la clase base es llamado directamente. Como en la lista no indicas nada, se llama al constructor por defecto. Deberías escribir lo siguiente:

Operador(const std::string &dato)
    : Base(dato), Derivada_A(dato), Derivada_B(dato)
{}

Es importante que el constructor de la clase Base esté antes que los otros.

2
  • 1
    Mateo ¿No es eso un typo?, en el constructor de Operador con referencia a cadena estás llamando al mismo constructor Operador con referencia a cadena. Commented el 29 ene. a las 11:59
  • @Paper tienes razón. Gracias
    – Mateo
    Commented el 29 ene. a las 15:37

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.