[C++] Średnia arytmetyczna liczb nieujemnych

[anilahcim]

Mam problem w rozwiązaniu zadania:

Wczytuj kolejne liczby całkowite z klawiatury. Zakończ wczytywanie gdy użytkownik wprowadzi liczbę ujemną. Wypisz na ekran średnią arytmetyczną wszystkich wprowadzonych liczb nieujemnych.

To jest moje rozwiązanie:

Kod: Zaznacz cały

#include <iostream>
using namespace std;

int main()
{
	int x, suma, ilosc;
	double srednia;
	ilosc = 0;
	suma = 0;

	do
	{
		cout<< "Wpisz liczbe calkowita: " << endl;
		cin>> x;
		if (x>=0)
			suma += x;
			ilosc++;
			srednia=suma/ilosc;
	}
	while (x>=0);
		cout<< "Srednia arytmetyczna liczb nieujemnych: " << srednia << endl;

system("pause");
return 0;
}

Wynik wychodzi mi zły i nie wiem gdzie jest błąd. Nie widzę też różnicy w kolejności 16 i 17 wiersza (suma i ilość), a jeśli zamienię je ze sobą dostaję różne wyniki. Na przykład, w powyższej wersji, jeśli wpisuję liczby: 10, 5, 5, 20, -10 otrzymuję wynik 8. Jeśli zamienię wiersz 16 i 17, wynik jest 7.

Proszę o pomoc i z góry dziękuję.

[lemoid]

porównaj do siebie te kody

Kod: Zaznacz cały

#include <iostream>
using namespace std;
 
int main() {
double suma=0.0, ilosc=0.0;
double x, srednia=0.0;
 
while (cin>>x && x>=0)
{
ilosc++;
suma+=x;
 
}
srednia=suma/ilosc;
cout << srednia << endl;
}

[anilahcim]

Dzięki za odpowiedź. A dlaczego funkcja do...while tu nie pasuje? W Twoim kodzie rzeczywiście wynik wychodzi poprawnie (chociaż dalej nie rozumiem, na czym polega błąd u mnie), ale gdzie wstawić "cout<< "Wpisz liczbe calkowita: " << endl;"? Jeśli wstawię przed pętlą, to pojawi się tylko raz, jeśli w pętli albo po niej, to najpierw trzeba wpisać liczbę z klawiatury, a dopiero potem pokazuje się komunikat, żeby wpisać liczbę.

[knrdk]

W swoim kodzie zapomniałaś (?) o nawiasach klamrowych po IF.

[anilahcim]

No tak Dzięki!

[royas]

Jeszcze jedna ważna rzecz: jeśli suma i ilość jest intem to ich dzielenie również da inta, czyli nawet jeśli średnia jest doublem, otrzymamy wynik zaokrąglony do liczby całkowietej.