Duda con problema en C

[JuanLVBR]

Iniciado por haidion

media tiene que ser del tipo float, millas y litros son int entonces va %d en lugar de %f. Y en media = pusiste dos iguales y eso es una operacion logica.

Código c:

main()
{
    int millas = 0;
    int litros = 0;
    int repeticiones = 0;
    float media;
   
 
 
    while(repeticiones <=3){
    printf("introduzca millas recorridas):\n");
    scanf("%d", &millas);
    printf("introduzca litros usados:\n");
    scanf("%d", &litros);
 
    media = float(millas)/float(litros);
    printf("la media es %f\n", media);
 
    repeticiones += 1;
 
    }
 
 
    return 0;
}
 

En la línea 16 de ese código lo que se está haciendo es decirle al compilador que quieres declarar una variable float millas y una float litros y lógicamente generará un error en compilación, si lo que se intentaba era un cast lo correcto sería cambiar los paréntesis y ponerlos en el tipo, no en el nombre:

Código:

    media = (float)millas/(float)litros;

Con un único cast es suficiente:

Código:

    media = (float)millas/litros;

En el resto del código lo único que está mal es como bien dicen la condición del while o inicializar la variable repetición con otro valor para que se hagan 3 repeticiones y no 4 y los tipos de datos (float, int, ya sabes).

Y para finalizar, para calcular la media de las tres medias, podrías declarar un vector como te han dicho y luego recorrerlo para hacer la media, pero... Si es estático ¿de qué tamaño? ¿Si le da por calcular 4 medias otro día? Además estamos perdiendo memoria a lo tonto, es mejor tener un float que X floats (dependiendo del número de medias que quiera, en este caso 3), yo lo haría así (te lo añado a tu código para que te sea más fácil verlo, sólo es la declaración de un nuevo float y dos instrucciones nuevas además de las correcciones que te he escrito):

Código:

#include<stdio.h>

int main()
{
    int millas = 0;
    int litros = 0;
    int repeticiones = 0;
    float media;
    float sumaMedias=0;
   


    while(repeticiones<3){
        printf("introduzca millas recorridas:\n");
        scanf("%d", &millas);
        printf("introduzca litros usados:\n");
        scanf("%d", &litros);

        media = (float)millas/litros;
        printf("la media es %f\n", media);
        sumaMedias += media;

        repeticiones += 1;

    }
    
    printf("\n\nLa media total es %f.\n", sumaMedias/repeticiones);

    return 0;
}

Ojo, lo que has pedido es la media de las medias, no la media del total, si quieres la media del total se haría de otro modo, deberías sumar todos los litros acumulándolos y todas las millas para mostrar en el printf la división entre el total de millas y el de litros.