[SOLUCIONADO] Error C++ al intentar pasar objetos de clase deriavada a clase base

[erikat]

Muy buenas! No se si el titulo del tema explique bien el error pero tratare de explicarlo lo mejor que pueda.

Tengo una clase base llamada Vehicle y una clase derivada llamada scooter. Posteriormente creo varios objetos de la clase scooter y, me pide el ejercicio, que cree un vector de punteros a objetos de tipo Vehicle y que asigne los punteros a los objetos scooter.

Dicho esto, aqui esta el codigo:

class Vehicle{
protected:
string brand, plate; //Deberia ser Private
public:
Vehicle(string b, string p) : brand (b), plate (p){}
string get_brand(){
return brand;
}
string get_plate(){
return plate;
}
void set_brand (string b1){
brand = b1;
}
virtual void print();
};

//Task 5

class Scooter : public Vehicle{
public:
Scooter(string b, string p) : Vehicle(brand,plate){}
virtual void print(){
cout << "Scooter " << brand << ", Plate: " << plate << endl;
}
};

int main(){
int n = 5;

Vehicle *vehicle[3*n+2];
int i = 0;

for (i=0; i<=n; i++){

ostringstream convert,convert1,convert2;
convert << "DU-CC" << n;
Scooter sc("Honda 110cc",convert.str());
vehicle[i] = static_cast<Vehicle*> (&sc);

}
return 0;
}

La verdad que estoy en proceso de aprendizaje y primera vez que tengo que trabajar con clases bases y derivadas, por lo que no se si lo que estoy haciendo es correcto. Estando asi, el error que me da es

undefined reference to ´vtable for Vehicle´

[CalgaryCorpus]

En la clase vehicle dijiste que print() es virtual, pero no indicaste un cuerpo para la funcion, por ejemplo,

Código C++:

virtual void print() {}
 

Una opcion es definir a la clase base como abstracta, haciendo que el metodo print sea virtual puro asi:

Código C++:

virtual void print() = 0;
 

[erikat]

Vale, muchas gracias.. funciono declarar la clase base como privada y ahora no me un error. Sin embargo no logro luego imprimir los datos.

Código C++:

#include<fstream>
#include<iostream>
#include <sstream>
using namespace std;
 
class Satnav{
private:
   int serialNo;
public:
    Satnav(int sn) : serialNo(sn){}
    string print (){
        cout << "With navi# " << serialNo;
    }
};
class Vehicle{
 protected:
    string brand, plate; //Deberia ser Private
 public:
    Vehicle(string b, string p) : brand (b), plate (p){}
    string get_brand(){
        return brand;
    }
    string get_plate(){
        return plate;
    }
    void set_brand (string b1){
        brand = b1;
    }
    virtual void print() = 0;
};
 
//Task 5
 
class Scooter : public Vehicle{
public:
     Scooter(string b, string p) : Vehicle(brand,plate){}
     virtual void print(){
     cout << "Scooter " << brand << ", Plate: " << plate << endl;
     }
};
 
//Task 6
class Car : public Vehicle{
protected:
    bool aircondition;
    Satnav *navigation;
 public:
    Car(string b, string p, bool ac = 1, Satnav *n=NULL) : Vehicle(brand,plate), aircondition(ac), navigation(n){}
    void set_navigation (Satnav *a){
        navigation = a;
    }
    bool get_aircondition(){
        return aircondition;
    }
    void no_aircondition(){
        aircondition = 0;
    }
    void printCarData(){
        cout << brand << ", plate" << plate;
        if (aircondition == 0){
            cout << ", no air condition";
        }
        else{
            cout << ", with air condition";
        }
    }
};
//Task 7
class Cabrio : public Car{
public:
    Cabrio(string b1, string p1, bool a, int *n) : Car(brand, plate, aircondition, navigation){}
    virtual void print(){
        cout << "Cabrio ";
        printCarData();
        cout << endl;
    }
};
//Task 8
class Hartop : public Car{
private:
    int seats;
public:
    Hartop(string b1, string p1, bool a, Satnav *n, int s) : Car(brand, plate, aircondition, navigation), seats(s){}
    virtual void print(){
    cout << "Hartop ";
    printCarData();
    cout << ", Seats:" << seats << endl;
    }
};
 
