A la hora de ejecutar el programa no hace nada y se termina.
Esta es una descripción muy vaga del problema, que además no se ajusta a la realidad. He probado tu código y me ha mostrado datos por pantalla:
Real nc1: 0.000000 Imaginario: 0.000000
Real nc1: 0.000000 Imaginario: 0.000000
Real nc1: 0.000000 Imaginario: 0.000000
Real nc1: 0.000000 Imaginario: 0.000000
Real nc1: 0.000000 Imaginario: 0.000000
Procura ser más claro y conciso en tu pregunta.
A base de analizar tu código, deduzco que esperabas esta salida:
Real nc1: 0.000000 Imaginario: 0.000000
Real nc1: 1.000000 Imaginario: 1.000000
Real nc1: 2.000000 Imaginario: 2.000000
Real nc1: 3.000000 Imaginario: 3.000000
Real nc1: 4.000000 Imaginario: 4.000000
Pero no se puede dar, porque en el bucle de rellenado de datos:
for(int i=0; i<n; i++){
nc1[i].i=i;
nc1[i].r=i;
}
Recorres la formación1 hasta la posición n
, cuyo valor se establece a 1
al principio de main
y no vuelve a cambiar en todo el programa. Por otro lado, reservas memoria sobre el puntero nc1
pero después realojas memoria sobre r1
, el puntero pasado a masMemoria
no cambia porque lo pasas por copia (el cambio de valor del puntero no sale de la función).
"Supongo" que en realidad querías hacer esto:
int main(){
int n = 1;
Complejo *nc1; // <-- Solo un puntero
nc1 = (Complejo *)malloc(sizeof(Complejo) * n);
for (int i = 0; i < n; ++i) {
nc1[i].i = i;
nc1[i].r = i;
}
n = 5; // <-- Nuevo valor de n.
nc1 = masMemoria(nc1, n); // Usamos n.
for (int i = 0; i < n; ++i) { // Ahora el bucle rellena toda la formación
nc1[i].i = i;
nc1[i].r = i;
}
for (int i = 0; i < n; ++i) {
printf("Real nc1: %f Imaginario: %f\n", nc1[i].r, nc1[i].i);
}
return 0;
}
Otras cosas a tener en cuenta.
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.
También debes tener en cuenta que estás pidiendo memoria dinámica pero no la estás liberando, no olvides la llamada a free
al finalizar el programa.
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(Complejo *original, int cantidad) {
Complejo *aux;
if ((aux = (Complejo *)realloc(original, sizeof(Complejo) * cantidad))) {
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(datos, 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);
}
free(datos);
return 0;
}
- También conocida como arreglo o en inglés array.
mem=masMemoria(mem, 5);
mem=masMemoria(mem, 5)
¿?