subprograma que calcula la media

[minette1988]

Hola, estoy haciendo un subprograma que me calcula la media de 5 números, pero me da fallos de compilación: subprograma_media.c:18: error: tipos en conflicto para ‘media’
subprograma_media.c:10: error: la declaración previa de ‘media’ estaba aquí

Código C:

#include <stdio.h>
#include <stdlib.h>
 
 
 
int main(){
          int i;
          int num;
          
          void media(int);
 
          for(i=0; i<=5; i++){
             printf("Numero: ");
             scanf("%d\n", &num);
          }
          media(num);
}
void media(int num, int t){
          int suma;
          double m;
          int i;
 
          for(i=0; i<t; i++){
              suma = suma + num;
          }
          m = suma/t;
 
          printf("La media es %lf\n", m);
}
 

¿Podéis decirme si está bien estructurado el programa?. Gracias.

[AlanChavez]

La linea 10, debe ir afuera de main.
Se llama prototipo de la función, no lo he compilado pero eso deberia solucionarlo.

[margarito04]

declarar una funcion dentro de otra no es muy común, aun q valido. Lo que si es q

void media(int);//linea 10

difiere de

void media(int num, int t)//linea 18

y los dos deben tener el mismo numero de parametros

[minette1988]

He modificado el programa, pero cuando lo compilo el resultado es incorrecto. Me sale como resultado la media es -405297833.000000

Código C:

#include <stdio.h>
#include <stdlib.h>
       
int main(){
         int i;
         int n=0;
         int num;
         void media( int, int );
      
         printf("Cuántos números quieres introducir?");
         scanf("%d", &n);
 
         for(i=1; i<=n; i++){
            printf("Numero: ");
            scanf("%d", &num);
         }
         media(n,num);
}
void media(int n_datos, int dato){
         int suma;
         double m;
         int i;
 
         for(i=1; i<=n_datos; i++){
           suma = suma + dato;
         }
         m = suma/n_datos;
       
         printf("La media es %lf\n", m);
}
 

[minette1988]

Ayudadme con este programa, por favor. Gracias.

[gonzo13]

Hola. El contenido de la línea 8 tiene que estar antes de la linea:

Código C:

int main()
 

Y por supuesto, borrar la línea 8. Saludos,
Gonzo

[minette1988]

Es así cómo dices ¿no?, lo he compilado , pero el resultado sigue siendo incorrecto.

Código C:

#include <stdio.h>
#include <stdlib.h>
 
void media( int, int );
       
int main(){
         int i;
         int n=0;
         int num;
         
      
         printf("Cuántos números quieres introducir?");
         scanf("%d", &n);
 
         for(i=1; i<=n; i++){
            printf("Numero: ");
            scanf("%d", &num);
         }
         media(n,num);
}
void media(int n_datos, int dato){
         int suma;
         double m;
         int i;
 
         for(i=1; i<=n_datos; i++){
           suma = suma + dato;
         }
         m = suma/n_datos;
       
         printf("La media es %lf\n", m);
}
 

[minette1988]

¿Qué es lo que tengo mal?

[CalgaryCorpus]

Realizar varias veces scanf, sin hacer nada mas, en un ciclo (lineas 15 a 18) no tiene sentido.
Es lo mismo que asignar un valor a una variable varias veces:

a = 1; // se va a perder este valor
a = 9; // se va a perder este valor
a = 4; // se va a perder este valor, si se continua asignando la misma variable
...

Por otro lado, y como consecuencia de esto, la funcion media() suma varias veces el mismo valor.

Recomiendo sumar en el ciclo en que lees, o bien leer hacia un arreglo y no a una variable int.

[minette1988]

¿Podrías explicarte mejor?. Gracias.

[minette1988]

He intetado hacerlo con vectores, pero el resultado me sale mal La media es -405607764.000000

Código C:

#include <stdio.h>
#include <stdlib.h>
 
      
int main(){
         int v[3];
         int i;
         
         void media( int[], int );
      
         
 
         for(i=0; i<3; i++){
            printf("Numero: ");
            scanf("%d", &v[i]);
         }
         media(v,3);
}
void media(int v[], int t){
         int suma;
         double m;
         int i;
 
         for(i=0; i<t; i++){
           suma = suma + v[i];
         }
         m = suma/t;
       
         printf("La media es %lf\n", m);
}
 

[CalgaryCorpus]

2 problemas:
- La linea 25 tiene sentido solo si la variable suma tiene un valor antes.

- En la linea 27, hay una division entre 2 enteros. Eso siempre da entero.

por ejemplo: 5 / 2 resulta 2, no resulta 2.5
y m = 5 / 2 demandara una conversion del 2 a un double y no de 2.5 a un double.

Hay varias posibilidades de solución a esto último:
- suma deberia ser float o double, no int. Asi, "suma / t " no sera una division de enteros
- Hacer un cast en la division: (float) suma / t, suma / (float) t, (float) suma / (float) t.

[minette1988]

Gracias por tu ayuda, se me había olvidado hacer el cast.