Skip to main content
se añadieron 39 caracteres en el cuerpo
Origen Enlace
Eequiis
  • 1.8k
  • 7
  • 21

Estás recorriendo desde el primer elemento (0) al último (i - 1), pero en todas las iteraciones compruebas si el siguiente elemento i (despuésestá después del último) tiene un sueldo inferior a 1000. 

Esto va a tener un comportamiento indefinido, ya que las variables no se inicializan por defecto, y pueden contener cualquier valor, positivo, negativo o cero. Por lo que algunas veces te mostrará el empleado k y otras no. 

En cualquier caso, va a ser incorrecto porque el hecho de mostrar el empleado k depende del empleado i, que todavía no se ha metido en la colección de empleados. 

Es más, si llegas a llenar el vector de empleados, se intentará acceder al empleado 2001 y podrá saltar un fallo de segmento por intentar acceder a una zona que no se ha reservado.

#define MAXIMO_NUMERO_EMPLEADOS 2000
#define SUELDO_MIL 1000

struct empleado
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero[15];
};

typedef struct empleado Empleado;

struct coleccionEmpleados
{
    Empleado empleados[MAXIMO_NUMERO_EMPLEADOS];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
}; 

typedef struct coleccionEmpleados ColeccionEmpleadosColeccionEmpleados;

ColeccionEmpleados coleccionEmpleados;    // coleccionEmpleados != struct coleccionEmpleados

...

void mostrarDatosMil()
{
    for (int empleado = 0; empleado < coleccionEmpleados.numeroEmpleados; empleado++)
    {
        if (coleccionEmpleados.empleados[empleado].sueldo > SUELDO_MIL) printf(...);
    }
}

Estás recorriendo desde el primer elemento al último (i - 1), pero compruebas si el siguiente elemento (después del último) tiene un sueldo inferior a 1000. Esto va a tener un comportamiento indefinido, ya que las variables no se inicializan por defecto, y pueden contener cualquier valor, positivo, negativo o cero. Por lo que algunas veces te mostrará el empleado k y otras no. En cualquier caso, va a ser incorrecto porque el hecho de mostrar el empleado k depende del empleado i, que todavía no se ha metido en la colección de empleados. Es más, si llegas a llenar el vector de empleados, se intentará acceder al empleado 2001 y podrá saltar un fallo de segmento por intentar acceder a una zona que no se ha reservado.

#define MAXIMO_NUMERO_EMPLEADOS 2000
#define SUELDO_MIL 1000

struct empleado
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero[15];
};

typedef struct empleado Empleado;

struct coleccionEmpleados
{
    Empleado empleados[MAXIMO_NUMERO_EMPLEADOS];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
};
typedef struct coleccionEmpleados ColeccionEmpleados

ColeccionEmpleados coleccionEmpleados;    // coleccionEmpleados != struct coleccionEmpleados

...

void mostrarDatosMil()
{
    for (int empleado = 0; empleado < coleccionEmpleados.numeroEmpleados; empleado++)
    {
        if (coleccionEmpleados.empleados[empleado].sueldo > SUELDO_MIL) printf(...);
    }
}

Estás recorriendo desde el primer elemento (0) al último (i - 1), pero en todas las iteraciones compruebas si el elemento i (está después del último) tiene un sueldo inferior a 1000. 

Esto va a tener un comportamiento indefinido, ya que las variables no se inicializan por defecto, y pueden contener cualquier valor, positivo, negativo o cero. Por lo que algunas veces te mostrará el empleado k y otras no. 

En cualquier caso, va a ser incorrecto porque el hecho de mostrar el empleado k depende del empleado i, que todavía no se ha metido en la colección de empleados. 

Es más, si llegas a llenar el vector de empleados, se intentará acceder al empleado 2001 y podrá saltar un fallo de segmento por intentar acceder a una zona que no se ha reservado.

#define MAXIMO_NUMERO_EMPLEADOS 2000
#define SUELDO_MIL 1000

struct empleado
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero[15];
};

typedef struct empleado Empleado;

