VPF::Задача по массиву - Форум программистов

Creston

Дата 21.3.2015, 17:26 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 3
Регистрация: 20.3.2015

Репутация: нет
Всего: нет

Здравствуйте, у меня вот такой код:

Код


#include<iostream>
using namespace std;
int main ()
{
    int i,a[80],n=0;
    double avg=0;
    for (i=1;i<=4;i++) {
        cout << "Vvedite otsenky studenta" << endl;
        cin>>n;
        n=a[n];
        }
        avg+=a[n];
        avg/=4;
       cout << avg << endl;
 if (avg<=2.5) cout << "Student doljen bit otchislen" << endl;
        else cout << "Pust ychitsya" << endl;
        system("pause");
        return 0;
        }

В коде какая-то ошибка. Задача: ввести 4 оценки учащегося, если средний бал, не превышает 2.5, выдать сообщение, что он должен быть отчислен, если нет, то что пусть учится. У меня же неправильно считает переменную avg, не могу понять почему, то ли из-за массива, то ли еще хрен знает почему. Помогите, пжлст, кто чем сможет

kemiisto

Дата 21.3.2015, 17:36 (ссылка)

(нет голосов)

Загрузка ...

Дикий Кот. =^.^=

Награды: 1

Профиль
Группа: Участник Клуба
Сообщений: 3292
Регистрация: 29.7.2007

Репутация: 5
Всего: 160

Строки 9-10:

Код

cin>>n;
n=a[n];

Вы считываете значение, введённое с клавиатуры в переменную n, назначение которой совершенно непонятно из её названия. Затем Вы совершаете ещё более непонятное присваивание этой переменной n значение массива a с индексом n. Каким бы целям не служила переменная n, этот кусок кода - бред собачий.

--------------------

XMPP-канал.

Unit testing isn't enough. You need static typing.

Пей, кури, пиши на си! smile

Creston

Дата 21.3.2015, 17:40 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 3
Регистрация: 20.3.2015

Репутация: нет
Всего: нет

kemiisto, Все дело в том, что задача на массивы. n нужно исключительно, чтобы в нее записать файл, если то же самое делать без массива, все равно переменная avg принимает какие угодно значения, кроме правильных.
Я только новичок (3 недели как), как это лучше переписать, с учетом того, что дошел пока только до массивов?

kemiisto

Дата 21.3.2015, 18:10 (ссылка)

(голосов:1)

Загрузка ...

Дикий Кот. =^.^=

Награды: 1

Профиль
Группа: Участник Клуба
Сообщений: 3292
Регистрация: 29.7.2007

Репутация: 5
Всего: 160

Цитата(Creston @ 21.3.2015, 16:40

)

n нужно исключительно, чтобы в нее записать файл, если то же самое делать без массива, все равно переменная avg принимает какие угодно значения, кроме правильных.

Бессмыслица полная, уж извините. Да и массивы тут вообще не нужны, строго говоря. Но уж если так хочется с массивом, то что-то вроде:

Код


#include<iostream>

int main()
{
    int a[4];
    double avg = 0;
    
    for (int i = 0; i < 4; ++i) {
        std::cout << "Vvedite otsenky studenta" << std::endl;
        std::cin >> a[i];
        avg += a[i];
    }
    
    avg /= 4;
    std::cout << avg << std::endl;
    
    if (avg <= 2.5) {
        std::cout << "Student doljen bit otchislen" << std::endl;
    } else {
        std::cout << "Pust ychitsya" << std::endl;
    }
    
    system("pause");
    
    return 0;
}

Это сообщение отредактировал(а) kemiisto - 21.3.2015, 18:12

--------------------

XMPP-канал.

Unit testing isn't enough. You need static typing.

Пей, кури, пиши на си! smile

Creston

Дата 21.3.2015, 18:23 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 3
Регистрация: 20.3.2015

Репутация: нет
Всего: нет

kemiisto, Огромное спасибо.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Для новичков \| Следующая тема »