int main(){
    int n = 5;
 
    Satnav nav1(1);
    Satnav nav3(2);
    Satnav *nav2 = &nav3;
 
    Vehicle *vehicle[3*n+2];
    int i = 0;
 
    for (i=0; i<=n; i++){
 
        ostringstream convert,convert1,convert2;
        convert << "DU-CC" << n;
        Scooter sc("Honda 110cc",convert.str());
        vehicle[i] = static_cast<Vehicle*> (&sc);
 
        convert1 << "DU-Z" << n;
        Cabrio ca("BMW Z5",convert1.str(),1,0);
        vehicle[i+1] = static_cast<Vehicle *> (&ca);
 
        convert2 << "DU-TA" << n;
        Hartop ht ("Toyota Auris",convert2.str(),1,0,5);
        vehicle[i+2] = static_cast<Vehicle *> (&ht);
    }
 
    Hartop fg1("Ford Galaxy","DU-FG 1",1,&nav1,7);
    Hartop fg3("Ford Galaxy","DU-FG 2",1,&nav3,7);
    Hartop *fg2 = &fg3;
 
    vehicle[4] = reinterpret_cast<Vehicle*>(&fg1);
    vehicle[5] = reinterpret_cast<Vehicle*>(&fg2);
 
    int j=0;
    for (int j = 0; j <= 3*n+2; j++)
      vehicle[j]->print();
 
   return 0;
}
 

El programa compila pero parece que se quedara colgando en un bucle que no logro identificar.

Muchas gracias

[dehm]

Hola:

¿Estas líneas?:

Código C++:

vehicle[i] = static_cast<Vehicle*> (&sc);
vehicle[i+1] = static_cast<Vehicle *> (&ca);
vehicle[i+2] = static_cast<Vehicle *> (&ht);
vehicle[4] = reinterpret_cast<Vehicle*>(&fg1);
vehicle[5] = reinterpret_cast<Vehicle*>(&fg2);
 

¿Es la forma de pasar los objetos de la clase derivada a la clase base?
Lo pregunto porque no lo sé, y parece un tema interesante.

Por otro lado, yo siempre hago el método clásico

Código C++:

vehicle [i+1] = new Scooter("YAMAHA","123456");
vehicle[i+1]->print();
 

Por cierto, que tienes un fallo en los constructores:

Código C++:

Scooter(string b, string p) : Vehicle(brand,plate){}
 

debería ser:

Código C++:

Scooter(string b, string p) : Vehicle(b,p){}
 

(y así en los demás)

[erikat]

Iniciado por dehm

Hola:

¿Estas líneas?:

Código C++:

vehicle[i] = static_cast<Vehicle*> (&sc);
vehicle[i+1] = static_cast<Vehicle *> (&ca);
vehicle[i+2] = static_cast<Vehicle *> (&ht);
vehicle[4] = reinterpret_cast<Vehicle*>(&fg1);
vehicle[5] = reinterpret_cast<Vehicle*>(&fg2);
 

¿Es la forma de pasar los objetos de la clase derivada a la clase base?
Lo pregunto porque no lo sé, y parece un tema interesante.

Por otro lado, yo siempre hago el método clásico

Código C++:

vehicle [i+1] = new Scooter("YAMAHA","123456");
vehicle[i+1]->print();
 

Por cierto, que tienes un fallo en los constructores:

Código C++:

Scooter(string b, string p) : Vehicle(brand,plate){}
 

debería ser:

Código C++:

Scooter(string b, string p) : Vehicle(b,p){}
 

(y así en los demás)

Hola, gracias por tu respuesta,

bueno yo lo acabo de aprende y aun el programa no correo por lo que puedo estar 100% seguro pero segun lo que he investigado, si. Es la manera de manejar datos de una clase derivada como si fuesen de la clase base.

Muchas gracias por tus comentarios, los tomare en cuenta y a ver que pasa :)

[erikat]

Bueno ya la parte de las variables esta solucionada, y ya no se queda colgado pero sigue sin imprimir al usar el for

Código Python:

for (i=0; i<n; i++){
        ostringstream convert,convert1,convert2;
        convert << "DU-CC " << i+1;
        Scooter sc("Honda 110cc",convert.str());
        vehicle[i] = static_cast<Vehicle*> (&sc);
        //vehicle[i]->print(); CON ESTA LINEA IMPRIME PERO NO ES EL OBJETIVO
 
        convert1 << "DU-Z " << i+1;
        Cabrio ca("BMW Z5",convert1.str(),1,0);
        vehicle[i+1] = static_cast<Vehicle *> (&ca);
        //vehicle[i+1]->print();
 
        convert2 << "DU-TA " << i+1;
        Hartop ht ("Toyota Auris",convert2.str(),1,0,5);
        vehicle[i+2] = static_cast<Vehicle *> (&ht);
        //vehicle[i+2]->print();
    }
 
    Hartop fg1("Ford Galaxy","DU-FG 1",1,&nav1,7);
    Hartop fg3("Ford Galaxy","DU-FG 2",1,&nav3,7);
    Hartop *fg2 = &fg3;
 
    vehicle[4] = static_cast<Vehicle *> (&fg1);
    vehicle[4]->print();
    vehicle[5] = static_cast<Vehicle *> (fg2);
    vehicle[5]->print();
 
    for (int i=0; i<(3*n+2); i++) //NO IMPRIME
      vehicle[i]->print();
 

Da un error en la compilacion que dice: "pure virtual method called"

Agradeceria la ayuda :)