###Otras cosas a tener en cuenta.
Tu código padece de sobreingeniería, tanto el redactado del código como su flujo es difícil de seguir y propenso a errores, un ejemplo claro de estos problemas es la función masMemoria
, la cuál no sigue el principio de la mínima sorpresa:
¿Por qué recibe un puntero por parámetro y a la vez devuelve un puntero? Lo esperable de una función que pide memoria es que le digas cuánta memoria quieres y que la función te devuelva esta memoria, el parámetro adicional es desconcertante. Además, el parámetro adicional es inútil ya que los cambios sobre el mismo no saldrán de la función.
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.
El apelotonamiento también contribuye a que tu código sea difícil de seguir y comprender; en los años 90 teníamos pantallas de 80 caracteres de ancho y 25 líneas de alto, eso hacía necesario estrujar el código para que cupiera en pantalla la mayor cantidad posible; por suerte han pasado casi 30 años y ya no tenemos esa limitación.
Para acabar, la elección de nombres de tus variables empeora más la situación, los nombres que son acrónimos o una sola letra aportan muy poca información sobre el dato manejado y para colmo hace más probable que dos variables diferentes tengan nombres parecidos y provoque confusión (como en tu bucle de pintado, que imprimes nc1
en el texto pero usas el valor de r1
).
Resolviendo los problemas mencionados, tu código podría parecerse a:
typedef struct{
float imaginario, real;
} Complejo;
Complejo *masMemoria(int elementos) {
if (Complejo *aux = (Complejo *)malloc(sizeof(Complejo) * elementos)) {
return aux;
}
printf("Problemas al reservar memoria dinámica\n");
return NULL;
}
int main() {
int cantidad = 1;
Complejo *datos = (Complejo *)malloc(sizeof(Complejo) * cantidad);
for (int indice = 0; indice < cantidad; ++indice) {
datos[indice].imaginario = indice;
datos[indice].real = indice;
}
cantidad = 5;
datos = masMemoria(cantidad);
for (int indice = 0; indice < cantidad; ++indice) {
datos[indice].imaginario = indice;
datos[indice].real = indice;
}
for (int indice = 0; indice < cantidad; ++indice) {
printf("Real: %f Imaginario: %f\n", datos[indice].real, datos[indice].imaginario);
}
return 0;
}