Непонятная "ошибка" (проблема)

[_Sequence_]

Люди, помогите найти ошибку
Суть программы: "Найти сумму отрицательных элементов в массиве"

PHP код:

#include <iostream>

using namespace std;

int main()
{
    setlocale(LC_ALL, "Russian");
    int schet, a;
    int element;
    char reset;
    a = 0;
    do
    {
    cout << "Введите количество элементов в массиве: ";
    cin >> element;
        for (schet = 0;schet < element; schet++)
    {
    int massive[element];
    cin >> massive[element];
    if (massive[element] < 0)
        {
            a += massive[element];
        }
    }
    cout << "Сумма отрицательных элементов: " << a << endl;
    cout << "Хотите попробовать еще, нажмите +" << endl;
    cin >> reset;
    }
    while (reset == '+');
    return 0;
} 


При введении кол-ва эл-тов в массиве "4"; первое значение "-7", 3-е значение массива "-2", программа выдает не то, что нужно.

Заранее благодарен(=

[crymchanin]

int massive[element];

используйте динамическое выделение памяти.

[pufystyj]

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

Люди, помогите найти ошибку
Суть программы: "Найти сумму отрицательных элементов в массиве"

PHP код:

#include <iostream>

using namespace std;

int main()
{
    setlocale(LC_ALL, "Russian");
    int schet, a;
    int element;
    char reset;
    a = 0;
    do
    {
    cout << "Введите количество элементов в массиве: ";
    cin >> element;
        for (schet = 0;schet < element; schet++)
    {
    int massive[element];
    cin >> massive[element];
    if (massive[element] < 0)
        {
            a += massive[element];
        }
    }
    cout << "Сумма отрицательных элементов: " << a << endl;
    cout << "Хотите попробовать еще, нажмите +" << endl;
    cin >> reset;
    }
    while (reset == '+');
    return 0;
} 


При введении кол-ва эл-тов в массиве "4"; первое значение "-7", 3-е значение массива "-2", программа выдает не то, что нужно.

Заранее благодарен(=

1) Вы пишете int massive[element]; в цикле, то есть обьявляете каждный раз массив по новому, наверное ошибка кроеться в этому. Сделайте так:

#include <iostream>

using namespace std;

int main()
{
    setlocale(LC_ALL, "Russian");
    int schet, a;
    int element;
    char reset;
    a = 0;
    do
    {
    cout << "Введите количество элементов в массиве: ";
    cin >> element;

    int massive[element];

        for (schet = 0;schet < element; schet++)
    {
    cin >> massive[element];
    if (massive[element] < 0)
        {
            a += massive[element];
        }
    }
    cout << "Сумма отрицательных элементов: " << a << endl;
    cout << "Хотите попробовать еще, нажмите +" << endl;
    cin >> reset;
    }
    while (reset == '+');
    return 0;
}

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

[pufystyj]

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

int massive[element];

используйте динамическое выделение памяти.

Это типа так модно? Зачем? Тем более он(а) ещё только учиться и может этого не знать.

Не парьте парню (или девушке).

[Granus]

Это типа так модно? Зачем?

Затем, что это неправильный с++ код, несмотря на то, что некоторые компиляторы позволяют так делать.

_Sequence_, вы создаете массив с element элементами. Нумерация элементов такого массива - от 0 до element-1, Вы же обращаетесь к элементу с индексом element. В зависимости от реализации, Вы можете получить от некого мусорного значения до ошибки выполнения.
У Вас есть цикл по переменной schet, её и используйте для индексирования. И, как уже сказал pufystyj, вынесите объявление массива из цикла.

[pufystyj]

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

Затем, что это неправильный с++ код, несмотря на то, что некоторые компиляторы позволяют так делать.

_Sequence_, вы создаете массив с element элементами. Нумерация элементов такого массива - от 0 до element-1, Вы же обращаетесь к элементу с индексом element. В зависимости от реализации, Вы можете получить от некого мусорного значения до ошибки выполнения.
У Вас есть цикл по переменной schet, её и используйте для индексирования. И, как уже сказал pufystyj, вынесите объявление массива из цикла.

Ах да, не заметил. Но все же, чем здесь динамическое выделение памяти спасет? Статические переменные отменили?

[Granus]

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

Но все же, чем здесь динамическое выделение памяти спасет? Статические переменные отменили?

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

[pufystyj]

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

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

Ну в принципе да, но я о записи, зачем писать явно, если это не важно.

[Granus]

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

Ну в принципе да, но я о записи, зачем писать явно, если это не важно.

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

[crymchanin]

Прямо холивар) А пример так никто и не написал (я в том числе)
Динамическое выделение памяти:

C++

int* massive = new int[element];

C

int* massive = (int*)malloc(element);

Ну и не забываем освобождать память после:

C++

delete[] massive;

C

free(massive);