проблема с программой

[skorpionsej]

Добрый день, у меня к вам такой вопрос: мне нужно было
В одномерном массиве, состоящем из n вещественных элементов, вычислить:
1) количество элементов массива, лежащих в диапазоне от А до В;
2) сумму элементов массива, расположенных после максимального элемента.
Упорядочить элементы массива по убыванию модулей элементов.

#include "stdafx.h"
#include <math.h>
#include <iostream>
using namespace std;

int _tmain(int argc, _TCHAR* argv[])
{
	int massiv[50], n, S=0, A, B, N, abc, temp;
	setlocale(LC_ALL, "russian");
	cout<<"Введите количество элементов массива"<<endl;
	cin>>n;
	cout<<"Элементы массива"<<endl;
	for (int i=0; i<=n-1; i++)     // создаем массив n-х элементов
	{
		 massiv[i] = rand()%100-50; // отнимаем 50, чтобы были отрицательные числа
		 cout<<massiv[i]<<endl;
	}
	cout<<"Введите номер элемента А"<<endl;
	cin>>A;
	cout<<"Введите номер элемента B"<<endl;
	cin>>B;
	if (B>A) N=B-A-1;
	else N=A-B-1;
	cout<<"Число элементов от А до В = "<<N<<endl;    //  находим число элементов в диапазоне от А до В
	int imax = 0;
	for (int i=imax ; i<=n-1; i++)
		{
			if ( massiv[i] > massiv[imax] ) imax = i;
		}
	cout<<"Номер максимального элемента = "<<imax<<"  Его значение = "<< massiv[imax++]<<endl;
	for (int j=imax; j<=n-1; j++)
	{
		S=S+massiv[j];
	}
	cout<<"Сумма элементов после максимального = "<<S<<endl;  //  находим сумму элементов после максимального
	cout<<""<<endl;
	cout<<"Упорядоченный массив"<<endl;
	for (int i=0;i<n-1;i++)                                   // упорядочиваем массив
		for (int h=i+1; h<n; h++)
			if (massiv[h] > massiv[i])
			{
				temp = massiv[i];
				massiv[i] = massiv[h];
				massiv[h] = temp;
			}
			for (int i=0; i<n; i++)
			cout<<massiv[i]<<endl;
	system("pause");
	return 0;
}

Не уверен что правильно реализовал первый пункт. Все остальное работает вполне корректно. Подскажите, если что не правильно

[three_cats]

Поправил. И еще: там же по модулю берутся величины при сортировке.
Подключаем time.h и добавляем функцию srand(), чтобы рандомайзер работал, а то у тебя одинаковые значения все время выдавались.
Второй пункт считает правильно.

#include <math.h>
#include <iostream>
#include <time.h>
using namespace std;

int main()
{
        srand(static_cast<unsigned int> ( time(NULL)));
	int massiv[50], n, S=0, A, B, N, abc, temp;
	setlocale(LC_ALL, "russian");
	cout << "Введите количество элементов массива: ";
	cin >> n;
	cout << "Элементы массива: " << endl;
	for (int i=0; i<=n-1; i++)     // создаем массив n-х элементов
	{
		 massiv[i] = rand()%100-50; // отнимаем 50, чтобы были отрицательные числа
		 cout << massiv[i] << " ";
	}
	cout << "\nВведите номер элемента А: ";
	cin >> A;
	cout << "\nВведите номер элемента B: ";
	cin >> B;
	N = (int)fabs(B - A);
	cout << "\nЧисло элементов от А до В = " << N << endl;    //  находим число элементов в диапазоне от А до В
	int imax = 0;
	for (int i=imax ; i<=n-1; i++)
		{
			if ( massiv[i] > massiv[imax] ) imax = i;
		}
	cout << "\nНомер максимального элемента = " << imax << ", eго значение = " << massiv[imax++] << endl;
	for (int j=imax; j<=n-1; j++)
	{
		S=S+massiv[j];
	}
	cout << "Сумма элементов после максимального = " << S << endl;  //  находим сумму элементов после максимального
	cout << endl;
	cout << "Упорядоченный массив: " << endl;
	for (int i=0;i<n-1;i++)                                   // упорядочиваем массив
		for (int h=i+1; h<n; h++)
			if (fabs(massiv[h]) > fabs(massiv[i]))
			{
				temp = massiv[i];
				massiv[i] = massiv[h];
				massiv[h] = temp;
			}
			for (int i=0; i<n; i++)
			cout << massiv[i] << " ";
	system("pause");
	return 0;
}

[skorpionsej]

я так понял, что мысль на счет первого задания у меня верна?
а с модулем я что-то не дружу, компилятор ругается, в принципе он и без fabs упорядочивал хорошо.
а с time.h действительно интересней стало

[skorpionsej]

Упорядочил по модулю и сделал небольшое ветвление из-за А и В.

	if (B>A) N=B-A;
	else N=A-B;

Думаю, все что нужно реализовал?

#include "stdafx.h"
#include <math.h>
#include <iostream>
#include <time.h>
using namespace std;

int main()
{
    srand (static_cast <unsigned int> (time(NULL)));
	int massiv[50], n, S=0, A, B, N, temp;
	setlocale(LC_ALL, "russian");
	cout << "Введите количество элементов массива: ";
	cin >> n;
	cout << "Элементы массива: " << endl;
	for (int i=0; i<=n-1; i++)     // создаем массив n-х элементов
	{
		 massiv[i] = rand()%100-50; // отнимаем 50, чтобы были отрицательные числа
		 cout << massiv[i] << " ";
	}
	cout<<endl;
	cout<<"\nВведите номер элемента А: ";
	cin>>A;
	cout<<"Введите номер элемента B: ";
	cin>>B;
	if (B>A) N=B-A;
	else N=A-B;
	cout<<"\nЧисло элементов от А до В = " <<N<<endl;    //  находим число элементов в диапазоне от А до В
	int imax = 0;
	for (int i=imax ; i<=n-1; i++)
		{
			if ( massiv[i] > massiv[imax] ) imax = i;
		}
	cout<<"\nНомер максимального элемента = " <<imax<< ", eго значение = " <<massiv[imax++]<<endl;
	for (int j=imax; j<=n-1; j++)
	{
		S=S+massiv[j];
	}
	cout<<"Сумма элементов после максимального = " <<S<< endl;  //  находим сумму элементов после максимального
	cout<<endl;
	cout<<"Упорядоченный массив:"<<endl;
	for (int i=0;i<n-1;i++)                                   // упорядочиваем массив
		for (int h=i+1; h<n; h++)
			if (abs(massiv[h]) > abs(massiv[i]))
			{
				temp = massiv[i];
				massiv[i] = massiv[h];
				massiv[h] = temp;
			}
			for (int i=0; i<n; i++)
			cout<<massiv[i]<<endl;
			cout<<endl;
	system("pause");
	return 0;
}

[three_cats]

skorpionsej, да, верно все!