struct coleccionEmpleados
{
    Empleado empleados[MAXIMO_NUMERO_EMPLEADOS];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
}; 

typedef struct coleccionEmpleados ColeccionEmpleados;

ColeccionEmpleados coleccionEmpleados;    // coleccionEmpleados != struct coleccionEmpleados

...

void mostrarDatosMil()
{
    for (int empleado = 0; empleado < coleccionEmpleados.numeroEmpleados; empleado++)
    {
        if (coleccionEmpleados.empleados[empleado].sueldo > SUELDO_MIL) printf(...);
    }
}
Recomendaciones con structs 2
Origen Enlace
Eequiis
  • 1.8k
  • 7
  • 21
#define SUELDOMAXIMO_NUMERO_EMPLEADOS 2000
#define SUELDO_MIL 1000

struct empleado
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero[15];
};

typedef struct empleado Empleado;

struct coleccionEmpleados
{
    Empleado empleados[2000];empleados[MAXIMO_NUMERO_EMPLEADOS];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
};
typedef struct coleccionEmpleados ColeccionEmpleados

ColeccionEmpleados coleccionEmpleados;    // coleccionEmpleados != struct coleccionEmpleados

...

void mostrarDatosMil()
{
    for (int empleado = 0; empleado < coleccionEmpleados.numeroEmpleados; empleado++)
    {
        if (coleccionEmpleados.empleados[empleado].sueldo > SUELDOSUELDO_MIL) printf(...);
    }
}

Incluso puedes definir métodos dentro de los structs para trabajar con ellos de forma más cómoda y lógica. Por ejemplo:

#define OPCION_MOSTRAR_MIL 3

struct coleccionEmpleados
{
    Empleado empleados[MAXIMO_NUMERO_EMPLEADOS];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
    void mostrarEmpleadosMil()
    {
        for (unsigned empleado = 0; empleado < numeroEmpleados; empleado++) printf(...);
    }
};

typedef struct coleccionEmpleados ColeccionEmpleados;

ColeccionEmpleados coleccionEmpleados;    // Llamada implícita al constructor sin parámetros

int main(int argc, char * argv[])
{
    ...
    switch (opcion)
    {
        ...
        case OPCION_MOSTRAR_MIL:
            coleccionEmpleados.mostrarEmpleadosMil();
            break;
        ...
    }
}

Así no tendrías métodos libres que manejen estructuras, sino que las estructuras proveen métodos para manejarlas adecuadamente.

#define SUELDO 1000

struct empleado
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero[15];
};

typedef struct empleado Empleado;

struct coleccionEmpleados
{
    Empleado empleados[2000];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
};
typedef struct coleccionEmpleados ColeccionEmpleados

ColeccionEmpleados coleccionEmpleados;    // coleccionEmpleados != struct coleccionEmpleados

...

void mostrarDatosMil()
{
    for (int empleado = 0; empleado < coleccionEmpleados.numeroEmpleados; empleado++)
    {
        if (coleccionEmpleados.empleados[empleado].sueldo > SUELDO) printf(...);
    }
}
#define MAXIMO_NUMERO_EMPLEADOS 2000
#define SUELDO_MIL 1000

struct empleado
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero[15];
};

typedef struct empleado Empleado;

struct coleccionEmpleados
{
    Empleado empleados[MAXIMO_NUMERO_EMPLEADOS];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
};
typedef struct coleccionEmpleados ColeccionEmpleados

ColeccionEmpleados coleccionEmpleados;    // coleccionEmpleados != struct coleccionEmpleados

...

void mostrarDatosMil()
{
    for (int empleado = 0; empleado < coleccionEmpleados.numeroEmpleados; empleado++)
    {
        if (coleccionEmpleados.empleados[empleado].sueldo > SUELDO_MIL) printf(...);
    }
}

Incluso puedes definir métodos dentro de los structs para trabajar con ellos de forma más cómoda y lógica. Por ejemplo:

#define OPCION_MOSTRAR_MIL 3

