Найти среднее значение элементов массива (без учёта максимального и минимального) и вывести результат на экран

@Fenix333666 · Регистрация: 25.12.2015

Студворк — интернет-сервис помощи студентам

Условие задачи (добавлено спустя сутки после начала обсуждения, так что не удивляйтесь):

В некоторых видах спортивных состязаний выступление каждого спортсмена оценивается несколькими судьями, затем из всей совокупности оценок удаляются наиболее высокая и наиболее низкая, а для оставшихся вычисляется среднее арифметическое, которое и идет в зачет спортсмену. Если наиболее высокую оценку выставляли несколько судей, то из совокупности оценок удаляется только одна такая оценка; аналогично поступают с наиболее низкими оценками.
Даны натуральное число n, действительные положительные числа f1, ..., fn (n 3). Считая, что числа f1, ..., fn – это оценки, выставленные судьями одному из участников соревнований, определить оценку, которая пойдет в зачет этому спортсмену.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
#include <iostream> 
#include <stdio.h> 
#include <conio.h> 
using namespace std;
int main()
{
    int const n = 5;
    int i;
    float max, min;
    double f[n], S;
    for (i = 0; i < n; i++)
    {
        cout << "f["<<i<<"] = ";
        cin >> f[i];
    }
    max = f[0];
    min = f[0];
    while (i <= n);
    {
        if (f[n] > max)
        {
            f[n] = max;
        }
        if (f[n] < min)
        {
            f[n] = min;
        }
    }
    S = 0;
    for (i = 0; i < n; i++)
        while (i <= n)
        {
            if (i != max)
                if (i != min)
                {
                    S = S + f[n];
                    S = S / n;
                }
        }
    cout << "S = " << S;
    return 0;
}

Прошу помочь с программой. Знаю, что где-то есть ошибка, но не могу найти ее.

@olya7 · 25.12.2015, 12:36

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

while (i <= n);

думаю ; в конце строчки излишняя

@Fenix333666 · 25.12.2015, 12:40 **[ТС]**

Упс. Спасибо) Но результат все тот же

@olya7 · 25.12.2015, 12:45

в этом же цикле не меняется счетчик элементов
в последнем цикле то же самое

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

for (i = 0; i < n; i++)
while (i <= n)

эта конструкция тоже странная
да и весь код странноватый..
Добавлено через 3 минуты

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
#include <iostream>
#include <stdio.h>
#include <conio.h>
using namespace std;
int main()
{
int const n = 5;
int i;
float max, min;
double f[n], S;
for (i = 0; i < n; i++)
{
cout << "f["<<i<<"] = ";
cin >> f[i];
}
max = f[0];
min = f[0];
i=0;
while (i <n)
{
if (f[i] > max) max = f[n];
if (f[i] < min) min = f[n];
i++;
}
S = 0;
for (i = 0; i < n; i++)
 
if (i != max && i != min )
 
    S+=f[i];
 
 
S/=n;
cout << "S = " << S;
return 0;
}

@Fenix333666 · 25.12.2015, 12:55 **[ТС]**

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

в этом же цикле не меняется счетчик элементов
в последнем цикле то же самое
Сообщение от Fenix333666
for (i = 0; i < n; i++)
while (i <= n)
эта конструкция тоже странная
да и весь код странноватый..
}

Спасибо огромное) Можете объяснить что было исправлено и как? Просто я совсем недавно начал изучать язык C++

@olya7 · 25.12.2015, 15:28

я просто привела в порядок обновление счетчиков в циклах, и немного подправила код

@sourcerer · 25.12.2015, 15:55

olya7, с отступами Вы зря так поступили. Было-то ведь заметно лучше.

Добавлено через 14 минут
Fenix333666, вот:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
#include <iostream>
 
int main()
{
    using std::cout;
    using std::cin;
 
    const int ARR_SIZE = 5;
    double arr[ARR_SIZE];
 
    cout << "Please enter values of array elements:\n";
    for (int i = 0; i < ARR_SIZE; i++)
    {
        cout << "arr[" << i << "] = ";
        cin >> arr[i];
    }
 
    double max = arr[0];
    double min = arr[0];
    for (int i = 0; i < ARR_SIZE; i++)
    {
        if (arr[i] > max)
            max = arr[i];
        if (arr[i] < min)
            min = arr[i];
    }
 
    double sum_of_elements_exept_max_and_min = 0;
    for (int i = 0; i < ARR_SIZE; i++)
        if (i != max && i != min )
            sum_of_elements_exept_max_and_min += arr[i];
 
    cout << "\nAverage value of all elements (exept maximal and minimal elements) is: "
         << sum_of_elements_exept_max_and_min / ARR_SIZE;
 
    return 0;
}

@Fenix333666 · 26.12.2015, 08:30 **[ТС]**

gru74ik, не совсем понятно, как исключаются максимальный и минимальный элементы

Ввожу ваш код - все работает так как нужно. На моем коде считаются все элементы

@sourcerer · 26.12.2015, 08:36

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

