Errores en código

[eferion]

Buenos días.

Bienvenido al foro.

Cuando tengas dudas de este estilo no es necesario, ni recomendable, que pongas todo el código. Lo que tienes que hacer es intentar presentar la menor porción de código, que a ser posible que compile, y en el que se pueda reproducir el error.

Si resulta que tu programa tiene 20k líneas de código... también lo vas a publicar así? no, verdad?

No uses variables globales

Código C++:

//declaraciones
int n,i,a,z=1;
ofstream archivo;//archivo de texto
 
empleado varempleado [1];//empleado
 

En pocas ocasiones estará justificado el uso de variables globales. Tu programa no es el caso.

¿Por qué es conveniente evitar su uso?

Por varias razones. Una de ellas es que estas variables existen durante toda la vida del programa, con lo que hay que ser muy cuidadoso a la hora de inicializar su valor cuando procede. Por ejemplo, si reutilizas "i" como contador y en una de estas se te olvida inicializarlo a 0 tu programa se comportará de forma extraña y te costará bastante esfuerzo encontrar el problema.

Si resulta que utilizas un valor compartido entre varias funciones lo puedes pasar como argumento, pero, en serio, intenta no usar variables globales.

Por ejemplo, archivo lo usas en la función reporte... no hay nada de malo en usar una variable local:

Código C++:

void reporte(){
  // Yo, personalmente, prefiero mostrar este tipo de mensajes
  // DESPUES de haber completado la operación
  // Piensa que si, por algúna razón, el salvado falla el usuario va
  // a pensar que la información se ha volcado al fichero correctamente
  system ("cls");
  cout<<"Reporte generado."<<endl;        
  
  ofstream archivo;
  archivo.open("Empleados.txt",ios::app);
  archivo<<"Datos del empleado";
  archivo<<endl;
  archivo.close();
}
 

Por otro lado...

Código C++:

empleado varempleado [1];//empleado
 

Un array de tamaño 1????? para eso ahórrate el array, código más eficiente y más claro:

Código C++:

empleado varempleado;//empleado
 

main no necesita declaración

La línea 28:

Código C++:

int main( );
 

La puedes eliminar con mucho cariño. Las declaraciones de las funciones únicamente son necesarias en dos supuestos:

• La función va a ser llamada desde otro fichero
• La función va a ser llamada desde el mismo fichero pero dicha llamada se encuentra ántes que la implementación de la función

Hacer llamadas al main desde tu código es, por decirlo suavemente, un tremendo error. ¿Tú diseñarías tu casa de tal forma que para ir del baño al salon tuvieses que cruzar la puerta principal? no, verdad? main es la entrada a tu programa... no lo uses para otros propósitos.

Cuidado al declarar las variables y con los rangos

Código C++:

