El programa debe pedir una cantidad indefinida de números
Si necesitas almacenar una cantidad indefinida de números, tienes dos opciones:
- Crear una formación con espacio suficiente como para que quepa una cantidad de números "indefinida".
- Crear una formación con espacio para una cantidad de números arbitraria y redimensionar dicha cantidad cuando sea necesario.
Voy a centrarme en la segunda.
Puedes crear una formación solicitando memoria con malloc
(Memory, ALLOCation, alojamiento de memoria):
int cantidad = 100;
int *numeros = (int*)malloc(sizeof(int) * cantidad);
Si en algún momento la cantidad de elementos de la formación se te queda corta, puedes realojar la memoria con realloc
(REALLOCation, realojar):
numeros = realloc(numeros, cantidad *= 2);
Así pues, tu código podría quedar así:
int cantidad = 10;
int indice = 0;
int *numeros = (int*)malloc(cantidad);
for (int numero = 1; numero; ++indice)
{
if (indice == cantidad)
{
numeros = (int*)realloc(numeros, cantidad *= 2);
}
printf("Dame un numero: ");
scanf("%d", &numero);
numeros[indice] = numero;
}
Quisiera resaltar los siguientes cambios que he hecho a tu código:
- Nombres de variables autoexplicativos: la variables cuyo nombre es una sola letra pueden ser adecuadas en ámbitos pequeños y localizados, más aún si dicha letra guarda una relación con la misión de la variable (
x
para coordenadas horizontales, i
para índices...), pero variables como f
para cantidad de elementos, r
para el resultado de restar dos números, b
para sumatorio de números positivos, c
para sumatorio de números negativos y e
para comprobar si un número es par o impar es una elección de nombres nefasta y contraintuitiva.
- No declares variables que no necesitas: declaras
a
y d
para no usarlas, además declaras variables que sólo usas para almacenar el resultado de una expresión cuyo resultado después usas para tomar una decisión; podrías usar la expresión directamente sin crear la variable.
- Reduce el ámbito de las variables: en versiones antiguas de c, era necesario declarar las variables al inicio de los ámbitos. Hoy día no es necesario y se aconseja que la declaración de variables esté esté lo más cerca posible de su lugar de uso, lo cuál facilita entender su cometido y entender el código en su conjunto.
- Reduce la cantidad de llamadas a
printf
: Cada llamada a función tiene un coste adicional a la llamada e si, debe cambiar el ámbito la pila local. Si puedes hacer más cosas con menos llamadas, el código funcionará de manera más eficiente.
- He usado un
for
, no un do
-while
: el bucle for
combina con la mínima cantidad de código escrito una variable de control de ámbito acotado, un incremento y una condición de salida.
Con los consejos anteriores, el código en total podría parecerse a:
#include <stdio.h>
#include <stdlib.h>
int main()
{
int cantidad = 10;
int indice = 0;
int *numeros = (int*)malloc(cantidad);
for (int numero = 1; numero; ++indice)
{
if (indice == cantidad)
{
numeros = (int*)realloc(numeros, cantidad *= 2);
}
printf("Dame un numero: ");
scanf("%d", &numero);
numeros[indice] = numero;
}
int positivos = 0, negativos = 0;
for(int i = 0; i != indice; ++i)
{
printf("%d %s\n", numeros[i], (numeros[i] % 2) ? "impar" : "par");
if (numeros[i] < 0)
negativos += numeros[i];
else
positivos += numeros[i];
}
printf("Suma de positivos: %d\n"
"Suma de negativos: %d\n"
"Diferencia %d", positivos, negativos, positivos - negativos);
return 0;
}