Pregunta sobre mi diseño para mi tarea en C++

Question

Primero, esto es para mi clase CS296, así que todo tiene que estar en la función main() y no puedo usar nada más que bucles, cambios y declaraciones if.

De todos modos, el programa le pide al usuario que ingrese el número de estudiantes en una clase, seguido de sus nombres. A continuación, muestra qué estudiante debe estar al frente de la línea y qué estudiante debe estar en la parte posterior de la línea en función de los estudiantes que se alinearon alfabéticamente de A-Z.

El programa se ejecuta, pero creo que el diseño es pobre. Específicamente con cómo estoy usando la variable de conteo. Esperaba tal vez obtener algunos comentarios al respecto. ¡Gracias!

int main(void) 
{ 
    string front, back, student; 
    unsigned short count = 1, students; 

    cout << "Enter the number of students in the class. Valid range is 1 - 25." << endl 
     << endl 
     << "Students: "; 
    cin >> students; 

    while (students < 1 || students > 25) 
    { 
     cout << endl 
      << "ERROR: Valid range is 1 - 25" << endl 
      << endl 
      << "Students: "; 
     cin >> students; 
    } 

    cout << endl 
     << "Enter the name of student " << count << ": "; 
    cin.ignore(); 
    getline(cin, student); 

    front = back = student; 

    for (count = 2; count <= students; count++) 
    { 
     cout << "Enter the name of student " << count << ": "; 
     getline(cin, student); 

     if (student < front) 
      front = student; 
     else if (student > back) 
      back = student; 
    } 

    cout << endl 
     << front << " should be at the head of the line." << endl 
     << back << " should be at the end of the line." << endl; 

    return 0; 
} 

Answer 1

No, parece correcto. Las especificaciones requieren ingresar un recuento y luego ingresar a muchos estudiantes, que es exactamente lo que has hecho.

No me malinterpreten, el programa es malo, pero eso se debe a especificaciones ridículas como "todo tiene que estar en la función main()", no tiene nada que ver con su implementación.

Para este nivel de tarea, me gustaría obtener algo así de mis alumnos. Sería un cambio agradable del código que ni siquiera compila, nombres de variables de un solo carácter en todas partes, o un marasmo sin formatear de indentación incorrecta y pestañas/espacios mixtos :-)

Answer 2

Parece un diseño perfectamente razonable para mí desde una perspectiva algorítmica. Solo está encontrando el mínimo y el máximo de una lista mientras lee la lista, y acelera la parte difícil (inicializando primero y último en el primer elemento). Se puede usar un do-while para que sea un poco más agradable, así:

do {// leído en, comparar al máximo, mínimo } tiempo (contar < 25)

Answer 3

Teniendo en cuenta que sólo se almacene dos estudiantes, no hay ninguna necesidad particular de limitar el número de entradas a solo 25. Si el usuario (usted) desea agregar más de 25 nombres, no hay cambios necesarios excepto en la validación del número. De hecho, no hay una razón obvia para que el usuario cuente la cantidad de estudiantes; el código podría seguir leyendo a los estudiantes hasta llegar a EOF en cin. Sin embargo, si la especificación dice "debes ingresar el número de nombres", entonces tu código tiene que hacer eso, por tonto que sea. Pero debe permitir EOF temprano y terminar el ciclo con elegancia si el usuario solo ingresa 3 nombres en lugar de los 20 prometidos.

Es lamentable que no pueda usar funciones propias; puede usar uno para solicitar y leer el siguiente nombre, reduciendo la repetición en su código.

Teniendo en cuenta estas limitaciones, su código es razonable. Podría evitar el "nombre de lectura fuera del ciclo" a costa de verificaciones más complejas dentro del ciclo. La compensación es probablemente a favor de lo que has escrito.