średnia ocen - dlaczego zawsze wyświetla 0?

Question

#include <stdio.h>

int main (void)
{
    int ile_liczb, index;
    float suma = 0;
    float srednia = suma/ile_liczb;

    puts("Podaj ile liczb");
    scanf("%d", &ile_liczb);

    for (index = 0; index < ile_liczb; ++index)
    {
        float ocena;
        puts("Wprowadz ocene:");
        scanf("%f", &ocena);

        suma = suma + ocena;
    }

    printf("Srednia to: %f", srednia);
}

dlaczego za każdym razem wyświetlana średnia = 0.00?

Comments

Prócz tego na co zwrócili uwagę Ci koledzy, zwróć jeszcze z main() 0 (zero). A jeszcze lepiej EXIT_SUCCESS z <stdlib.h>

Answer 1

Najpierw obliczasz średnią a dopiero potem wczytujesz ilość liczb oraz liczysz sumę musisz zmienić kolejność.

Answer 2

float srednia = suma/ile_liczb;

Kompilator wykonuje instrukcje od góry do dołu, czyli u Ciebie wylicza średnią kiedy suma zawsze będzie równa 0, a 0 przez cokolwiek (oprócz 0) daje 0. Daj ta instrukcje po tym jak policzysz sumę i wszystko będzie ok.

Answer 3

Tak jeszcze w ramach poprawienia, to przerzuć tworzenie zmiennej "ocena" przed pętle. Wewnątrz pętli nie powinno się tworzyć zmiennych, przynajmniej nie znam takiego przypadku.

Comments

A ja znam wiele. Zasięg zmiennej powinien być jak najmniejszy. Od tego jest kompilator by go nie wyręczać pseudo-optymalizacjami. Tu użycie zmiennej ocena w ciele pętli jest jak najbardziej ok. Taka zmienna poza pętlą jest przecież zbędna.

---

O widzisz, dobrze wiedzieć. Nie zdawałem sobie sprawy z tego, że kompilator zajmuje się takimi rzeczami. Chciałem pomóc, a wyszło na odwrót :D

Answer 4

Gdy umieszczam po pętli:

float srednia = suma / ile_liczb;

kompilator wypluwa błąd: ISO 90 forbis mixed declarations and code [-Wpedantic]

#include <stdio.h>

int main (void)
{
    float ocena, suma = 0;
    int ile_liczb, index;

    puts("Podaj ile liczb");
    scanf("%d", &ile_liczb);

    for (index = 0; index < ile_liczb; ++index)
    {
        puts("Wprowadz ocene:");
        scanf("%f", &ocena);
        suma = suma + ocena;
    }

    float srednia = suma / ile_liczb;
    printf("Srednia to: %f\n", srednia);

    system("pause");
    return 0;
}