obliczanie sumy i średniej w tablicy

Question

#include <stdio.h>
#include <stdlib.h>

int read_vector_float(float *vec, int size, float stop_value)
{
    int zmienna = 0;
    float n = 0;
    if(size <= 0)
    {
        return -1;
    }
        do
        {
            if(scanf("%f", &n) != 1)
            {
                return -2;
            }
            if(n == stop_value)
            {
                break;
            }
            else
            {
                *vec = n;
                zmienna++;
            }
        }
        while(zmienna < size);

    return zmienna;
}

int main()
{
    float tab[101] = {0};
    float *ptab = tab;
    int size = 0;
    float srednia = 0;
    float *psrednia = &srednia;
    
    printf("Wpisz liczby: ");
    
    size = read_vector_float(ptab, 101, 0);
    
        float srednia = 0;
        for(int i = 0; i <= size; i++)
        {
            srednia = srednia + *(tab + i);
        }
        printf("%f", srednia);
    
        float srednia = 0;
        for(int i = 0; i <= size; i++)
        {
            srednia = srednia + *(tab + i);
        }
        printf(" %f", srednia/size);
    

    
    return 0;
}

Funkcja read vector pobiera od użytkownika ciąg liczb i zwraca ilość pobranych liczb. Mój problem polega na tym że nie wiem dlaczego program zamiast zsumować liczby to wypisuje pierwszą podaną liczbę przez co zamiast średniej jest wypisana ta liczba podzielona przez ilość pobranych liczb. Zatem gdzie tkwi błąd przy sumowaniu?

Comments

Może najpierw usuń błędy które zgłasza kompilator:

prog.c:18:18: warning: comparing floating point with == or != is unsafe [-Wfloat-equal]
            if(n == stop_value)
               ~ ^  ~~~~~~~~~~
prog.c:45:15: error: redefinition of 'srednia'
        float srednia = 0;
              ^
prog.c:38:11: note: previous definition is here
    float srednia = 0;
          ^
prog.c:50:22: warning: implicit conversion increases floating-point precision: 'float' to 'double' [-Wdouble-promotion]
        printf("%f", srednia);
        ~~~~~~       ^~~~~~~
prog.c:52:15: error: redefinition of 'srednia'
        float srednia = 0;
              ^
prog.c:38:11: note: previous definition is here
    float srednia = 0;
          ^
prog.c:57:30: warning: implicit conversion increases floating-point precision: 'float' to 'double' [-Wdouble-promotion]
        printf(" %f", srednia/size);
        ~~~~~~        ~~~~~~~^~~~~
3 warnings and 2 errors generated.

A jak nie zgłasza, dodaj przełączniki ostrzeżeń do swojego procesu kompilacji.

---

Z jakiego korzystasz kompilatora? Visual Studio Code nie zgłosił mi takich błędów

---

Bo nie włączyłeś ostrzeżeń. Używam g++ oraz clang++. Tego ostatniego możesz użyć w Visual Studio.

Answer 1

Hej, pierwsz sprawa:

Nie powinno się inicjalizować zmiennych tego samego typu, z tą samą nazwą, mój kompilator przykładowo, wgl tego nie przepuścił ;) Ale to tylko taka uwaga i sugestia..

Co do Twojego problemu: proponuję ci użyć Debuggera, i dać breakpointa na linii 24, wgl. ta linia musi cię zainteresować, daj znać jak Ci poszło ;)

Comments

    do
    {
        if (scanf("%f", &n) != 1)
        {
            return -2;
        }
        if (n == stop_value)
        {
            break;
        }
        else
        {

            *(vec + zmienna) = n;
            zmienna++;
        }
    } while (zmienna < size);

    cout << "\nDisplaying address using pointers: " << endl;

    for (int i = 0; i < zmienna; ++i)
    {
        cout << "vec + " << i << " = " << *(vec + i) << endl;
    }

zwróć uwagę jak to teraz działa, zastanów się co było źle? Zauważ, że za każdym razem nadpisywałeś to samo miejsce w pamięci! Dlatego wynik wychodził Ci w taki a nie inny sposób, kolejna kwestia jest taka, że tablica musi być globalna.

 

Powodzenia ;)

---

Dzięki, teraz zadziałało, domyślam się że źle zapisywałem wskaźnik w funkcji