Наибольшее отрицательное значение в массиве - C++

@selevit · Регистрация: 08.10.2008

Всем доброго времени суток. У меня тут возникла небольшая загвоздка, нужно в массиве найти первый максимальный элемент среди отрицательных элементов, стоящих правее элемента, равного Т.
Вот код, но он работает немного некорректно.

C++

#include <iostream>
using namespace std;
int main()
{
    setlocale(LC_ALL,"");
    int A[20],n,T,tmp,i,j,max,i_max;
    cout<<"Введите количество элементов массива: ";
    cin>>n;
    for(int i=0;i<n;i++)
    {
        cout<<"A["<<i<<"]= ";
        cin>>A[i];
    }
    cout<<"Введите разделяющий элемент массива Т: ";
    cin>>T;
    for(i=0;i<n;i++)
        if(A[i]==T)
        {
            for(j=i+1;j<n-1;j++)
            {
                if(A[j]<0)
                    if(A[j]>A[j+1])
                {
                    A[j+1]=A[j];
                    max=A[j+1];
                    i_max=j;
                }
            }
        }
        cout<<j<<endl;
        cout<<max<<endl;
    system("pause");
    return 0;
}

Заранее спасибо =)

@neske · 08.06.2010, 19:16

CyBOSSeR, прошу прощения за такие вопросы, но все же:
Пытаюсь разобраться, почему работает ваш вариант с функцией neg_el, но т.к. я в теории слабоват, до финиша еще не дошел (;

В общем, как работает сам алгоритм поиска максимального элемента?
В функцию передается два аргумента (случайных, по порядку?), и если функция возвращает true, он запоминает максимальный из этих двух, и т.д. по аналогии. Это я так думаю.
Книги я читал, но вот этого, как я помню, не было.

@selevit · 08.06.2010, 20:56 **[ТС]**

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

C++

#include <iostream>
#include <vector>
#include <set>
#include <algorithm>
#include <functional>
#include <iterator>
 
int main(){
    int cnt, t;
    std::vector<int> vec;
 
    std::cout << "Количество элементов: ";
    std::cin >> cnt;
    while ( cnt-- ){
        std::cout << "Значение (осталось " << cnt << "): ";
        std::cin >> t;
        vec.push_back(t);
    }
    std::cout << "Значение - разделитель: ";
    std::cin >> t;
    std::vector<int>::const_iterator start = std::find(vec.begin(), vec.end(), t);
    if ( start == vec.end() ){
        std::cerr << "Значение " << t << " не найдено!" << std::endl;
        return 1;
    }
    if ( ++start == vec.end() ){
        std::cerr << "Значение " << t << " последнее в массиве!" << std::endl;
        return 1;
    }
    std::set<int, std::greater<int> > set;
    for ( std::vector<int>::const_iterator i = start; i != vec.end(); ++i )
        if ( *i < 0 )
            set.insert(*i);
 
    if ( set.empty() ){
        std::cerr << "В массиве нет отрицательных значений, идущих после " << t << std::endl;
        return 1;
    }
    std::cout << "Массив:" << std::endl;
    std::copy(vec.begin(), vec.end(), std::ostream_iterator<int>(std::cout, " "));
    std::cout << std::endl;
 
    std::cout << "Максимальный отрицательный элемент в массиве, идущий после " << t << " : "
            << *set.begin() << std::endl;
 
    return 0;
}

Кстати, если ввести все положительные элементы, то прога вылетает =)

@CyBOSSeR · 08.06.2010, 21:43

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

В функцию передается два аргумента (случайных, по порядку?)

По порядку. В качестве первого аргумента в функцию сравнения передается текущий максимальный элемент (в самом начале за максимальный берется первый), в качестве второго аргумента текущий элемент. Если функция возвращает true максимальному элементу присваивается значение текущего элемента.

@neske · 08.06.2010, 22:03

CyBOSSeR, огромное спасибо.

@easybudda · 08.06.2010, 23:04

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

Кстати, если ввести все положительные элементы, то прога вылетает =)

Бесит это слово! Что значит "вылетает"? Это компьютерная программа, а не Гагарин!

Наибольшее отрицательное значение в массиве

@selevit · 09.06.2010, 00:34 **[ТС]**

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

Это компьютерная программа, а не Гагарин!

