1

Tengo el siguiente código:

#include<string>
#include<iostream>
#include<iomanip>
#include<limits>
#include<algorithm>
#define hrmax1 40
#define hrmax2 80
using namespace std;

void check()
{
    cin.clear();
    cin.ignore(numeric_limits<streamsize>::max(), '\n');
}

int main()
{
    int limemp, noemp = 1, horas, hrextra;
    double extra, ingr, jnor, total;
    string opc = "si";
    string *name[noemp];
    string *surname[noemp];
    char ace = 130, aca = 160, aco = 162, acu = 163, acn = 164, interr = 168;
    do {
        cout << "Introduzca la cantidad de empleados a registrar (use 0 para salir): ";
        while (!(cin >> limemp) || limemp < 0)
        {
            cout << "Intente de nuevo: ";
            check();
        }
        if (limemp == 0)
        {
            return 0;
        }
        cout << "Introduzca el salario por hora: ";
        while (!(cin >> ingr) || limemp < 0)
        {
            cout << "Intente de nuevo: ";
            check();
        }
        cin.ignore(numeric_limits<streamsize>::max(), '\n');
        system("cls");
        for (noemp = 1; noemp <= limemp; noemp++)
        {
            name[noemp] = new string;
            surname[noemp] = new string;
            cout << "==DATOS DEL EMPLEADO #" << noemp << ": ENTRADA==" << endl << endl;
            cout << "Introduzca el/los nombre(s) del empleado #" << noemp << ": ";
            getline( cin, *name[noemp] );
            cout << "Introduzca los apellidos del empleado #" << noemp << ": ";
            getline( cin, *surname[noemp] );
            cout << "Cantidad de horas [a la semana] en las que labora " << *name[noemp] << " " <<*surname[noemp] << ": ";
            while (!(cin >> horas))
            {
                cout << "Intente de nuevo: ";
                check();
            }
            system("cls");
            cout << "==DATOS DEL EMPLEADO #" << noemp << ": SALIDA==" << endl << endl;
            jnor = ingr * horas;
            {
                if (noemp >= 1 || noemp <= 9)
                cout << "ID del empleado: #00" << noemp;
                else if (noemp >= 10 || noemp <= 99)
                cout << "ID del empleado: #0" << noemp;
                else if (noemp >= 100)
                cout << "ID del empleado: #" << noemp;
            }
            cout << endl << "Nombre del empleado: " << *surname[noemp] << ", " << *name[noemp] << endl;
            cout << "Horas laboradas: " << horas << endl;
            cout << "Salario por hora: $" << ingr << endl;
            cout << "Total a pagar - Jornada normal: $" << jnor << endl;
            if (horas > hrmax1 && horas < hrmax2)
            {
                hrextra = horas - hrmax1;
                extra = 2.73 * hrextra;
                total = jnor + extra;
                cout << "Horas extra: " << hrextra << endl;
                cout << "Excedente a pagar: $" << extra << endl;
                cout << "Total a pagar - J. normal ($" << jnor << ") + Horas extra ($" << extra << "): $" << total << endl;
            }
            system("pause");
            system("cls");
            horas = 0;
            cin.ignore(numeric_limits<streamsize>::max(), '\n');
        }
        cout << endl << interr << "Quieres reiniciar el programa? (Si/No) -> ";
    cond: //Etiqueta para saber si se reinicia el programa o si se cierra.
        cin >> opc; //Se lee la opción.
        transform(opc.begin(), opc.end(), opc.begin(), ::tolower); //Se convierte la entrada en minúsculas, haciendo que nuestra entrada sea insensible.
        if (opc == "si" || opc == "s") //Condiciones a evaluar. Si la entrada es "Si":
        {
            horas = 0;
            ingr = 0;
            system("cls");             //Se limpia la consola.
            continue;                  //Y se continúa el bucle.
        }
        if (opc == "no" || opc == "n") //Si la entrada es "No":
        {
            system("cls");
            cout << "Cerrando...\n";
            return 0;                  //Sólo se "finaliza la ejecución de una función y devuelve el control a la función de llamada" -Microsoft.
        }
        else                           //Si la entrada que se ha dado no coincide con las de arriba:
        {
            while (!((opc == "si" || opc == "s") || (opc == "no" || opc == "n")))   //Se hace otro bucle con el cuál se evitará el cierre o reinicio del programa.
            {
                cout << "Intenta de nuevo: ";
                cin.clear();                                                        //Se limpia la entrada errónea
                cin.ignore(numeric_limits<streamsize>::max(), '\n');                //y la descartamos.
                goto cond;                                                          //Por último hacemos referencia a nuestra etiqueta.
            }
        }
    } while (opc == "si" || opc == "s");        
    return 0;
}

