Tengo un problemilla con un ejercicio de una (proto)base de datos de empleados, que consiste en crear una aplicación de consola para almacenar datos introducidos por el usuario en un vector de elementos de una clase "Empleado". Dicha clase es abstracta y de ella heredan otras tres llamadas "Jefe", "Ingeniero" e "Investigador" y todas ellas son muy simples, tienen como miembros arrays de caracteres únicamente para almacenar los datos (nombre, salario, experiencia...).
Cuando digo que son simples me refiero a que no hay punteros ni vectores ni nada complejo dentro de ellas. El programa debe tener la opción de guardar los datos en un fichero binario y poder cargar dichos datos de vuelta al volver a ejecutar el programa.
Todo funciona correctamente, pero al intentar visualizar los datos una vez cargados del fichero binario el programa "casca" y no entiendo por qué.
Esta es la clase padre 'Empleado' :
class Empleado
{
public:
virtual ~Empleado() {} ;
virtual void MostrarDatos() = 0;
protected:
char mTipoEmpleado[16];
char mNombre[100];
char mApellido[100];
float mSalario;
private:
};
Esta es una de las clases que heredan de Empleado, en este caso es la clase 'Jefe' :
class Jefe : public Empleado
{
public:
Jefe();
Jefe(char Nombre[], char Apellido[], float Salario, unsigned int reunionesSemanales, unsigned int diasVacacionesAnuales);
virtual ~Jefe() {}
void MostrarDatos();
protected:
unsigned int mReunionesSemanales;
unsigned int mDiasVacacionesAnuales;
private:
};
Esta es la función Main:
int main(int argc, char** argv) {
cout << endl << " ///////////////////////////////////////////////////////////" << endl;
cout << " // BIENVENIDO AL GESTOR DE LA BASE DE DATOS DE EMPLEADOS //" << endl;
cout << " // //" << endl;
cout << " ///////////////////////////////////////////////////////////" << endl;
int eleccion = 0;
bool salir = false;
vector<Empleado*> empleados;
while (!salir)
{
cout << endl << " Escoge una opcion por favor : " << endl;
cout << endl << " 1.- Agregar empleado" << endl;
cout << " 2.- Eliminar empleado" << endl;
cout << " 3.- Guardar base de datos" << endl;
cout << " 4.- Cargar base de datos" << endl;
cout << " 5.- Mostrar base de datos" << endl;
cout << " 6.- Salir" << endl;
cout << " Opcion : ";
cin >> eleccion;
switch (eleccion)
{
case 1:
{
system("cls");
bool salirTipoEmpleado = false;
int tipoEmpleado = 0;
while (!salirTipoEmpleado)
{
cout << endl << " Escoge tipo de empleado por favor : " << "\n";
cout << endl << " 1.- Jefe" << "\n";
cout << " 2.- Ingeniero" << "\n";
cout << " 3.- Investigador" << "\n";
cout << " 4.- Volver" << "\n";
cout << " Opcion : ";
cin >> tipoEmpleado;
switch (tipoEmpleado)
{
case 1:
{
empleados.push_back(new Jefe);
system("cls");
cout << " Nuevo Jefe incluido con exito en la base de datos." << "\n";
}
break;
case 2:
{
empleados.push_back(new Ingeniero);
system("cls");
cout << " Nuevo Ingeniero incluido con exito en la base de datos." << "\n";
}
break;
case 3:
{
empleados.push_back(new Investigador);
system("cls");
cout << " Nuevo Investigador incluido con exito en la base de datos." << "\n";
}
break;
case 4:
salirTipoEmpleado = true;
break;
system("cls");
} // Fin switch
} // Fin while(!salirTipo...)
} // Fin case 1
break;
case 2:
{
system("cls");
if (empleados.size() == 0)
cout << " Base de datos vacia." << "\n";
else
{
unsigned int indice = -1;
while (indice < 0 || indice >= empleados.size())
{
cout << " La base de datos tiene " << empleados.size() << " empleados." << "\n";
cout << " Elige empleado a borrar (primer empleado es el numero 0) : ";
cin >> indice;
if (indice < 0 || indice >= empleados.size()) cout << " Fuera de rango." << "\n";
else
{
empleados.erase(empleados.begin() + indice);
cout << " Empleado " << indice << " borrado." << "\n";
indice = 0;
if (empleados.size() == 0) break;
}
}
}
}
break;
case 3:
{
ofstream guardarDatos("Base_datos_Empleados.dat", ios::binary);
GuardarDatosBinario("Base_datos_Empleados.dat", empleados);
if (guardarDatos.good()) cout << " Base de datos guardada correctamente." << "\n";
else cout << " Error al guardar Base de datos." << "\n";
}
break;
case 4:
{
ifstream cargarDatos("Base_datos_Empleados.dat", ios::in | ios::binary | ios::ate);
if (cargarDatos.is_open())
{
empleados = LeerArchivoBinario("Base_datos_Empleados.dat");
}
else cout << " Error al cargar datos." << "\n";
}
break;
case 5:
{
system("cls");
if (empleados.size() == 0)
{
cout << " Base de datos vacia.";
break;
}
else cout << " La base de datos tiene " << empleados.size() << " empleados. " << "\n";
for (Empleado* emp : empleados)
{
emp->MostrarDatos();
cout << endl;
}
}
break;
case 6:
{
empleados.erase(empleados.begin(), empleados.end());
if (empleados.size() == 0)
cout << " Base de datos borrada. " << "\n";
else cout << " Fallo al borrar la base de datos. " << " \n";
} // Fin case 5
salir = true;
break;
} // Fin switch(eleccion)
} // Fin while(!salir)
return 0;
}
Esta es la función que guarda los datos en un fichero binario. Toma como argumentos un nombre de archivo y un vector de punteros a 'Empleado':
void GuardarDatosBinario(const char* archivoBinario, const vector<Empleado*> empleados)
{
ofstream guardarDatos(archivoBinario, ios::binary);
for (const Empleado* emp : empleados)
guardarDatos.write(reinterpret_cast<char*>(&emp), sizeof(emp));
}
Y esta es la que carga los datos del archivo binario en el vector de punteros a Empleados :
vector<Empleado*> LeerArchivoBinario(const char* archivoBinario)
{
vector<Empleado*> temporal;
ifstream cargarDatos(archivoBinario, ios::binary);
Empleado* empleadoTemp;
while (cargarDatos.read(reinterpret_cast<char*>(&empleadoTemp),sizeof(empleadoTemp)))
temporal.push_back(empleadoTemp);
return temporal;
}
Aparentemente los datos se cargan porque esta línea
cout << " La base de datos tiene " << empleados.size() << " empleados. " << "\n";
incluida justo antes del bucle para visualizar los datos del vector de punteros 'Empleado' responde el número correcto (2, 3, 7, 10...) pero inmediatamente después al empezar a recorrer el vector aparece la temida "segmentation fault".
Este es el bucle
for (Empleado* emp : empleados)
{
emp->MostrarDatos(); //-----> AQUÍ APARECE EL ERROR.
cout << endl;
}
Y ésta es la función para visualizar los datos, en este caso de la clase 'Jefe' pero las otras son iguales:
void Jefe::MostrarDatos()
{
std::cout << " Tipo empleado : " << mTipoEmpleado << "\n";
std::cout << " Nombre : " << mNombre << "\n";
std::cout << " Apellido : " << mApellido << "\n";
std::cout << " Salario : " << mSalario << "\n";
std::cout << " Reuniones Semanales : " << mReunionesSemanales << "\n";
std::cout << " Dias de vacaciones anuales : " << mDiasVacacionesAnuales << "\n";
}
En fin que no entiendo por qué no funciona. Me pregunto si tendrá algo que ver que al ser 'Empleado' una clase abstracta no se pueda 'grabar' tan fácilmente en un fichero binario debido a la 'vtable' y que haya que usar serialización o algo por el estilo, que de momento es ciencia-ficción para mí.
Cualquier orientación al respecto es más que bienvenida.