Среднее значение

[Dima-War]

Требовалось написать программу, которая запрашивает числа и считает среднее значение. Число 9999 говорит о том, что последняя цифра введена и выводит среднее значение. В моей программе число 9999 тоже приписывается к переменной, считающей сумму чисел. Я смог исправить этот недочет, но можно ли сделать как то покрасивее?

#include <iostream>
#include <conio.h>
#include <clocale>

using namespace std;

int main()
{
	setlocale (LC_CTYPE,"Russian");
	int a = 0, b = 0, c = 0, n = 0;         //а - введенное число, b - сумма введенных чисел, c - кол-во введенных чисел, n - ср. арифм.
	do
	{
	cout << "Введите число: ";
	cin >> a;
	b += a;
	c++;
	}
	while (a != 9999);

	c = c - 1;
	b = b - 9999;
	n = b / c;
	cout << "Среднее значение " << n << endl;

	_getch();
	return 0;
}

[Dima-War]

Неужели нету других вариантов?

[Vago]

#include <iostream>
#include <clocale>

using namespace std;

int main() {
	
   setlocale (LC_CTYPE,"Russian");
   // a - введенное число, s - сумма введенных чисел, n - кол-во введенных чисел, sa - ср. арифм.
   int      a , s = 0, n = 0 ; 
   double   sa ;
   
   cout << "Введите число: " ; cin >> a;
   while (a != 9999) {
      s += a ;
      n++ ;
      cout << "Введите число: " ; cin >> a;
   }
   if ( n > 0 ) {
      sa = (double)s / n ;
      cout << "Среднее значение " << sa << endl;
   }

   return 0 ;

}

[Dima-War]

Да... Проверку на ноль я забыл поставить. Спасибо.

[Smitt&Wesson]

Вот более "элегантное" решение.

#include <iostream>
#include <clocale>
#include <stdlib.h>

using namespace std;

int main() {
	
   setlocale (LC_CTYPE,"Russian");
   // a - введенное число, s - сумма введенных чисел, n - кол-во введенных чисел
   double      a = 0 , s = 0, n = 0;

   m1:cout << "Введите число: " ; cin >> a;
      if(a != 9999){
      s += a;
      if(a != 0)n++;}
      else{
      cout << "Среднее значение " << s/n << endl;
      system("pause");
      return 0 ;
      }
      goto m1;
}

[Vago]

Сообщение от Smitt&Wesson

Вот более "элегантное" решение.

#include <iostream>
#include <clocale>
#include <stdlib.h>

using namespace std;

int main() {
	
   setlocale (LC_CTYPE,"Russian");
   // a - введенное число, s - среднее арифм.
   int      a = 0, s = 0 ; 

   do
   {
      s = (s + a)/2 ;
      cout << "Введите число: " ; cin >> a;
   }while(a != 9999);
   cout << "Среднее значение " << s << endl;
   system("pause");
   return 0 ;

}

Запустил, ввёл 1, 2, 4, 9999, получил 2 ...
Запустил, ввёл 1, 1, 1, 9999, получил 0 ...
Запустил, ввёл 3, 3, 3, 3, 9999, получил 2 !

В чём, простите, состоит "элегантность"?...

[Smitt&Wesson]

Сообщение от Vago

В чём, простите, состоит "элегантность"?...

Я код исправил. Собственно пытался сотворить по этой статье, но чего-то недопонял. Разбираться дальше не стал.
Кстати, это именно тот случай где уместно применение непопулярного оператора goto.

[Vago]

Сообщение от Smitt&Wesson

Я код исправил.

#include <iostream>
#include <clocale>
#include <stdlib.h>

using namespace std;

int main() {
	
   setlocale (LC_CTYPE,"Russian");
   // a - введенное число, s - сумма введенных чисел, n - кол-во введенных чисел
   double      a = 0 , s = 0, n = 0;

   m1:cout << "Введите число: " ; cin >> a;
      if(a != 9999){
      s += a;
      if(a != 0)n++;}
      else{
      cout << "Среднее значение " << s/n << endl;
      system("pause");
      return 0 ;
      }
      goto m1;
}

Запустил, ввёл 2, 0, 0, 2, 9999, получил 2 !