[Estructuras] ¿Por qué me muestra basura?

Iniciado por Zodiak98, 22 Mayo 2016, 22:57 PM

0 Miembros y 1 Visitante están viendo este tema.

Zodiak98

Verán, el programa que tengo que hacer es el siguiente.
Leer N cantidad de estudiantes, y dentro de otra estructura sus notas (parte teórica y parte práctica), como un máximo de 5 calificaciones para cada alumno. Es decir, si digo que son 3 notas prácticas serán 3 notas prácticas para todos los alumnos, lo mismo para la parte teórica.

En fin, todo chévere, pero cuando intento mostrar las calificaciones que ingresé previamente, me muestra valores basura.

¿Alguien podría echarme una mano? El programa lo estoy haciendo en Code::Blocks.
Acá les dejo el código fuente que llevo hasta ahora:

Código (cpp) [Seleccionar]

#include <iostream>
#include <cstdlib>

using namespace std;

//Variables globales para la cantidad de prácticos y parciales.
int nPracticos, nParciales;

struct Notas {
    int practicos[];
    int parciales[];
    float promedio;
};
typedef struct Notas Notas;

struct Estudiante {
    char nombre[25];
    char apellido[35];
    char cedula[10];
    char sexo;
    int edad;
    Notas notas;
};
typedef struct Estudiante Estudiante;

int leerEstudiantes();
int leerEvaluaciones(char tipo[]); //Para leer la cantidad de practicos y parciales.

void llenarEstudiante(Estudiante& estudiante, int id);
void mostrarEstudiante(const Estudiante& estudiante, int id);

int main() {
    int nEstudiantes;
    nEstudiantes = leerEstudiantes();
    nPracticos   = leerEvaluaciones((char*)"Practicos");
    nParciales   = leerEvaluaciones((char*)"Parciales");

    system("cls");

    Estudiante estudiante[nEstudiantes];

    //Llenando cada uno de los estudiantes.
    for(int est = 0; est < nEstudiantes; est++) {
        llenarEstudiante(estudiante[est], est);
    }

    system("cls");

    //Mostrar datos de los estudiantes.
    for(int est = 0; est < nEstudiantes; est++) {
        mostrarEstudiante(estudiante[est], est);
    }

    return 0;
}

int leerEstudiantes() {
    int nEstudiantes;

    do {
        cout << "Introduzca la cantidad de estudiantes: ";
        cin >> nEstudiantes;

        if(nEstudiantes <= 0) {
            cerr << "Cantidad de estudiantes invalida." << endl;
        }
    }while(nEstudiantes <= 0);

    return nEstudiantes;
}

int leerEvaluaciones(char tipo[]) {
    int cantidad;

    do{
        cout << "Ingrese la cantidad de " << tipo << ": ";
        cin >> cantidad;

        if(cantidad <= 0 || cantidad > 5) {
            cerr << "Cantidad incorrecta." << endl;
        }
    }while(cantidad <= 0 || cantidad > 5);

    return cantidad;
}

void llenarEstudiante(Estudiante& estudiante, int id) {
    //Variables para calcular el promedio.
    int pPracticos = 0, pParciales = 0;

    //Lectura de datos generales.
    cout << "Ingrese nombre de estudiante [" << (id + 1) << "]: ";
    cin >> estudiante.nombre;
    cout << "Ingrese apellido de estudiante [" << (id + 1) << "]: ";
    cin >> estudiante.apellido;
    cout << "Ingrese cedula de estudiane [" << (id + 1) << "]: ";
    cin >> estudiante.cedula;
    cout << "Ingrese edad de estudiante [" << (id + 1) << "]: ";
    cin >> estudiante.edad;
    cout << "Ingrese sexo de estudiante [" << (id + 1) << "] (M/F): ";
    cin >> estudiante.sexo;

    //Lectura de notas prácticas.
    for(int pr = 0; pr < nPracticos; pr++) {
        do {
            cout << "Ingrese nota del practico [" << (pr + 1) << "]: ";
            cin >> estudiante.notas.practicos[pr];

            if(estudiante.notas.practicos[pr] < 0 || estudiante.notas.practicos[pr] > 10) {
                cerr << "Nota incorrecta." << endl;
            }
        }while(estudiante.notas.practicos[pr] < 0 || estudiante.notas.practicos[pr] > 10);
    }

    //Lectura de notas parciales.
    for(int pa = 0; pa < nParciales; pa++) {
        do {
            cout << "Ingrese nota del parcial [" << (pa + 1) << "]: ";
            cin >> estudiante.notas.parciales[pa];
            if(estudiante.notas.parciales[pa] < 0 || estudiante.notas.parciales[pa] > 10) {
                cerr << "Nota incorrecta." << endl;
            }
        }while(estudiante.notas.parciales[pa] < 0 || estudiante.notas.parciales[pa] > 10);
    }

    //Calculando el promedio del estudiante:
    //Sumatoria de Prácticos
    for(int pr = 0; pr < nPracticos; pr++) {
        pPracticos += estudiante.notas.practicos[pr];
    }
    //Sumatoria de Parciales
    for(int pa = 0; pa < nParciales; pa++) {
        pParciales += estudiante.notas.parciales[pa];
    }
    //Cálculo del promedio.
    estudiante.notas.promedio = pPracticos/nPracticos*0.20 + pParciales/nParciales*0.50;
}