El problema es el siguiente: el programa se comienza a romper si el empleado es el número 5. Esto significa que, si al inicio establecemos que se van a registrar 5 o más empleados, el programa se romperá y dejará de funcionar hasta el sexto empleado.

Es decir:

==DATOS DEL EMPLEADO #4: ENTRADA==

Introduzca el/los nombre(s) del empleado #4: Elliot
Introduzca los apellidos del empleado #4: Stiko IV
Cantidad de horas [a la semana] en las que labora Elliot Stiko IV: 45
==DATOS DEL EMPLEADO #4: SALIDA==

ID del empleado: #004
Nombre del empleado: Stiko IV, Elliot
Horas laboradas: 45
Salario por hora: $500
Total a pagar - Jornada normal: $22500
Horas extra: 5
Excedente a pagar: $13.65
Total a pagar - J. normal ($22500) + Horas extra ($13.65): $22513.7
Presione una tecla para continuar . . .
/*Aquí el programa se pausa para revisar resultados; al presionar una tecla la consola se limpia*/

Así se muestra desde el primer empleado hasta el cuarto empleado, al menos al utilizar la misma cantidad de horas laboradas (es decir, 45).

Aquí se pone feo:

==DATOS DEL EMPLEADO #5: ENTRADA==

Introduzca el/los nombre(s) del empleado #5: Elliot
Introduzca los apellidos del empleado #5: Stiko V
Cantidad de horas [a la semana] en las que labora Elliot Stiko V: 45
/*Nótese que aún estoy utilizando el mismo valor que en la entrada del empleado anterior.*/
==DATOS DEL EMPLEADO #5: SALIDA==

ID del empleado: #005
Nombre del empleado: Stiko V, Elliot
Horas laboradas: 45
Salario por hora: $5.77579e-317                /*Aquí empieza a dejar valores gigantescos, a pesar de que el salario por hora es de $500, entonces $500 * 45hrs = $22513.7*/
Total a pagar - Jornada normal: $2.5991e-315   /*El mismo comportamiento se ve aquí*/
Horas extra: 5
Excedente a pagar: $13.65                      /*Aquí también*/
Total a pagar - J. normal ($2.5991e-315) + Horas extra ($13.65): $13.65 /*Y en estas dos salidas: J. normal y hrs. extra.*/
Presione una tecla para continuar . . .
==DATOS DEL EMPLEADO #6: ENTRADA==

Introduzca el/los nombre(s) del empleado #6: Elliot
Introduzca los apellidos del empleado #6: Stiko VI
Cantidad de horas [a la semana] en las que labora Elliot Stiko VI: 45
/*Aquí también utilizo las 45hrs*/

Y el programa se rinde aquí.

==DATOS DEL EMPLEADO #6: SALIDA==

ID del empleado: #006
Nombre del empleado: Stiko VI, /*No escribió el nombre o el resto de la salida.*/
--------------------------------
Process exited after 475.8 seconds with return value 3221225477

¿Hay alguna explicación detrás de esto?

