Foro de elhacker.net

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>

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:

*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í:

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í:

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í:

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().

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:

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:

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:

#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