Одномерный массив в с++

[pro100oper]

Доброго время суток!
Есть проблемка. В коде, найденном на этом форуме, есть непонятная часть:

#include "stdafx.h"
#include <conio.h>
#include <iostream>
#include <math.h>
#include <windows.h>

using namespace std;

int main()
{
const int n=5;
int m[n];
int i;
cout<<"VVEDITE "<<n; cout<<" celih cheisel\n"<<endl;
for (i=0;i<n;i++) cin >>m[i];
for (i=0;i<n;i++) cout<<m[i]<<' ';
//номер минимального элемента массива
int min=m[0];
int nom_min=0;
for (i = 1; i < n; i++)
        {
        if (m[i] < min){
                min = m[i];
                nom_min = i+1;
                };
        };
cout<<"\nNomer min elementa"<<nom_min; cout<<";";
 
//отрицательные элементы номера
int n1 = 0, n2 = 0;
 
//определение номера первого отрицательного элемента
for (i = 0; i < n; i++) {
    if (m[i] < 0) {
        n1 = i;
        break;
    }
}
 
//определение номера второго отрицательного элемента
for (i = n1+1; i < n; i++) {
    if (m[i] < 0) {
        n2 = i;
        break;
    }
}

//проверка на существование границ отрицательных элементов
if ((n2 != 0) && (n1 != n2)) {
    int sum_mezhdu = 0;
    //нахождение суммы элементов между первым и последним отрицательным элементом
    for (i = n1+1; i < n2; i++) {
        sum_mezhdu += m[i];
    };
    cout<<"\nSumma mezhdu pervim i vtorim otriczatelnim elementom"<<sum_mezhdu;
} else 
    cout<<"\nERROR";
 
//упорядочение элементов массива по модулю больших единицы
int k, pr_per=0;
double mFoo;
for (i = 0; i < n; i++) {
    if ( fabs(m[i]) <= 1 ) {
        mFoo = m[i];
        for (k = i; k > pr_per; k-- ) {
            m[k] = m[k-1];
        }
        m[pr_per++] = mFoo;
    }
}

cout<<"\n massiv posle preobrazheniya"<<endl;
        
//вывод массива после преобразования
for (i = 0; i <= n; i++)
        {
        cout<< m[i];
        };
 
_getch();
return 0;
}

упорядочение элементов массива по модулю больших единицы - здесь массивы выводится в делается в упорядоченном виде, но после массива появляется еще некое число. Как это число можно убрать?

спасибо!

[_Bers]

Рассмотрим код:

   double mFoo;
    for (i = 0; i < n; i++) {
        if ( fabs( m[i]) <= 1 ) { //: error C2668: fabs: неоднозначный вызов перегруженной функции
            mFoo = m[i];
            for (k = i; k > pr_per; k-- ) {
                m[k] = m[k-1];
            }
            m[pr_per++] = mFoo; //: warning C4244: =: преобразование 'double' в 'int', возможна потеря данных
        }
    }

Итого: 1 ошибка компиляции , 1 мина замедленного действия

Далее ответ на твой вопрос:

//вывод массива после преобразования
for (i = 0; i <= n; i++)  //<---- условие прерывания цикла. Какой индекс имеет последний элемент массива?
        {
        cout<< m[i];
        };

Выход за пределы диапазона. Если в массиве 5 элементов, то последний элемент имеет индекс 4. Посмотри внимательно на условие прерывание цикла.

Резюмируя: когда ты писал код, ты думал о номерах элементов. А должен был думать об индексах. Индексы начинаются с нуля.

[pro100oper]

А можно по подробней, как исправить это в программе?

[Glaciuse]

//вывод массива после преобразования

for (i = 0; i < n; i++)
{
     cout<< m[i];
};

[_Bers]

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

А можно по подробней, как исправить это в программе?

http://www.programmersforum.ru/showthread.php?t=34061

[pro100oper]

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

//вывод массива после преобразования

for (i = 0; i < n; i++)
{
     cout<< m[i];
};

спасибо! но я уже позже сам допер)))