bledne wyniki sredniej artymetycznej

Question

cześć, w programie do średniej artymetycznej nie wychodzi mi za nic właściwy wynik a powinien : |

 

#include <iostream>

using namespace std;

unsigned srednia(unsigned suma, unsigned liczba, unsigned n)
{
    cout<<"Podaj ile liczb do sredniej"<<endl;
    cin>>n;
for(unsigned i=1; i<=n; i++)
{
    cout<<"Podaj "<<i<<" liczbe"<<endl;
    cin>>liczba;
    suma+=liczba;
}

cout<<"srednia wynosi: "<<suma/n<<endl;

}
int main()
{
unsigned suma;
unsigned liczba;
unsigned n;
cout<<srednia(suma,liczba,n);      // mimo iż dobre typy zmiennych to wynik po przecinku też nie wychodzi dobry
cout<<" <-- nie wiem co to O.O"<<endl;   // tutaj po zakończonym programie wyskakują mi jakieś dziwne cyfry
return 0;

}

 

W zakomentowanych miejscach jest jeszcze pare pytań :)

Answer 1

Po pierwsze średnia najczęściej będzie wychodzić jako liczba lub cyfra zmienno przecinowa. Zastanów się nad typem zmiennej.

Po wtóre "unsigned" dodajesz przed nazwą zmiennej np "unsigned long" oznacza to że zakres zmiennej przesunie sie i będzie liczony od zera w górę.

Tak w ogóle to chyba masz źle napisaną funkcję "srednia" - chodzi mi o typ, no a poza tym skoro to nie jest "VOID" to powinna zwracać jakąś wartość.

 

 

zachęcam do zapoznania się z : http://www.algorytm.edu.pl/wstp-do-c/typy-zmiennych.html

oraz https://www.youtube.com/watch?v=HHplT1A4_A4

Answer 2

Źle napisałeś funkcję srednia, jeśli funkcja ma coś zwracać to zamiast cout napisz return.  Zmienne suma, liczba i n powinny być nad funkcją srednia. Po drugie przed unsigned musisz podac typ zmiennej, a najlepiej zamiast unsigned wpisz po prostu int. Najlepiej by było jakbyś to obejżał: https://www.youtube.com/watch?v=HHplT1A4_A4

Answer 3

Nie wiem czy o to ci chodziło.

Ale działa.

#include <iostream>

using namespace std;

void srednia()
{
    double liczba;
    unsigned int n;
    double suma;
    cout<<"Podaj ile liczb do sredniej"<<endl;
    cin>>n;
    for(unsigned i=1; i<=n; i++)
    {
        cout<<"Podaj "<<i<<" liczbe"<<endl;
        cin>>liczba;
        suma+=liczba;
    }

    cout<<"srednia wynosi: "<<suma/n<<endl;

}
int main()
{
    srednia();      // mimo iż dobre typy zmiennych to wynik po przecinku też nie wychodzi dobry
    cout<<" <-- nie wiem co to O.O"<<endl;   // tutaj po zakończonym programie wyskakują mi jakieś dziwne cyfry
    return 0;

}

 

Answer 4

Tutaj masz działający kod, trochę źle napisałeś swoją funkcję : 

 

#include <iostream>

using namespace std;

double srednia ()
{
    int n;
    double liczba;
    double suma=0;
    cout<<"Podaj ile liczb do sredniej"<<endl;
    cin>>n;
    
    for(int i=1; i<=n; i++)
    {
    cout<<"Podaj "<<i<<" liczbe"<<endl;
    cin>>liczba;
    suma+=liczba;
    }

    return suma/n;
}

int main()
{
    cout<<" średnia wynosi : "<< srednia() << endl;
    return 0;
}