Напишіть текст помилки будь ласка.

"числа не перевищують по модулю 100000" - це не завдання для вас, це вхідна умова. Звісно, те, що ви її перевіряєте - це чудово, але ця гілка не має спрацьовувати ніколи. Завданням могло б бути "ігнорувати числа, що перевищують 1000000" чи "якщо якесь із чисел перевищує, вивести повідомлення "помилка".
Причому перевіряєте ви неправильно - в умові "не перевищують по модулю 100000", а ви модуль не обчислюєте. Натомість ви обчислюєте модуль, коли шукаєте максимум і мінімум - а там же не сказано "модуль максимального значення".

Біда у тому, що якщо я так зроблю, то ви не запам'ятаєте, як це правильно робити (бо зробив би я, а не ви) - ви запам'ятаєте, що треба не робити, а питати на форумах. Так що зробіть самі. Знову не вийде - знову спитаєте.

А вам і не треба ці умови записувати, вам ДАНО, що вони виконуються. Так само вам дано, що на вході будуть саме числа, а не літери - ви ж не перевіряєте, чи не літери на вході.

По-хорошому - так. Але можете залишити - він не має впливати на результат, бо і так виконується. Ну, хіба що виправити - там же "не перевищують", а у вас - менше.

Можливо там потрібто ввести умову за якої буде вибивати помилку? (Якщо числа по модулю перевищуватимуть 100000)

До речі, в unsigned short int 1000000 може не влізти. І ніхто не казав, що числа будуть додатними.

А break ви нащо робите?
Тип n чому не виправили?
Ну і теги [ code ] та вирівнювання не забувайте. Якщо хочете, щоб вам допомагали - зробіть так, щоб це було зручно.

виправлений код

#include <iostream>
#include <cmath>
using namespace std;
int main()
{
int s, max, min, n, k;
             cin>>k;
         cin >> n;
        min = max = n;

        for(s=1;s<k;  s++)
        {
                cin >> n;
                if (n>max) max=n;
                else if (n<min) min=n;
          }
        int c=max-min;
        cout<<c;
}