Aprendiendo el uso basico de apuntadores ! no me da el resultado correcto

[harofenix]

Que tal, apenas estoy aprendiendo el uso basico de apuntadores, hice este programita para calcular el promedio, sin embergo, me da mal el promedio, la pregunta es: que estoy haciendo mal ?

#include <stdio.h>

Código [Seleccionar] Expandir


void SolicitudEingresoDeValores(float valor1, float valor2, float valor3, float valor4, float valor5);
float Promedio(float *val1, float *val2, float *val3, float *val4, float *val5);
/*declaracion de apuntadores*/
float *val1, *val2, *val3, *val4, *val5;

int main(int argc, char **argv)
{
float average;
float valor1 = 0, valor2 = 0, valor3 = 0, valor4 = 0, valor5 = 0;
SolicitudEingresoDeValores(valor1, valor2, valor3, valor4, valor5);
average = Promedio(val1, val2, val3, val4, val5);
printf("%10.2f \n", average);
return 0;
}

void SolicitudEingresoDeValores(float valor1, float valor2, float valor3, float valor4, float valor5)
{

printf("ingrese el primer valor: \n");
scanf("%f", &valor1); /*inicializo puntero*/
val1 = &valor1;
printf("ingrese el primer valor: \n");
scanf("%f", &valor2);/*inicializo puntero*/
val2 = &valor2;
printf("ingrese el primer valor: \n");
scanf("%f", &valor3);/*inicializo puntero*/
val3 = &valor3;
printf("ingrese el primer valor: \n");
scanf("%f", &valor4);/*inicializo puntero*/
val4 = &valor4;
printf("ingrese el primer valor: \n");
scanf("%f", &valor5);/*inicializo puntero*/
val5 = &valor5;

}
float Promedio(float *val1, float *val2, float *val3, float *val4, float *val5)
{

float promedio;
promedio = ( *val1 + *val2 + *val3 + *val4 + *val5 ) / 5;
return promedio;
}

[SrCooper]

No soy ningún experto, pero probablemente esto se deba a que en la funcion de ingreso estas dando a los punteros una dirección de memoria local dentro de la función y cuando se sale de esta función la dirección se vuelve indeterminada (creo).

Para dar valor a los punteros deberías hacer lo siguiente:

Código (c) [Seleccionar] Expandir

*val1 = valor;
Con esto le estás indicando al programa que asigne el valor al puntero, no su dirección de memoria. Sin embargo para hacer esto antes tienes que haber reservado memoria para el puntero con la función malloc() y después tienes que liberarla con la función free().

El programa quedaría entonces así:

Código (c) [Seleccionar] Expandir

void SolicitudEingresoDeValores();
float Promedio();

/* Declaracion de los punteros */
float *val1, *val2, *val3, *val4, *val5;

int main(int argc, char **argv)
{
float average;

/* Reservamos espacio en la memoria para poder asignar valores a los punteros */
val1 = malloc(sizeof(float));
val2 = malloc(sizeof(float));
val3 = malloc(sizeof(float));
val4 = malloc(sizeof(float));
val5 = malloc(sizeof(float));

SolicitudEingresoDeValores();

average = Promedio();
printf("%f \n", average);

/* Una vez que lo hemos usado, liberamos este espacio */
free(val1);
free(val2);
free(val3);
free(val4);
free(val5);

return 0;
}

void SolicitudEingresoDeValores()
{
 /* Como cambia de valor, con una variable sera suficiente */
 float valor;

 printf("ingrese el primer valor: \n");
 scanf("%f", &valor);
 *val1 = valor; // Asignamos el valor, no la direccion de memoria (pues es una variable local)
 
 printf("ingrese el segundo valor: \n");
 scanf("%f", &valor);
 *val2 = valor;
 
 printf("ingrese el tercer valor: \n");
 scanf("%f", &valor);
 *val3 = valor;
 
 printf("ingrese el cuarto valor: \n");
 scanf("%f", &valor);
 *val4 = valor;
 
 printf("ingrese el quinto valor: \n");
 scanf("%f", &valor);
 *val5 = valor;
}

float Promedio()
{
 return ( *val1 + *val2 + *val3 + *val4 + *val5 ) / 5;
}

Como ya has reservado espacio para los punteros, la función scanf() se podría usar simplemente así:

Código (c) [Seleccionar] Expandir

scanf("%f", val1); sin necesidad de un variable local, poniendo como parámetros directamente los punteros. Por lo tanto la función quedaría así:

Código (c) [Seleccionar] Expandir

void SolicitudEingresoDeValores()
{
  printf("ingrese el primer valor: \n");
  scanf("%f", val1);
 
  printf("ingrese el segundo valor: \n");
  scanf("%f", val2);
 
  printf("ingrese el tercer valor: \n");
  scanf("%f", val3);
 
  printf("ingrese el cuarto valor: \n");
  scanf("%f", val4);
 
  printf("ingrese el quinto valor: \n");
  scanf("%f", val5);
}

Por supuesto aún se podrían mejorar muchas cosas, como utilizar arrays en lugar de punteros separados para poder realizar medias de un número variable de valores en cada programa. Sin embargo esto cumple con lo que querías en el programa original utilizando punteros.

Un saludo

PD: En la función malloc() se tiene que indicar el tamaño de la variable (en bytes). Para ello se utiliza la función sizeof().

[rir3760]

Que tal, apenas estoy aprendiendo el uso basico de apuntadores, hice este programita para calcular el promedio, sin embergo, me da mal el promedio, la pregunta es: que estoy haciendo mal ?

El problema principal en el programa es de diseño.

1) No necesitas de las variables "globales", ademas hay que evitar su uso siempre que sea posible.

2) La función "SolicitudEingresoDeValores" se debe modificar pasando las direcciones de la cinco variables con la intención de modificarlas dentro de la función:

Código (c) [Seleccionar] Expandir

void SolicitudEingresoDeValores(float *a, float *b, float *c, float *d, float *e)
{
printf("ingrese el primer valor: \n");
scanf("%f", a);

printf("ingrese el primer valor: \n");
scanf("%f", b);

printf("ingrese el primer valor: \n");
scanf("%f", c);

printf("ingrese el primer valor: \n");
scanf("%f", d);

printf("ingrese el primer valor: \n");
scanf("%f", e);
}

3) La función "Promedio" no requiere de punteros ya que no altera las variables, solo necesita sus valores para calcular el promedio:

Código (c) [Seleccionar] Expandir

float Promedio(float a, float b, float c, float d, float e)
{
return (a + b + c + d + e) / 5;
}

4) Ambas funciones se utilizan en main de esa forma:

Código (c) [Seleccionar] Expandir

#include <stdio.h>

void SolicitudEingresoDeValores(float *a, float *b, float *c, float *d, float *e);
float Promedio(float a, float b, float c, float d, float e);

int main(void)
{
float a, b, c, d, e;
   
SolicitudEingresoDeValores(&a, &b, &c, &d, &e);
printf("%10.2f \n", Promedio(a, b, c, d, e));
   
return 0;
}

/* ... */

Un saludo