Error al dimensionar vectores en C++

Question

Saludos.

Es mi primera vez en foros, estoy aprendiendo C++, y se me ha presentado un problema; tengo que hacer un programa que en primer momento me permita agregar datos de alumnos a una arreglo.

El problema es que que no importa la cantidad de alumnos que le indique al programa que quiero ingresar; porque me permite ingresar solamente dos alumnos y luego aparece una ventana que dice que el programa dejó de funcionar.

He intentando hacerlo de dos maneras, con switch, y con switch y con funciones, y en ambos caso me sucede lo mismo; aquí les copio el código del intento de switch y funciones, desde ya gracias a quien se tome el tiempo de ayudarme.

#include<iostream>
#include<stdlib.h>

using namespace std;    
void agregar();

int main(){

    //muestra el menu, y permite ejecutar la opcion 8.(salir del programa)

    int opcion;

    cout << "Que desea hacer?" << endl;
    cout << endl;
    cout << "1.Aregar alumno" << endl;
    cout << "2.Agregar notas de alumno" << endl;
    cout << "3.modificar notas de alumno" << endl;
    cout << "4.Eliminar alumno" << endl;
    cout << "5.Mostrar lista de aprobados" << endl;
    cout << "6.Mostrar list de reprobados" << endl;
    cout << "7.Mostrar toda la lista de alumnos" << endl;
    cout << "8.Salir del programa" << endl;
    cout << endl;
    cin >> opcion;

    switch(opcion){
        case 1:{
            agregar();
            break;
        }
    }

    system ("pause");
    return 0;
}

void agregar(){

    // variables utilizadas en void agregar()
    int m = 0, n = 0, i, r = 0;
    string codigo[m], nombre[m], grado[m];

    cout << "Cuantos alumnos desea agregar? ";
    cin >> m;
    n = n + m;

    cout << "ingrese los datos: " << endl;
    for(i = 0; i < m; i++){
        cout << "nombre del alumno " << i + 1 << ": ";
        cin >> nombre[i];

    }

    cout << "Desea ingresar mas alumnos? ";
    cout << "1.SI";
    cout << "2.No";
    cin >> r;

    if(r = 1){
        m = 0;
        void agregar();
    }else{
        m = 0;
        int main();
    }
    system ("pause");
}

Answer 1

Arreglos de tamaño variable (VLA en Inglés).

Tal y como ha mencionado gbianchi tu problema reside en la declaración de los arreglos, por lo tanto, cambiar el orden en que se pide el tamaño de los arreglos y su definición puede resolver el problema, siempre y cuando tu compilador soporte Arreglos de tamaño Variable (AtV).

Los AtV no pertenecen al C++ estándar si no que son una extensión del compilador (Lee sobre la extensión en el compilador GCC). Cada compilador tiene sus propias extensiones de manera que si el código te compila es que tu compilador dispone de esta extensión, pero en otros compiladores podría estar esta extensión desactivada o no existir y te mostraría un error. Así pues: tu código no sería portable.

Propuesta.

Usa std::vector, esta plantilla de la librería estándar de C++ puede ser usada como un arreglo y se le puede modificar el tamaño en tiempo de ejecución:

using vector_string = std::vector<string>;

// variables utilizadas en void agregar()
int m = 0, n = 0, i, r = 0;
vector_string codigo, nombre, grado;

std::cout << "Cuantos alumnos desea agregar? ";
std::cin >> m;
n = n + m;

codigo.resize(n);
nombre.resize(n);
grado.resize(n);

El método std::vector::resize cambia el tamaño del vector para que sea capaz de almacenar la cantidad de elementos indicada. Después, la inserción de datos queda sin cambios:

std::cout << "ingrese los datos: " << std::endl;
for(i = 0; i < m; ++i){
    std::cout << "nombre del alumno " << i + 1 << ": ";
    std::cin >> nombre[i];
}

Otras cosas a tener en cuenta.

• No se aconseja usar using namespace std, lee esta pregunta para saber por qué.
• Tampoco se aconseja usar std::endl ya que hace flush del stream, pudiendo causar problemas de rendimiento, lee esta pregunta para tener más detalles.
• Usa el pre-incremento en lugar del post-incremento, este artículo te explica el por qué.

• Esta instrucción:

  if(r = 1){
  

  Siempre será verdadera, porque estás asignando el valor 1 a la variable r. Tras asignar, se evalúa r como dato booleano. En C++ cualquier valor diferente a cero (0) se considera verdadero (true). Si quieres hacer una comparación usa el doble igual (==):

  if(r == 1){
  

Answer 2

El error esta en la inicializacion de los vectores:

// variables utilizadas en void agregar()
    int m = 0, n = 0, i, r = 0;
    string codigo[m], nombre[m], grado[m];

    cout << "Cuantos alumnos desea agregar? ";
    cin >> m;
    n = n + m;

    cout << "ingrese los datos: " << endl;
    for(i = 0; i < m; i++){
        cout << "nombre del alumno " << i + 1 << ": ";
        cin >> nombre[i];

    }

el vector nombre donde tratas de guardar los nombres esta inicializado en 0 (con 0 elementos), por lo tanto, cuando preguntas cuantos alumnos quiere ingresar, eso no modifica el tamaño del vector porque el mismo fue inicializado antes.

Deberias iniacializar el vector cuando sepas realmente cuantos alumnos se desean ingresar.. basta con modificar el orden de tu codigo.

// variables utilizadas en void agregar() int m = 0, n = 0, i, r = 0;

    cout << "Cuantos alumnos desea agregar? ";
    cin >> m;
    n = n + m;
    string codigo[m], nombre[m], grado[m];
    cout << "ingrese los datos: " << endl;
    for(i = 0; i < m; i++){
        cout << "nombre del alumno " << i + 1 << ": ";
        cin >> nombre[i];

    }

En ningun caso, igual, estoy mirando el resto del codigo, donde hay varios errores mas.

fijate por ejemplo aqui:

if(r = 1){
        m = 0;
        void agregar();
    }else{
        m = 0;
        int main();
    }

Estas llamando recursivamente a agregar, pero ya estas dentro de esa funcion.. todas estas llamadas deberian estar estructuradas de otra forma. por ejemplo dentro de un while que sepa cuando terminar la ejecucion de todos los puntos. tambien estas haciendo m = 0 sin ningun sentido aparente, porque al volver a llamar a la funcion, la m es otra, porque esta en otro scope.

Mi consejo es que leas un poco mas de como funciona el lenguaje.