Skip to main content
se añadieron 3 caracteres en el cuerpo
Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86
  1. Tu código es no : Como programa en C++ tu código deja mucho que desear, no usas absolutamente nada del lenguaje en el que dices que tu código está redactado.

  2. <stdio.h> y <stdlib.h> son cabeceras de C, si realmente las necesitas en un programa (presuntamente) C++ deberás usar las versiones adaptadas a dicho lenguaje: <cstdio>, y <cstdlib>. Lee este hilo para saber más detalles.

  3. La cabecera <conio.h> no es estándar, en consecuencia no ha sido adaptada a C++. Lee este hilo para saber más detalles.

  4. Estas leyendo cadenas de caracteres sobre una formación1 de números, int nombres [11][50]; debería ser char nombres [11][50];.

  5. De hecho, si lo que pretendes es programar en C++, en lugar de una formación bidimensional de caracteres debería ser un contenedor de cadenas, por ejemplo: std::list<std::string> nombres;.

  6. En C++ se aconseja que el ámbito de las variables sea lo más pequeño posible; esto significa que en lugar de declarar la variable de indizado de los for al principio del programa y reusarla (posiblemente de manera erronea) en cada for, deberías declararla en cada forfor.

  7. En C++ la comunicación por consola con el usuario se hace con flujos (stream) de datos, para escribir en consola se usa std::cout (console out) y para leer de la consola se usa std::cin (console in), todos esos printf y scanf son funciones de C.

  8. En C++ (y también en C) las formaciones1 se indexan desde 0, lo que quiere decir que en este bucle no lees el primer elemento (además del problema mencionado en el punto 4):

     for(i=1;i<11;i++){
         printf("Capture el nombre del alumno %d: ", i);
         scanf("%s",&nombres[i]);
     }
    
  9. En los bucles, deberías favorecer el preincremento frente al postincremento, salvo que tengas un motivo para no hacerlo (que rara vez será así). Lee este artículo para más detalles.

  10. Para copiar (en tu caso, sobrescribir) datos de una posición de una formación1 bidimensional a otra posición de la misma, no hay que igualar; de hecho, al hacerlo lo más posible se que el programa falle en tiempo de ejecución, la manera correcta (en C++) es usar std::copy:

     while(i < n - 1) {
         std::copy(std::begin(nombres[i + 1]), std::end(nombres[i + 1]), std::begin(nombres[i]));
         i++;
     }
    
  11. Los valores se devuelven con return, la macro EXIT_SUCCESS no fuerza la salida del programa, es simplemente el valor que deberá ser devuelto por el programa en caso de finalizar de forma correcta. Así que deberías escribir:

     return EXIT_SUCCESS;
    
  1. Tu código es no : Como programa en C++ tu código deja mucho que desear, no usas absolutamente nada del lenguaje en el que dices que tu código está redactado.

  2. <stdio.h> y <stdlib.h> son cabeceras de C, si realmente las necesitas en un programa (presuntamente) C++ deberás usar las versiones adaptadas a dicho lenguaje: <cstdio>, <cstdlib>. Lee este hilo para saber más detalles.

  3. La cabecera <conio.h> no es estándar, en consecuencia no ha sido adaptada a C++. Lee este hilo para saber más detalles.

  4. Estas leyendo cadenas de caracteres sobre una formación1 de números, int nombres [11][50]; debería ser char nombres [11][50];.

  5. De hecho, si lo que pretendes es programar en C++, en lugar de una formación bidimensional de caracteres debería ser un contenedor de cadenas, por ejemplo: std::list<std::string> nombres;.

  6. En C++ se aconseja que el ámbito de las variables sea lo más pequeño posible; esto significa que en lugar de declarar la variable de indizado de los for al principio del programa y reusarla (posiblemente de manera erronea) en cada for, deberías declararla en cada for.

  7. En C++ la comunicación por consola con el usuario se hace con flujos (stream) de datos, para escribir en consola se usa std::cout (console out) y para leer de la consola se usa std::cin (console in), todos esos printf y scanf son funciones de C.

  8. En C++ (y también en C) las formaciones1 se indexan desde 0, lo que quiere decir que en este bucle no lees el primer elemento (además del problema mencionado en el punto 4):

     for(i=1;i<11;i++){
         printf("Capture el nombre del alumno %d: ", i);
         scanf("%s",&nombres[i]);
     }
    
  9. En los bucles, deberías favorecer el preincremento frente al postincremento, salvo que tengas un motivo para no hacerlo (que rara vez será así). Lee este artículo para más detalles.

  10. Para copiar (en tu caso, sobrescribir) datos de una posición de una formación1 bidimensional a otra posición de la misma, no hay que igualar; de hecho, al hacerlo lo más posible se que el programa falle en tiempo de ejecución, la manera correcta (en C++) es usar std::copy:

     while(i < n - 1) {
         std::copy(std::begin(nombres[i + 1]), std::end(nombres[i + 1]), std::begin(nombres[i]));
         i++;
     }
    
  11. Los valores se devuelven con return, la macro EXIT_SUCCESS no fuerza la salida del programa, es simplemente el valor que deberá ser devuelto por el programa en caso de finalizar de forma correcta. Así que deberías escribir:

     return EXIT_SUCCESS;
    
  1. Tu código es no : Como programa en C++ tu código deja mucho que desear, no usas absolutamente nada del lenguaje en el que dices que tu código está redactado.

  2. <stdio.h> y <stdlib.h> son cabeceras de C, si realmente las necesitas en un programa (presuntamente) C++ deberás usar las versiones adaptadas a dicho lenguaje: <cstdio> y <cstdlib>. Lee este hilo para saber más detalles.

  3. La cabecera <conio.h> no es estándar, en consecuencia no ha sido adaptada a C++. Lee este hilo para saber más detalles.

  4. Estas leyendo cadenas de caracteres sobre una formación1 de números, int nombres [11][50]; debería ser char nombres [11][50];.

  5. De hecho, si lo que pretendes es programar en C++, en lugar de una formación bidimensional de caracteres debería ser un contenedor de cadenas, por ejemplo: std::list<std::string> nombres;.

  6. En C++ se aconseja que el ámbito de las variables sea lo más pequeño posible; esto significa que en lugar de declarar la variable de indizado de los for al principio del programa y reusarla (posiblemente de manera erronea) en cada for, deberías declararla en cada for.

  7. En C++ la comunicación por consola con el usuario se hace con flujos (stream) de datos, para escribir en consola se usa std::cout (console out) y para leer de la consola se usa std::cin (console in), todos esos printf y scanf son funciones de C.

  8. En C++ (y también en C) las formaciones1 se indexan desde 0, lo que quiere decir que en este bucle no lees el primer elemento (además del problema mencionado en el punto 4):

     for(i=1;i<11;i++){
         printf("Capture el nombre del alumno %d: ", i);
         scanf("%s",&nombres[i]);
     }
    
  9. En los bucles, deberías favorecer el preincremento frente al postincremento, salvo que tengas un motivo para no hacerlo (que rara vez será así). Lee este artículo para más detalles.

  10. Para copiar (en tu caso, sobrescribir) datos de una posición de una formación1 bidimensional a otra posición de la misma, no hay que igualar; de hecho, al hacerlo lo más posible se que el programa falle en tiempo de ejecución, la manera correcta (en C++) es usar std::copy:

     while(i < n - 1) {
         std::copy(std::begin(nombres[i + 1]), std::end(nombres[i + 1]), std::begin(nombres[i]));
         i++;
     }
    
  11. Los valores se devuelven con return, la macro EXIT_SUCCESS no fuerza la salida del programa, es simplemente el valor que deberá ser devuelto por el programa en caso de finalizar de forma correcta. Así que deberías escribir:

     return EXIT_SUCCESS;
    
Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86

