Проблема с массивами на C++.

[Creston]

Здравствуйте, помогите, пожалуйста, решить задачку, у меня что-то не хочет.
Задача: Ввести в массив оценки студентов, посчитать среднее арифметическое, если оно меньше 2,5 написать, что студент должен быть отчислен.
Вот код, который я написал:

#include<iostream>
using namespace std;
int main ()
{
    setlocale (0,"rus");
    const int n=4;
    int A[n];    
    int i,m=0;
    double avg=0;
    for (i=1;i<=4; i++) {
        cout << "Введите оценку студента";
        cin>>A[m];
        }
        for (i=1;i<=4;i++) {
        avg = (avg +A[m])/4;
        }
        
        cout << avg << ' ';
        cout << A[n];
        if (avg<=2.5) cout << "Студент должен быть отчислен" << endl;
        else cout << "Пусть учится" << endl;
        system ("pause");
return 0;
}

Он неправильно считает и выдает неправильный результат. Скажите, в чем ошибка*+?

[p51x]

Ошибка в том, что массивы нумеруются с 0

[Creston]

Исправил на нули в цикле for все равно та же ерунда, укажите, в каких строчках ошибка?

[p51x]

Во-первых, у вас в задании "оценки студентов", т.е. скорее всего надо не массив оценок, а массив студентов с оценками.

Во-вторых, "Исправил на нули в цикле for" этого мало. Надо и вторую границу правильную сделать.

В-третьих,

for (i=1;i<=4; i++) {
cout << "Введите оценку студента";
cin>>A[m];
}

Зачем вам цикл, если вы индекс его не используете? Почему вы все оценки пишите в один и тот же элемент?

В-четвертых,

for (i=1;i<=4;i++) {
avg = (avg +A[m])/4;
}

Вы уверены, что знаете значение термина "среднее арифметическое"? Сколько раз вы тут поделите на 4? Почему вы прибавляете один и тот же элемент массива?

В-пятых,

cout << A[n];

Зачем выводить один элемент массива? Тем более, что его не существует (нумерация с 0)...

В-шестых,

avg<=2.5

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

[Creston]

"Во-первых, у вас в задании "оценки студентов", т.е. скорее всего надо не массив оценок, а массив студентов с оценками"

Напишите, пжлст, как его сделать?

"В-третьих,
Код:

for (i=1;i<=4; i++) {
cout << "Введите оценку студента";
cin>>A[m];
}

Зачем вам цикл, если вы индекс его не используете? Почему вы все оценки пишите в один и тот же элемент?"

Для того, чтобы строку "Введите оценку студента" он ввел i раз. Оценки в один элемент: как на уровне кода написать в разные элементы?


"Почему вы прибавляете один и тот же элемент массива?"
А как прибавить другой элемент? Написать m+1?

"В-пятых,
Код:

cout << A[n];

Зачем выводить один элемент массива? Тем более, что его не существует (нумерация с 0)..."

Как вывести все элементы массива?

Это не домашнее задание, я изучаю С++ самостоятельно, в перерывах на работе, и срок моего программирования только 2 недели. Поэтому много ошибок

[min@y™]

for (i = 0; i < n; i++) // смотри внимательно

и выкинуть переменную m.

[p51x]

Напишите, пжлст, как его сделать?

Вариантов много. Например, можно сделать структуру/класс студент и в нем поля для оценок, у же их будет массив.

Для того, чтобы строку "Введите оценку студента" он ввел i раз.

Тогда хотя бы писали б "Введите iтую оценку студента".

Оценки в один элемент: как на уровне кода написать в разные элементы?

Вы серьезно? Вы почтовые ящики видели? А квартиры? И многое другое? Вот с массивами таже фигня. Есть имя и есть номер элемента. А теперь сами ответьте как писать не в а[0], а в другой.

А как прибавить другой элемент? Написать m+1?

У вас есть массив с элементами под номерами 0, 1, 2, ... и есть счетчик цикла, который бегает 0, 1, 2, ... И теперь самое сложное это их совместить.

Как вывести все элементы массива?

По старинке в цикле или по модному, типа

copy(Arr,Arr+5,ostream_iterator<int>(cout,"\n"));