0

Ejecuto el programa, cuando entra en el primer for este no avanza hacia buscar el numero mayor, solo se queda estancado en salario: y procede a cerrarse. Cual es el error que se presenta? No lo logro encontrar. No sé si se encuentra en las condicionales o tiene que ver con algo del buffer. Cuando ingreso salario lo hago como, ejemplo: 3000000, supongo que eso no tiene que afectar, sigue siendo 'float'. Necesito hacer espacio para que me deje enviar la pregunta, lo siento si tienes que leer esto.

#include <iostream>
#include <conio.h>
#include <string.h>

using namespace std;

int main (){
    int cant_emp;
    char crg[] = {}; 
    
    cout << "Cantidad de empleados: "; cin >> cant_emp; 
    
    struct empleados{
        char nombre[20];
        int edad;
        float salario;
    }emp[cant_emp];
    
    for (int i = 0; i < cant_emp; i++){ //Ingreso de datos
        fflush(stdin);
        cout << i+1 << endl << "Nombre: "; cin.getline(emp[i].nombre, 20, '\n');
        
        do{ //Edad
            cout << "Edad: "; cin >> emp[i].edad;
            if (emp[i].edad<=17 && emp[i].edad > 0){
                cout << "Debes ser mayor de edad: "; cin >> emp[i].edad;
            }
            if (emp[i].edad<=0){
                cout <<"Tu edad debe ser un numero positivo distinto de 0: "; cin >> emp[i].edad;
            }
        }while(emp[i].edad<18);

        cin.ignore();
        do{ //Salario
            cout << "Salario: "; cin >> emp[i].salario; //Aqui el programa se detiene.
        }while(emp[i].salario <= 0);

        if (emp[i].salario > 3000000){
            strcat(crg, "Directivo");
        }
        else{
            if (emp[i].salario <= 3000000 && emp[i].salario > 1000000){
                strcat(crg, "Senior");
            }
            else{
                if (emp[i].salario <= 1000000 && emp[i].salario >= 400000){
                    strcat(crg, "Junior");
                }               
                else{
                    if (emp[i].salario > 80000 && emp[i].salario < 4000000){
                        strcat(crg, "Practicante"); 
                    }
                    else{
                        strcat(crg, "En serio, estas en esta empresa por tan poco dinero?");
                        }
                    }
                }
            }
        cout << endl;
    }
    cout << endl;
    
    for (int j = 0;  j < cant_emp; j++){
        fflush(stdin);
        cout << j+1 << endl;
        cout << "Nombre: " << emp[j].nombre << endl;
        cout << "Edad:" << emp[j].edad << endl;
        cout << "Salario :" << emp[j].salario << endl << endl;
        cout << "Cargo: " << crg;
        }

    getch();
    return 0;
}
2
  • 1
    Al contrario, no veo que se cierre nada sino que se ejecuta infinitamente. Claramente es porque queda un salto de línea luego de leer el número de empleados. Danos la entrada con la que estás probando a ver si obtenemos el mismo resultado.
    – Mateo
    Commented el 3 mar. 2023 a las 3:46
  • Por cierto, no deberías usar fflush(stdin). Más información.
    – Mateo
    Commented el 3 mar. 2023 a las 4:05

1 respuesta 1

1

El problema está en las líneas como esta:

strcat(crg, "Directivo");

En un compilador como gcc recibes Segmentation Fault.

crg es un arreglo vacío. Los arreglos son de tamaño fijo. No se pueden modificar.

Llegado a cierto punto, strcat lee una dirección de memoria que no debería y obtienes ese error, una excepción o simplemente el programa crashea.


Solución:

  • Dale al arreglo crg un tamaño más grande, lo suficiente como para no tener problemas.

  • Estás usando C++ en el año 2023. Considera usar la librería de cadenas en lugar de usar las viejas funciones de C.

    Con esta opción sugiero que reemplaces los arreglos de caracteres por std::string, strcat por el operador += o append. También tiene su propia alternativa a cin.getline.

2
  • La alternativa a cin.getline puede ser getline()? o cin.get? Commented el 4 mar. 2023 a las 18:22
  • @MatíasBilz la alternativa a cin.getline es la función getline a secas. Se usa así: getline(cin, string). cin.get no consume el delimitador.
    – Mateo
    Commented el 4 mar. 2023 a las 20:00

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.