¿Que hay de malo con mi programa en C++?

Muchas cosas.


  1. Tu código es no : Como programa en C++ tu código deja mucho que desear, no usas absolutamente nada del lenguaje en el que dices que tu código está redactado.

  2. <stdio.h> y <stdlib.h> son cabeceras de C, si realmente las necesitas en un programa (presuntamente) C++ deberás usar las versiones adaptadas a dicho lenguaje: <cstdio>, <cstdlib>. Lee este hilo para saber más detalles.

  3. La cabecera <conio.h> no es estándar, en consecuencia no ha sido adaptada a C++. Lee este hilo para saber más detalles.

  4. Estas leyendo cadenas de caracteres sobre una formación1 de números, int nombres [11][50]; debería ser char nombres [11][50];.

  5. De hecho, si lo que pretendes es programar en C++, en lugar de una formación bidimensional de caracteres debería ser un contenedor de cadenas, por ejemplo: std::list<std::string> nombres;.

  6. En C++ se aconseja que el ámbito de las variables sea lo más pequeño posible; esto significa que en lugar de declarar la variable de indizado de los for al principio del programa y reusarla (posiblemente de manera erronea) en cada for, deberías declararla en cada for.

  7. En C++ la comunicación por consola con el usuario se hace con flujos (stream) de datos, para escribir en consola se usa std::cout (console out) y para leer de la consola se usa std::cin (console in), todos esos printf y scanf son funciones de C.

  8. En C++ (y también en C) las formaciones1 se indexan desde 0, lo que quiere decir que en este bucle no lees el primer elemento (además del problema mencionado en el punto 4):

     for(i=1;i<11;i++){
         printf("Capture el nombre del alumno %d: ", i);
         scanf("%s",&nombres[i]);
     }
    
  9. En los bucles, deberías favorecer el preincremento frente al postincremento, salvo que tengas un motivo para no hacerlo (que rara vez será así). Lee este artículo para más detalles.

  10. Para copiar (en tu caso, sobrescribir) datos de una posición de una formación1 bidimensional a otra posición de la misma, no hay que igualar; de hecho, al hacerlo lo más posible se que el programa falle en tiempo de ejecución, la manera correcta (en C++) es usar std::copy:

     while(i < n - 1) {
         std::copy(std::begin(nombres[i + 1]), std::end(nombres[i + 1]), std::begin(nombres[i]));
         i++;
     }
    
  11. Los valores se devuelven con return, la macro EXIT_SUCCESS no fuerza la salida del programa, es simplemente el valor que deberá ser devuelto por el programa en caso de finalizar de forma correcta. Así que deberías escribir:

     return EXIT_SUCCESS;
    

Me falla en la parte de case 3

Tu error (mencionado en el punto 9) es que estás igualando posiciones de una formación1 con la idea de sobrescribir los datos:

while(i<n-1){
nombres[i]=nombres[i+1]; // <---- MAL!!!
i++;
}

nombres es una formación1 de tamaño estático; cada uno de sus elementos tiene una posición fijada en memoria por el compilador, con tu instrucción de igualación estás pretendiendo cambiar esa posición de memoria lo cuál es muy probable que haga fallar el programa en tiempo de ejecución.


  1. También conocida como arreglo o en inglés array.