void mostrarEstudiante(const Estudiante& estudiante, int id) {
    cout << "Nombre de estudiante [" << (id + 1) << "]: " << estudiante.nombre << endl;
    cout << "Apellido de estudiante [" << (id + 1) << "]: " << estudiante.apellido << endl;
    cout << "Cedula de estudiante [" << (id + 1) << "]: " << estudiante.cedula << endl;
    cout << "Sexo de estudiante [" << (id + 1) << "]: " << estudiante.sexo << endl;
    cout << "Edad de estudiante [" << (id + 1) << "]: " << estudiante.edad << endl;

    for(int j = 0; j < nPracticos; j++) {
        cout << "Nota del practico [" << (j + 1) << "] del estudiante [" << (id + 1) << "]: " << estudiante.notas.practicos[j] << endl;
    }

    for(int j = 0; j < nParciales; j++) {
        cout << "Nota del parcial [" << (j + 1) << "] del estudiante [" << (id + 1) << "]: " << estudiante.notas.parciales[j] << endl;
    }

    cout << "Promedio del estudiante [" << (id + 1) << "]: " << estudiante.notas.promedio << endl;
    cout << "-------------------" << endl;
}

engel lex

estás accediendo mal a la estructura, el array es la estructura... por lo menos

cout << "Ingrese nombre de estudiante [" << (id + 1) << "]: ";
    cin >> estudiante.nombre;


pero a cual estudiante te ferienes? al primero? al último?

tienes que decirle
cout << "Ingrese nombre de estudiante [" << (id + 1) << "]: ";
    cin >> estudiante[id].nombre;


por lo menos en
cin >> estudiante.notas.practicos[pr];

estás tratando de acceder a la posición pr de la variable prácticos de el struct notas de un estudiante indefinido
El problema con la sociedad actualmente radica en que todos creen que tienen el derecho de tener una opinión, y que esa opinión sea validada por todos, cuando lo correcto es que todos tengan derecho a una opinión, siempre y cuando esa opinión pueda ser ignorada, cuestionada, e incluso ser sujeta a burla, particularmente cuando no tiene sentido alguno.

Zodiak98

Eso lo indico en el main, es decir, la función leerEstudiante recibe por referencia a sólo un único estudiante.
Y en la función main le indicó cuál de ellos dentro del arreglo de Estudiantes.
Código (cpp) [Seleccionar]

//Llenando cada uno de los estudiantes.
    for(int est = 0; est < nEstudiantes; est++) {
        llenarEstudiante(estudiante[est], est);
    }


Ahí le estoy pasando cada uno de los estudiantes a la función llenarEstudiante. Y como es por referencia, estoy modificando el elemento original.

Lo mismo con la función mostrarEstudiante, no entiendo por qué me muestra esa basura. :/

engel lex

dime la linea donde te muestra basura mejor -.-
El problema con la sociedad actualmente radica en que todos creen que tienen el derecho de tener una opinión, y que esa opinión sea validada por todos, cuando lo correcto es que todos tengan derecho a una opinión, siempre y cuando esa opinión pueda ser ignorada, cuestionada, e incluso ser sujeta a burla, particularmente cuando no tiene sentido alguno.

Zodiak98

Acá:
Código (cpp) [Seleccionar]

for(int j = 0; j < nPracticos; j++) {
        cout << "Nota del practico [" << (j + 1) << "] del estudiante [" << (id + 1) << "]: " << estudiante.notas.practicos[j] << endl;
    }

    for(int j = 0; j < nParciales; j++) {
        cout << "Nota del parcial [" << (j + 1) << "] del estudiante [" << (id + 1) << "]: " << estudiante.notas.parciales[j] << endl;
    }


Eso se encuentra dentro de la función mostrarEstudiante, y dicha función recibe a sólo un único elemento, y luego en el main llamo a esa función enviándole cada elemento del arreglo Estudiante.

Código (cpp) [Seleccionar]

for(int est = 0; est < nEstudiantes; est++) {
        mostrarEstudiante(estudiante[est], est);
    }

engel lex

no veo la memoria reservada en los array dentro de notas, lo probable es que no sepa donde guardar los datos (es raro que no te haya dado error fuera de indice)

en llenar tienes que inicializar esos int[]
estudiante.notas.practicos = new int[nPracticos];

creo que allí está el error, ya que en c++ lo array no son dinamicos, tienes que reasignar su tamaño, tu simplemente dejaste dicho que eran punteros y al escribir simplemente hacen un overflow
El problema con la sociedad actualmente radica en que todos creen que tienen el derecho de tener una opinión, y que esa opinión sea validada por todos, cuando lo correcto es que todos tengan derecho a una opinión, siempre y cuando esa opinión pueda ser ignorada, cuestionada, e incluso ser sujeta a burla, particularmente cuando no tiene sentido alguno.

Zodiak98

Damn, no sabía eso. Ya se solucionó el problema, ¡gracias! :)