void agregar( )
{
  // ...
  for (i=1; i<300; i++){
    empleado varempleado [i];
    varempleado[i].primer_nombre;
 

En ese código tienes tres problemas:

• En cada iteración, estás creando un array de "i" elementos... nota que "i" va creciendo en cada iteración
• con cualquier llamada a "varempleado[i]" estás accediendo a un elemento no válido
• La última línea del ejemplo anterior no hace absolutamente nada

Intentando adivinar tus intenciones, lo que tendrías que hacer es declarar varempleado fuera del for. Si declaras la variable dentro del for ésta se pierde y se vuelve a crear con cada iteración del bucle, luego los datos almacenados con i=1 ya no estarán disponibles con i=2.

Por otro lado, en C++ los rangos para un array de tamaño N van desde 0 hasta N-1. Esto tiene como implicaciones que tu for está mal declarado, tendría que ir de 0<=i<300 en vez de 0<i<300.

Dicho esto, tu código tendría que quedar más bien así:

Código C++:

void agregar()
{
  empleado varempleado[300];
  ofstream archivo;
  archivo.open("Empleados.txt",ios::app);
  for (int i=0; i<300; i++){
 

Ahora estás creando un arreglo de 300 elementos y el bucle va, en principio, a iterar sobre los 300 elementos.

Como te he comentado, la línea

Código C++:

varempleado[i].primer_nombre;
 

No hace absolutamente nada. No modificas esa variable ni tampoco usas su valor para nada. No se cual es tu intención en este punto, pero al menos así no se hace.

Cuidado con las entradas / salidas

En la función agregar():

Código C++:

cout<<"Numero de nomina:"<<endl;
    cout<<varempleado[i].numero_trabajador<<endl; // <<<---- AQUI!!!
    archivo<<"Numero de nomina: "<<;
             varempleado[i].numero_trabajador;
    archivo<<endl;
 

ese cout, no debería ser un cin? Se supone que quieres pedirle un dato al usuario, no?

Gestión de la ejecución

Código sacado de agregar():

Código C++:

cout<<"¿Desea agregar otro empreado?\n 1.- Si\n 2.- No\n";
    cin>>q;
    switch (q){
      
      case 1:
        system ("cls");
        //cout<<"Adios\n";
        //opcion();
        //agregar();
        break;
      case 2:    
        system ("cls");
        cout<<"Hasta luego\n";
        //opcion();
        main();
        break;
      default:
        system ("cls");
        cout<<"Invalido\n";
    }//switch
 

Esto está mal planteado.

Este código está dentro de un for, luego si el usuario dice que quiere seguir introduciendo datos, simplemente deja que el bucle siga funcionando. Cuando el usuario no desee introducir más datos, símplemente añade el break y deja que la ejecución abandone el bucle, después cerrará el archivo y abandonará la función retornando al main... en el main tienes ya hecho un bucle que pregunta al usuario por la nueva operación a realizar.

El único problema que esto plantea es que si pones un break dentro de un switch, este break afecta únicamente al switch... luego el for se seguirá ejecutando. Cambia el switch por una cadena de if y ya está. Si además quieres que, ante una entrada no válida, se le siga preguntando al usuario, introduce todo este código dentro de un while

Código C++:

do
{
   cout<<"¿Desea agregar otro empreado?\n 1.- Si\n 2.- No\n";
    cin>>q;
    if( q == 2 )
    {
        system ("cls");
        cout<<"Hasta luego\n";
        break;
    }
    else if( q != 1 )
    {
        system ("cls");
        cout<<"Invalido\n";
    }
} while( q != 1 && q != 2 );
 

Terminando con agregar()

Al final, en esta función estás pidiendo al usuario datos para añadir nuevos empleados a un fichero... no estás almacenando estos datos en memoria para posteriores usos y los datos los estás guardando sobre la marcha... no necesitas un array para hacer esto.

Además, el bucle no tiene por qué estar limitado a 300. Teniendo en cuenta que ya tenemos un código que nos permite salir del bucle cuando no queremos más inserciones, podemos sustituir el for por un while(true). while ejecuta el bucle mientras se cumpla la condición, como ésta será siempre cierta, la ejecución se quedará encerrada en el bucle para siempre... hasta que se encuentre con un break, que será el que hemos añadido en el punto anterior.

Código C++:

void agregar()
{
  empleado varempleado;
  ofstream archivo;
  archivo.open("Empleados.txt",ios::app);
  while( true ){
 
    //agregar datos
    cout<<"Numero de nomina:"<<endl;
    cout<<varempleado.numero_trabajador<<endl;
    archivo<<"Numero de nomina: "<<;
             varempleado.numero_trabajador;
    archivo<<endl;
    
    cout<<"Primer nombre:"<<endl;
    cin>>varempleado.primer_nombre;
    archivo<<"Primer nombre: "<<varempleado.primer_nombre;
    archivo<<endl;
 
    // ...
 
    //muestra de los datos capturados
    cout<<"_______________________________________________________"<<endl;
    cout<<"nomina: "<<varempleado.numero_trabajador<<endl;//i<<endl;
    cout<<"Primer nombre: "<<varempleado.primer_nombre<<endl;
    cout<<"Segundo nombre: "<<varempleado.segundo_nombre<<endl;
    // ...
 

Y, bueno, por no hacer este mensaje más largo, creo con esto tienes ya unas guías para corregir otros problemas de tu código... dale caña que este programa necesita un buen repaso.

Un saludo.