struct coleccionEmpleados
{
    Empleado empleados[MAXIMO_NUMERO_EMPLEADOS];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
    void mostrarEmpleadosMil()
    {
        for (unsigned empleado = 0; empleado < numeroEmpleados; empleado++) printf(...);
    }
};

typedef struct coleccionEmpleados ColeccionEmpleados;

ColeccionEmpleados coleccionEmpleados;    // Llamada implícita al constructor sin parámetros

int main(int argc, char * argv[])
{
    ...
    switch (opcion)
    {
        ...
        case OPCION_MOSTRAR_MIL:
            coleccionEmpleados.mostrarEmpleadosMil();
            break;
        ...
    }
}

Así no tendrías métodos libres que manejen estructuras, sino que las estructuras proveen métodos para manejarlas adecuadamente.

Recomendaciones con structs
Origen Enlace
Eequiis
  • 1.8k
  • 7
  • 21

En cualquier caso, considera seguir las convenciones de C/C++ (dices queetiquetas tu código espregunta como C++ pero utilizas bibliotecas de C, y no usas ninguna característica de C++), comosalvo el new empleados[2000]): escribir el nombre de funciones y variables empezando con minúscula (lower camel case por ejemplo), y los #define ALGO en mayúscula (sin empezar por guiones, ya que estos tienen un uso reservado al compilador).

#define SUELDO 1000

typedef struct empleado
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero [15];genero[15];
} Empleado;;

typedef struct empleado Empleado;

struct coleccionEmpleados
{
    Empleado[]Empleado empleados;empleados[2000];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
};
typedef ColeccionEmpleados;struct coleccionEmpleados ColeccionEmpleados

ColeccionEmpleados coleccionEmpleados;    // coleccionEmpleados != struct coleccionEmpleados

...

void mostrarDatosMil()
{
    for (int empleado = 0; empleado < coleccionEmpleados.numeroEmpleados; empleado++)
    {
        if (coleccionEmpleados.empleados[empleado].sueldo > SUELDO) printf(...);
    }
}

En cualquier caso, considera seguir las convenciones de C/C++ (dices que tu código es C++ pero utilizas bibliotecas de C, y no usas ninguna característica de C++), como escribir el nombre de funciones y variables empezando con minúscula (lower camel case por ejemplo), y los #define ALGO en mayúscula (sin empezar por guiones, ya que estos tienen un uso reservado al compilador).

#define SUELDO 1000

typedef struct
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero [15];
} Empleado;

typedef struct
{
    Empleado[] empleados;
    unsigned numeroEmpleados;
} ColeccionEmpleados;

En cualquier caso, considera seguir las convenciones de C/C++ (etiquetas tu pregunta como C++ pero utilizas bibliotecas de C, y no usas ninguna característica de C++, salvo el new empleados[2000]): escribir el nombre de funciones y variables empezando con minúscula (lower camel case por ejemplo), y los #define ALGO en mayúscula (sin empezar por guiones, ya que estos tienen un uso reservado al compilador).

#define SUELDO 1000

struct empleado
{
    char nombre[30];
    char apellido[30];
    int edad;
    float sueldo;
    char genero[15];
};

typedef struct empleado Empleado;

struct coleccionEmpleados
{
    Empleado empleados[2000];
    unsigned numeroEmpleados;
    coleccionEmpleados() : numeroEmpleados(0) {}    // Constructor C++
};
typedef struct coleccionEmpleados ColeccionEmpleados

ColeccionEmpleados coleccionEmpleados;    // coleccionEmpleados != struct coleccionEmpleados

...

void mostrarDatosMil()
{
    for (int empleado = 0; empleado < coleccionEmpleados.numeroEmpleados; empleado++)
    {
        if (coleccionEmpleados.empleados[empleado].sueldo > SUELDO) printf(...);
    }
}
se añadieron 1228 caracteres en el cuerpo
Origen Enlace
Eequiis
  • 1.8k
  • 7
  • 21
Loading
Origen Enlace
Eequiis
  • 1.8k
  • 7
  • 21
Loading