4
  • 2
    Aparte de lo que puse, tienes la variable noemp sin inicializar. Osea, son varios errores. Intenta eliminar algunos.
    – Trauma
    el 26 mar. 2020 a las 7:06
  • Lo que dejaste [en tu última edición] solucionó el problema, pero a pesar de que se podía compilar, al ejecutarlo el getline se saltaba. Esta vez utilicé el cin.ignore(numeric_limits<streamsize>::max(), '\n'); antes y después del bucle for y el programa funciona de maravilla. ¡Te lo agradezco mucho! :D
    – skidd10pp
    el 26 mar. 2020 a las 7:08
  • Por cierto, borraste tu respuesta :(
    – skidd10pp
    el 26 mar. 2020 a las 7:09
  • Acabo de utilizar vector, pero ahora devuelve otro valor de error: 3221225725. No puedo avanzar del nombre.
    – skidd10pp
    el 26 mar. 2020 a las 19:37

1 respuesta 1

2

El número 3221225477 en hexadecimal es C0000005‬, que se corresponde con una violación de acceso.


La violación de acceso suele suceder cuando accedes a memoria que no pertenece a tu proceso. Muy probablemente se de cuando accedes a la formación name en una posición inexistente.

Esto seguramente pasa porque creas memoria para name usando una variable no inicializada:

int limemp, noemp, horas = 0, hrextra = 0;
//          ^^^^^ <--- noemp NO TIENE VALOR ESTABLECIDO
double extra = 0;
string *name = new string[noemp]; // ¿De qué tamaño es name?

La variable no inicializada tendrá un valor residual que puede ser desde cero a vete-tú-a-saber-cuánto, si el valor aleatorrio de noemp es menor al valor que lees sobre limemp, entonces tu acceso fallará en el bucle:

for (noemp = 1; noemp <= limemp; noemp++)
{
    cout << "Introduzca el nombre del empleado #" << noemp << ": ";
    // Fallo si limemp es menor que el valor que tenía noemp
    //           vvvvvvvvvvv
    getline(cin, name[noemp]);
    // Fallo si limemp es menor que el valor que tenía noemp
    //                                                vvvvvvvvvvv
    cout << "Cantidad de horas en las que labora " << name[noemp] << ": ";

Eso sin tener en cuenta que las formaciones en se indexan desde cero, es decir: van de 0 a tamaño-1, por lo que tu bucle está mal. De no hacerlo así, accederás a la posición posterior a la última reservada. Posiblemente querías hacer esto:

int limemp, horas = 0, hrextra = 0;
double extra = 0;
string name[TAMAÑO]; // Usa el tamaño que creas conveniente.

// ...

for (int noemp = 0; noemp != limemp; ++noemp) // De 0 a limemp-1
{
    // ...
5
  • No veo en el código que muestras que el tamaño de name esté definido por el usuario. el 26 mar. 2020 a las 13:50
  • O sea, lo que se hace es que se crea otra variable name, según el valor de noemp en el bucle for.
    – skidd10pp
    el 26 mar. 2020 a las 18:17
  • Ya vi mi error. A lo que me refería era que noemp depende un poco de limemp, que es definido por el usuario. En el bucle for, una vez que noemp sea menor o igual a limemp el bucle termina y se pregunta si se quiere repetir el programa.
    – skidd10pp
    el 26 mar. 2020 a las 18:20
  • Igualmente actualicé la publicación con el código hasta donde lo dejé.
    – skidd10pp
    el 26 mar. 2020 a las 18:23
  • Has modificado la publicación y sigues teniendo exáctamente el mismo error que antes (noemp puede ser menor que el valor que el usuario defina) pero además has añadido un error adicional: el uso de Formaciones de Tamaño Variable lee estos hilos para saber más del tema. el 28 mar. 2020 a las 17:51

Tu Respuesta

Al pulsar en “Publica tu respuesta”, muestras tu consentimiento a nuestros términos de servicio, política de privacidad y política de cookies

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