Произведение эл-ов массива расположенных между max и min эл.

[Chaynik]

программирую на с++, в вижл студио.
программа работает, НО...
мне нужно что б когда ibeg и iend располагались рядом, вместо произведения выдавало что они соседние, или когда они равны выводилось что все элементы одинаковые, а у меня выходи произведение, причем равное единице. в чем ошибка?

#include <iostream>
using namespace std;
int main() {
    setlocale (LC_ALL, "Russian");
    int n, s=0, k=0;
    cout <<"Введите количество элементов массива"<<endl;
    cin >> n;
    int*a=new int[n];
    cout<<"Введите элементы массива"<<endl;
    for (int i=0; i<n; i++) cin>>a[i];
    cout<<"Введенный массив"<<endl;
    for (int i=0; i<n; i++) cout <<a[i] <<"  ";
    // обработка массива
    int i, imax, imin, p=1;
    for (int i=0; i<n; i++) {if (a[i]>0) {s+=a[i], k++;}}
 
    for (i  = imax = imin = 0; i< n; i++) {
         if (abs(a[i]) > abs(a[imax])) imax = i;
         if (abs(a[i]) < abs(a[imin])) imin = i;
      }  
      cout << "\n max = " <<abs(a[imax])  <<"min = " << abs(a[imin]);
      int ibeg = imax < imin ? imax : imin;
      int iend = imax < imin ? imin : imax;
      cout << "\n ibeg = " << ibeg <<"    iend = " << iend;
      for ( i= ibeg + 1; i < iend; i++)
 
     if (iend==ibeg+1){cout<<"max и min - соседние";}
     if(iend==ibeg){cout<<"Все элементы одинаковые";}
     else if (a[i] > 0) p*=a[i];
       cout << "Произведение элементов массива расположенных между max и min по модулю эл.:  " << p<< endl;
      if (k>0) cout << "Сумма положительных элементов массива: "<<s <<endl;
        else cout << "Положительных элементов нет";
        system ("pause");
        return 0;
}

Сообщение отредактировано: Qraizer - 28.02.16, 16:52

[Qraizer]

M

Используем тег [code] для добавления текста программы в пост! Устное предупреждение! Читаем правила!

[amk]

Цитата Chaynik @ 28.02.16, 16:49

в чем ошибка?

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

Добавлено 28.02.16, 17:05
Конкретно, вывод сообщения о том, что они соседние оказался единственным оператором цикла, который ни разу не выполняется.

[Славян]

Цитата amk @ 28.02.16, 17:02

сам поймёшь

У 'нашего' Чайник'а женский пол, насколько я помню.

<можно поставить на автоудаление>

[Chaynik]

Славян а у тебя хорошая память

Добавлено 28.02.16, 17:52
amk, помоги исправить, пжл

[Славян]

Как-то так работает, кажись:

cout << "\n ibeg = " << ibeg <<"    iend = " << iend << endl;
    if( iend==ibeg+1 ) cout<<"max и min - соседние" << endl;
    else
        if( iend==ibeg ) cout << "Все элементы одинаковые" << endl;
        else
        {
            for( i=ibeg + 1; i < iend; i++)
                if( a[i]>0 ) p *= a[i];
            cout << "Произведение элементов массива расположенных между max и min по модулю эл.:  " << p<< endl;
        }
    if( k>0 ) cout << "Сумма положительных элементов массива: "<<s <<endl;
    else cout << "Положительных элементов нет";
    system ("pause");

[Chaynik]

Славян сработалоооооо благодарю

Добавлено 28.02.16, 18:05
Chaynik то есть, получается, я не туда вставила эти два условия, поэтому они не срабатывали?

Добавлено 28.02.16, 18:15
Славян то есть, получается, я не туда вставила эти два условия, поэтому они не срабатывали?

[Славян]

Цитата Chaynik @ 28.02.16, 18:04

то есть, получается, я не туда вставила эти два условия, поэтому они не срабатывали?

Ну как бы да. Понятно же, что в цикле часто проверять на 'соседнесть' как-то расточительно.

Добавлено 28.02.16, 18:31
Кстати, всё чуть неверно будет работать, если ввести все элементы отрицательные. Додумайте, как сие поправить.

[Chaynik]

Славян впредь буду внимательней
а может ты мне еще и с сортировкой поможешь?

[Chaynik]

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

Прикреплённая картинка

[Славян]

Цитата Chaynik @ 28.02.16, 18:32

а может ты мне еще и с сортировкой поможешь?

Это где строки матрицы?

Добавлено 28.02.16, 19:09

Цитата Chaynik @ 28.02.16, 18:37

почему неверно?.. по моему все так и должно быть...

Не, я вводил 6 элементов: -6, -5, -4, ..., -1.

[Chaynik]

Славян, ххмммм... надеюсь моему преподу не придет в голову ввести все отриц

Добавлено 28.02.16, 19:13
Славян, Упорядочить элементы массива по убыванию. вот условия сортировки...
препод сказал разобраться самим и сделать, т.к. мы этого не проходили... побудешь моим учителем?

[Славян]

Цитата Chaynik @ 28.02.16, 19:11

ххмммм... надеюсь моему преподу не придет в голову ввести все отриц

На препода надейся, а сама не ...

Цитата Chaynik @ 28.02.16, 19:11

препод сказал разобраться самим и сделать, т.к. мы этого не проходили...

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

[shm]

Цитата Chaynik @ 28.02.16, 19:11

Упорядочить элементы массива по убыванию

http://www.cplusplus.com/reference/algorithm/sort/

[Chaynik]

Славян, честное слово не объясняли..... а то бы сделала .. хоть начала бы...
условие написала уже "Упорядочить элементы массива по убыванию."
ладно я еще сама поищу, если не соображу ...

Добавлено 28.02.16, 19:24
shm, спасибо)