не совсем понятно, как исключаются максимальный и минимальный элементы

С помощью условия в 30 строке приведённого мной кода.

@Fenix333666 · 26.12.2015, 14:23 **[ТС]**

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

С помощью условия в 30 строке приведённого мной кода.

Оно у меня тоже есть. Но они не исключаются

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#include <iostream>
#include <conio.h> 
using namespace std;
int main()
{
    int const n = 5;
    int i;
    float max, min;
    double f[n], S;
    for (i = 0; i < n; i++)
    {
        cout << "f[" << i << "] = ";
        cin >> f[i];
    }
    max = f[0];
    min = f[0];
    for (i = 0; i < n; i++)
    {
        if (f[i] > max) max = f[n];
        if (f[i] < min) min = f[n];
    }
    S = 0;
    for (i = 0; i < n; i++)
    if (i != max && i != min)
    S += f[i];
    S /= n;
    cout << "S = " << S;
    _getch();
    return 0; 
}

@sourcerer · 26.12.2015, 14:30

Fenix333666, используйте другие имена (не max и min, а, скажем, max_value и min_value), либо не используйте директиву using (третья строка Вашего кода). Вместо неё используйте объявления using, либо квалификатор std с операцией разрешения контекста (два двоеточия :: ) около каждого имени из пространства имён std.

То есть, надо либо так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#include <iostream>
#include <conio.h>
using namespace std;
int main()
{
    int const n = 5;
    int i;
    double max_value, min_value;
    double f[n], S;
    for (i = 0; i < n; i++)
    {
        cout << "f[" << i << "] = ";
        cin >> f[i];
    }
    max_value = f[0];
    min_value = f[0];
    for (i = 0; i < n; i++)
    {
        if (f[i] > max_value) max_value = f[i];
        if (f[i] < min_value) min_value = f[i];
    }
    S = 0;
    for (i = 0; i < n; i++)
    if (i != max_value && i != min_value)
    S += f[i];
    S /= n;
    cout << "S = " << S;
    _getch();
    return 0;
}

либо так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
#include <iostream>
#include <conio.h>
using std::cout;
using std::cin;
int main()
{
    int const n = 5;
    int i;
    float max, min;
    double f[n], S;
    for (i = 0; i < n; i++)
    {
        cout << "f[" << i << "] = ";
        cin >> f[i];
    }
    max = f[0];
    min = f[0];
    for (i = 0; i < n; i++)
    {
        if (f[i] > max) max = f[i];
        if (f[i] < min) min = f[i];
    }
    S = 0;
    for (i = 0; i < n; i++)
    if (i != max && i != min)
    S += f[i];
    S /= n;
    cout << "S = " << S;
    _getch();
    return 0;
}

либо так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#include <iostream>
#include <conio.h>
 
int main()
{
    int const n = 5;
    int i;
    float max, min;
    double f[n], S;
    for (i = 0; i < n; i++)
    {
        std::cout << "f[" << i << "] = ";
        std::cin >> f[i];
    }
    max = f[0];
    min = f[0];
    for (i = 0; i < n; i++)
    {
        if (f[i] > max) max = f[i];
        if (f[i] < min) min = f[i];
    }
    S = 0;
    for (i = 0; i < n; i++)
    if (i != max && i != min)
    S += f[i];
    S /= n;
    std::cout << "S = " << S;
    _getch();
    return 0;
}

@olya7 · 26.12.2015, 15:36

тогда это

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

C++
1
2
3
4
5
6
max = f[0];
* * min = f[0];
* * for (i = 0; i < n; i++)
* * {
* * * * if (f[i] > max) max = f[i];
* * * * if (f[i] < min) min = f[i];

* * }

должно быть

C++
1
2
3
4
5
6
7
max = 0;
    min = 0;
    for (i = 1; i < n; i++)
    {
        if (f[i] > f[max]) max =i;
        if (f[i] < f[min]) min = i;
    }

@sourcerer · 26.12.2015, 15:40

olya7, не согласен.

@olya7 · 26.12.2015, 15:57

gru74ik, почему? находим индексы экстремумов и исключаем их при расчете суммы
либо, при условии нескольких экстремумов

