Tu código es c no c++: 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.
<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.La cabecera
<conio.h>
no es estándar, en consecuencia no ha sido adaptada a C++. Lee este hilo para saber más detalles.Estas leyendo cadenas de caracteres sobre una formación1 de números,
int nombres [11][50];
debería serchar nombres [11][50];
.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;
.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 cadafor
, deberías declararla en cada forfor
.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
(c
onsoleout
) y para leer de la consola se usastd::cin
(c
onsolein
), todos esosprintf
yscanf
son funciones de C.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]); }
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.
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++; }
Los valores se devuelven con
return
, la macroEXIT_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;
Tu código es c no c++: 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.
<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.La cabecera
<conio.h>
no es estándar, en consecuencia no ha sido adaptada a C++. Lee este hilo para saber más detalles.Estas leyendo cadenas de caracteres sobre una formación1 de números,
int nombres [11][50];
debería serchar nombres [11][50];
.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;
.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 cadafor
, deberías declararla en cada for.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
(c
onsoleout
) y para leer de la consola se usastd::cin
(c
onsolein
), todos esosprintf
yscanf
son funciones de C.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]); }
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.
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++; }
Los valores se devuelven con
return
, la macroEXIT_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;
Tu código es c no c++: 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.
<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.La cabecera
<conio.h>
no es estándar, en consecuencia no ha sido adaptada a C++. Lee este hilo para saber más detalles.Estas leyendo cadenas de caracteres sobre una formación1 de números,
int nombres [11][50];
debería serchar nombres [11][50];
.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;
.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 cadafor
, deberías declararla en cadafor
.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
(c
onsoleout
) y para leer de la consola se usastd::cin
(c
onsolein
), todos esosprintf
yscanf
son funciones de C.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]); }
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.
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++; }
Los valores se devuelven con
return
, la macroEXIT_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;
¿Que hay de malo con mi programa en C++?
Muchas cosas.
Tu código es c no c++: 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.
<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.La cabecera
<conio.h>
no es estándar, en consecuencia no ha sido adaptada a C++. Lee este hilo para saber más detalles.Estas leyendo cadenas de caracteres sobre una formación1 de números,
int nombres [11][50];
debería serchar nombres [11][50];
.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;
.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 cadafor
, deberías declararla en cada for.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
(c
onsoleout
) y para leer de la consola se usastd::cin
(c
onsolein
), todos esosprintf
yscanf
son funciones de C.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]); }
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.
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++; }
Los valores se devuelven con
return
, la macroEXIT_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.
- También conocida como arreglo o en inglés array.