А Вы каким компилятором пользуетесь? я недавно просто поставил VS 2010.

@easybudda · 09.06.2010, 01:04

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

А Вы каким компилятором пользуетесь?

ну вот только-что мелкомяглим cl от MSVS 2005 скомпилил

2010 качать/ставить ради того, чтобы эту програмку проверить, не буду!

@CyBOSSeR · 09.06.2010, 10:44

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

да не, вроде работает...

Работать-то работает, но если в массиве не будет ни одного отрицательного числа, max_element вернет указатель на первый элемент массива, что не есть хорошо.

Не по теме:

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

CyBOSSeR, огромное спасибо.

Не за что. Всегда рад помочь.

@easybudda · 09.06.2010, 10:47

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

Работать-то работает, но если в массиве не будет ни одного отрицательного числа, max_element вернет указатель на первый элемент массива, что не есть хорошо.

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

@neske 1519 / 886 / 89 Регистрация: 26.03.2010 Сообщений: 3,032
	08.06.2010, 19:16	#16
	CyBOSSeR, прошу прощения за такие вопросы, но все же: Пытаюсь разобраться, почему работает ваш вариант с функцией neg_el, но т.к. я в теории слабоват, до финиша еще не дошел (; В общем, как работает сам алгоритм поиска максимального элемента? В функцию передается два аргумента (случайных, по порядку?), и если функция возвращает true, он запоминает максимальный из этих двух, и т.д. по аналогии. Это я так думаю. Книги я читал, но вот этого, как я помню, не было. 0

@CyBOSSeR 2307 / 1680 / 86 Регистрация: 06.03.2009 Сообщений: 3,675
	08.06.2010, 21:43	#18
	Сообщение от neske В функцию передается два аргумента (случайных, по порядку?) По порядку. В качестве первого аргумента в функцию сравнения передается текущий максимальный элемент (в самом начале за максимальный берется первый), в качестве второго аргумента текущий элемент. Если функция возвращает true максимальному элементу присваивается значение текущего элемента. 1

@neske 1519 / 886 / 89 Регистрация: 26.03.2010 Сообщений: 3,032
	08.06.2010, 22:03	#19
	CyBOSSeR, огромное спасибо. 0

@easybudda Модератор 9958 / 5881 / 993 Регистрация: 25.07.2009 Сообщений: 11,119
	08.06.2010, 23:04	#20
	Сообщение от selevit Кстати, если ввести все положительные элементы, то прога вылетает =) Бесит это слово! Что значит "вылетает"? Это компьютерная программа, а не Гагарин! 1

@selevit 79 / 75 / 3 Регистрация: 08.10.2008 Сообщений: 296
	09.06.2010, 00:34 [ТС]	#21
	Сообщение от easybudda Это компьютерная программа, а не Гагарин! А Вы каким компилятором пользуетесь? я недавно просто поставил VS 2010. 0

@easybudda Модератор 9958 / 5881 / 993 Регистрация: 25.07.2009 Сообщений: 11,119
	09.06.2010, 01:04	#22
	Сообщение от selevit А Вы каким компилятором пользуетесь? ну вот только-что мелкомяглим cl от MSVS 2005 скомпилил 2010 качать/ставить ради того, чтобы эту програмку проверить, не буду! 0

@CyBOSSeR 2307 / 1680 / 86 Регистрация: 06.03.2009 Сообщений: 3,675
	09.06.2010, 10:44	#23
	Сообщение от easybudda да не, вроде работает... Работать-то работает, но если в массиве не будет ни одного отрицательного числа, max_element вернет указатель на первый элемент массива, что не есть хорошо. Не по теме: Сообщение от neske CyBOSSeR, огромное спасибо. Не за что. Всегда рад помочь. 0

@easybudda Модератор 9958 / 5881 / 993 Регистрация: 25.07.2009 Сообщений: 11,119
	09.06.2010, 10:47	#24
	Сообщение от CyBOSSeR Работать-то работает, но если в массиве не будет ни одного отрицательного числа, max_element вернет указатель на первый элемент массива, что не есть хорошо. Ну это да... Но решается добавлением костыля - проверять возвращаемое значение, если не отрицательное - писать, что нет отрицательных значений... 0

Опции темы

С++ для начинающих

Наибольшее отрицательное значение в массиве - C++