Cумма отрицательных элементов массива - C++ - Ответ 1358610

/*Дан массив из N элементов (вещественные числа). Вы-
числить: 1) сумму отрицательных элементов массива; 2) произ-
ведение элементов массива, расположенных между макси-
мальным и минимальным элементами. Упорядочить элементы
по возрастанию.*/
 
#include <iostream>
#include <Windows.h>
using namespace std;
int main ()
{
    int n=10;
    int i,j,x1,x2;
    float b,max=0, min=0,y1,y2;
    char str[256];
    float massiv[11];
    /*Такие переменные лучше инициализировать сразу*/
    float SummaMinus, PrzElem;
    /*В С/С++ массив начинается с 0-го индекса*/
    for (i=1; i<=n; i++) // ввод массива
    {
        AnsiToOem ("Введите элемент массива №",str);
        cout<<str<<i<<" ";
        cin>>massiv[i];
        cout<<endl;
    }
 
    for (i=1; i<=10; i++) // поиск мин и макс элемента х1 и х2 это номера позиций максим и миним в массиве
    {
        /* Зачем здесь вложенный цикл? И без него отработает хорошо */
        for (j=1; j<=10; j++)
        {
            if (massiv[j]>max)
            {
                max = massiv[j];
                x1=i;
            }
            if (massiv[j]<min)
            {
                min = massiv[j];
                x2=i;
            }
        }
    }
    y1 = massiv[x2+1];
    y2 = massiv[x1-1];
    /*Тут вычисляется не произведение _между_ элементами, а просто произведение 2х элементов, а именно
    следующего за минимальным и того, что перед максимальным*/
    PrzElem = y1*y2; //вычисление произведения между элементами
    cout<<" PROIZVEDENIE = "<<PrzElem<<endl;
 
    for (i=1; i<=n; i++) // сортировка массива по возрастанию
    {
        for (j=1; j<=n; j++)
        {
            if (massiv[j]>massiv[j+1])
            {
                b=massiv[j+1];
                massiv[j+1]=massiv[j];
                massiv[j]=b;
            }
        }
    }
    /* Не проверял, но по моему тут просто выведется 1 2 3 4 5 6 7 8 9 10*/
    for (i=1; i<=10; i++) // вывод массива
    {
        AnsiToOem (" Элемент №",str);
        cout<<str<<i<<endl;
    }
    /* И еще, сумму отрицательных элементов программа не считает*/
    system( "pause > nul" );
    return 0;
}