C++
1
2
3
4
5
6
7
8
9
10
11
12
max = f[0];
    min = f[0];
    for (i = 0; i < n; i++)
    {
        if (f[i] > max) max = f[i];
        if (f[i] < min) min = f[i];
    }
    S = 0;
    for (i = 0; i < n; i++)
    if (f[i] != max && f[i[ != min)
    S += f[i];
    S /= n;

@sourcerer · 26.12.2015, 16:10

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

почему?

olya7, в моём варианте мы изначально инициализируем переменные max и min значениями элемента массива (если быть точным, значением первого элемента массива). Таким образом, если далее встретится элемент больший, нежели значение, хранящееся в max, либо меньшее, нежели значение, хранящееся в min, то соответствующее значение сразу же будет занесено в соответствующую переменную. А если не встретится, то нужное значение уже присвоено - даже лишних телодвижений не надо совершать.
В Вашем варианте Вы инициализируете переменные max и min нулём. Не надо обладать сверхъестественными способностями, чтобы представить ситуацию, когда в массиве хранятся, скажем, только отрицательные значения. Тогда у вас максимально значение (значение переменной max) так и останется равным нулю. Но значения равного нулю не будет ни в одном элементе массива! На самом деле реальный max будет, скажем -5. Пример:

C++
1
2
int const n = 5;
double f[n] = {-9, -7, -5, -6, -8}; // Реальный max равен -5. Упс... а у Вас он равен нулю.

Или ситуацию, когда в массиве хранятся, только значения больше нуля:

C++
1
2
int const n = 5;
double f[n] = {3, 7, 2, 6, 8}; // Реальный min равен 2. Упс... а у Вас он равен нулю.

@olya7 · 26.12.2015, 16:13

gru74ik, Вы, возможно, не дочитали

C++
1
2
3
4
5
6
7
max = 0;
    min = 0;
    for (i = 1; i < n; i++)
    {
        if (f[i] > f[max]) max =i;
        if (f[i] < f[min]) min = i;
    }

я ищу индексы максимума и минимума
со второй реализацией - погорячилась, не учла строгое неравенство при поиске

@sourcerer · 26.12.2015, 16:21

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

Вы, возможно, не дочитали

Я дочитал. Но Ваш код говорит о том, что Вы исключаете значения, а не индексы:

C++
1
2
3
4
5
6
7
8
9
10
11
12
max = f[0];
    min = f[0];
    for (i = 0; i < n; i++)
    {
        if (f[i] > max) max = f[i]; //присваиваете значение, а не индекс
        if (f[i] < min) min = f[i]; //присваиваете значение, а не индекс
    }
    S = 0;
    for (i = 0; i < n; i++)
    if (f[i] != max && f[i] != min) //исключаете значение, а не индекс
        S += f[i];
    S /= n;

Добавлено через 2 минуты
Более того, в первом варианте у Вас выход за пределы массива (строки 21 и 22).

@olya7 · 26.12.2015, 16:24

gru74ik, лопухнулась)

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

либо, при условии нескольких экстремумов

не лопухнулась

предролагалось, что изменение
Вашего последнего кода

с последним замечанием полностью согласна

@Dimension · 26.12.2015, 17:41

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#include <bits/stdc++.h>
using namespace std;
int main() {
    int a[] = { 1,4,1,2};
    int ma = *max_element(begin(a), end(a));
    int mi = *min_element(begin(a), end(a));
    int c = 0,sum=0;
    int s = sizeof(a) / sizeof(a[0]);
    for (int i = 0;i < s;i++) {
        if (a[i] != ma && a[i] != mi) {
            sum += a[i], c++;
        }
    }
    cout << ((s > 2 && c) ? 1.0*sum / c : 0);
    return 0;
}

@Fenix333666 · 26.12.2015, 19:36 **[ТС]**

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

Fenix333666, используйте другие имена (не max и min, а, скажем, max_value и min_value), либо не используйте директиву using (третья строка Вашего кода). Вместо неё используйте объявления using, либо квалификатор std с операцией разрешения контекста (два двоеточия :: ) около каждого имени из пространства имён std.

Хм... Похоже, что нужно было сразу написать условие задачи. Прошу прощения

В некоторых видах спортивных состязаний выступление каждого спортсмена оценивается несколькими судьями, затем из всей совокупности оценок удаляются наиболее высокая и наиболее низкая, а для оставшихся вычисляется среднее арифметическое, которое и идет в зачет спортсмену. Если наиболее высокую оценку выставляли несколько судей, то из совокупности оценок удаляется только одна такая
оценка; аналогично поступают с наиболее низкими оценками.
Даны натуральное число n, действительные положительные числа
f1, ..., fn (n>=3). Считая, что числа f1, ..., fn – это оценки, выставленные
судьями одному из участников соревнований, определить оценку, которая пойдет в зачет этому спортсмену.

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@Fenix333666 0 / 0 / 0 Регистрация: 25.12.2015 Сообщений: 9
	25.12.2015, 12:40 [ТС]
	Упс. Спасибо) Но результат все тот же 0

@olya7 505 / 317 / 236 Регистрация: 18.02.2013 Сообщений: 754
	25.12.2015, 15:28
	я просто привела в порядок обновление счетчиков в циклах, и немного подправила код 0

@Fenix333666 0 / 0 / 0 Регистрация: 25.12.2015 Сообщений: 9
	26.12.2015, 08:30 [ТС]
	gru74ik, не совсем понятно, как исключаются максимальный и минимальный элементы Ввожу ваш код - все работает так как нужно. На моем коде считаются все элементы 0

@sourcerer Модератор 5288 / 2376 / 342 Регистрация: 20.02.2013 Сообщений: 5,773 Записей в блоге: 20
	26.12.2015, 15:40
	olya7, не согласен. 0