Простой средний расчет

isdigit проверяет, является ли символ цифрой. Тест достигается только после присвоения 0 номеру, а 0 — это управляющий код, а не цифра, поэтому isdigit(0) всегда ложно, и поэтому ваше условие while всегда истинно.

 ...
       number=0;
 ...
} while( !isdigit(number) );      

Вместо этого проверьте входной поток, чтобы определить, успешно ли он прочитал значение.

int main()
{
    const size_t COUNT = 100;
    int tab[COUNT];
    size_t n;   

    cin.tie(&cout); // ensures cout flushed before cin read
    // (not required if your runtime complies with that part of the standard)

    for (n = 0; n < COUNT; ++n ) {
        cout << "Give " << n+1 << " number : ";
        cin >> tab[n];

        if (!cin)
            break;
    }

    if (n > 0) // average is undefined if n == 0
        cout << average(tab, n) << endl;

    return 0;
}

Вы используете isdigit здесь неправильно - он используется для проверки того, является ли char числовым или нет - вы не можете использовать его для проверки int.

Вы, вероятно, захотите рассмотреть возможность использования специального значения для завершения ввода, например. -1 или -999. Если это неприемлемо, вам нужно будет прочитать строку, а не целое число, а затем решить, является ли оно числовым или нет.

Помимо неправильного использования isdigit(), который вместо этого должен использовать какой-то дозорный механизм, нет необходимости хранить числа в массиве. Текущей суммы и подсчета чисел достаточно для расчета среднего.

Кроме того, должна быть предусмотрена проверка введенных нулевых элементов, чтобы предотвратить ошибку деления на ноль.

Лучшим методом обнаружения ввода не числа является проверка состояния cin после считывания значения:

// ...
if (cin >> number)
{
  tab[n++] = number;
}
else
{
  break;  // break out of loop
}

Также помните, что могут быть и другие причины сбоя ввода, кроме неправильного ввода числа.

Есть несколько проблем с вашим кодом:

cin >> number;

Вы не проверяете, не удалось ли выполнить операцию извлечения потока. Простой способ сделать это — использовать оператор преобразования operator void*():

if (cin >> number)
  ... operation succeeded ...

Приведенный выше код эквивалентен проверке failbit и badbit .

Ваше использование isdigit() также неверно, поскольку вы передаете число (например, 1234 вместо символа (например, 'z'). Несмотря на это, добавление проверки ошибки операции извлечения потока устраняет необходимость в такой проверке на основе цифр.

Вы можете использовать lexical_cast из boost. Вы читаете ввод в строку, а lexical_cast проверяет, можно ли его преобразовать в строку. Это также гарантирует, что ваша строка не будет слишком длинной для преобразования, если это отрицательное число или число вообще.

#include <iostream>
#include <string>
#include <boost/lexical_cast.hpp>

using std::cout;
using std::cin;
using std::endl;
using std::string;
using boost::lexical_cast;
using boost::bad_lexical_cast;

double average(int tab[], int i){

    int sum=0;

    for(int j=0; j<i; ++j){
            sum+=tab[j];
    }
    return (double)sum/i;

}

int main()
{

    int tab[100]; //this is a fairly low level construct which might want to
                  // into a std::vector 

    string input; 

    int n;
    try{
        for (n = 0 ;n < 100; n++) {
           cout << "Give " << n+1 << " number : ";
           cin >> input;                          //read number into string
           tab[n]= lexical_cast<int>(input);     //conversion with lexical_cast
                                                 //if not possible exception is 
                                                 //thrown
        }
     }
     catch(bad_lexical_cast &){
        cout << "not a number" << endl;
     } 

    cout << average(tab, n) << endl;

    return 0;
}

isdigit сообщит вам, представляет ли код символа набора символов одну из цифр от 0 до 9.

Следовательно (я предполагаю, что вы используете ASCII), вы можете просто использовать символ и проверить его диапазон кода ASCII:

    int tab[100]; 
    int n = 0;    
    char c;

    while (n++ < 100)
    {
       cout << "Give " << n << " number : "; 
       cin >> c;
       if (c < 48 || c > 57)
          break;
       tab[n - 1] = c - 48;            
    }

    cout << average(tab, n - 1) << endl; 

    getch(); 
    return 0; 

Вы также можете использовать cin.getline и atoi или strtod:

int tab[100]; 
int n=0;    
int number=0; 
char input[10];

while (n++ < 100)
{     
   cout << "Give " << n << " number : ";
   memset(input, 0x00, 10);
   cin.getline(input, 10);
   number = atoi(input);
   if (number > 0)
      tab[n-1] = number; 
   else
      break;
}

cout << average(tab, n-1) << endl; 

getch(); 
return 0; 

Существуют и другие методы, которые вы можете использовать, однако они должны дать вам некоторые идеи.