Функциональный объект

@Anastasia_Galax · Регистрация: 27.02.2021

Author24 — интернет-сервис помощи студентам

Подскажите как из этого кода сделать так, чтоб использовался функциональный объект в классе, а не обычная функция ( что в классе определялся operator () // параметры ). Задание в коде: хранение метода сортировки от большего к меньшему и за алфавитом в случае равно.

C++

#include <algorithm>
#include <iostream>
#include <string>
 
using namespace std;
 
// класс чисел
class NUMBER
{
public:
    int value{}; // значения
    string title;
};
 
// метод сортировки от большего к меньшему
bool Greater(const NUMBER& a, const NUMBER& b)
{
    if (a.value == b.value) // если значения равны
    {
        return a.title < b.title; // то первое числа меньше второго
    }
 
    return a.value > b.value; // иначе первое больше второго числа
}
 
int main()
{
    setlocale(LC_ALL, "Russian");
    cout << "==================================================" << endl;
    cout << "> Имеется четыре(4) числа: x=24, y=30, a=30, z=28. " << '\n';
    cout << endl;
    cout << "> Отсортируем числа от большего к меньшему, " << endl << "если они равны, то сортируем за алфавитом." << endl;
    cout << "==================================================" << endl;
    cout << endl;
    cout << "> Результат:" << endl;
    cout << endl;
 
    NUMBER vecNUMBER[] // массив чисел
    {
        {24, "X"}, // х = 24
        {30, "Y"}, // y = 30
        {30, "A"}, // a = 30
        {28, "Z"}, // z = 28
    };
 
    // сортировка
    sort(begin(vecNUMBER), end(vecNUMBER), Greater);
 
    // вывод результата сортировки
    for (const auto& i : vecNUMBER)
    {
        cout << "\t" << i.value << ", " << i.title << endl;
    }
    cout << endl;
 
    system("pause");
    return 0;
}

@zss · 08.10.2021, 14:48

Нужно всего лишь поменять

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

bool Greater(const NUMBER& a, const NUMBER& b)

на

C++

1	bool operator<(const NUMBER& a, const NUMBER& b)

и убрать Greater в вызове

C++

1	sort(begin(vecNUMBER), end(vecNUMBER));

@Anastasia_Galax · 08.10.2021, 15:25 **[ТС]**

А если bool operator перенести в сам класс, ничего изменять не надо?

Добавлено через 1 минуту
Я сделала так, мне выдает ошибку, подскажите в чем дело

C++

#include <algorithm>
#include <iostream>
#include <string>
 
using namespace std;
 
// класс чисел
class NUMBER
{
public:
    NUMBER(int value, string title) {
        this->value = value;
        this->title = title;
    }
 
    int value;
    string title;
 
    bool operator()(const NUMBER& a, const NUMBER& b)
    {
        if (a.value == b.value) // если значения равны
        {
            return a.title < b.title; // то первое числа меньше второго
        }
 
        return a.value > b.value; // иначе первое больше второго числа
    }
};
 
 
int main()
{
    setlocale(LC_ALL, "Russian");
    cout << "==================================================" << endl;
    cout << "> Имеется четыре(4) числа: x=24, y=30, a=30, z=28. " << '\n';
    cout << endl;
    cout << "> Отсортируем числа от большего к меньшему, " << endl << "если они равны, то сортируем за алфавитом." << endl;
    cout << "==================================================" << endl;
    cout << endl;
    cout << "> Результат:" << endl;
    cout << endl;
 
    NUMBER vecNUMBER[] // массив чисел
    {
        {24, "X"}, // х = 24
        {30, "Y"}, // y = 30
        {30, "A"}, // a = 30
        {28, "Z"}, // z = 28
    };
 
    // сортировка
    sort(begin(vecNUMBER), end(vecNUMBER));
 
    // вывод результата сортировки
    for (const auto& i : vecNUMBER)
    {
        cout << "\t" << i.value << ", " << i.title << endl;
    }
    cout << endl;
 
    system("pause");
    return 0;
}

@Annemesski · 08.10.2021, 15:46

Anastasia_Galax, Вам показали что нужно перегружать оператор "меньше", а Вы перегружаете оператор "скобки". Если правильно определить оператор при определении его как член класса, то он должен принимать 1 параметр, поскольку вторым будет сам экземпляр класса передаваемый в функцию-член неявно:

C++

/* член класса */
class Foo
{
    int a;
public:
    bool operator<(const Foo &other) { return this->a < other.a; }
};
 
/*не член класса*/
class Foo2
{
    int a;
public:
    int val(void) { return a; }
};
 
bool operator<(const Foo2 &a, const Foo2 &b)
{
    return a.val() < b.val();
}

если для функции не члена хотите доступ к закрытым членам класса без геттеров, то объявляйте как дружественную

C++

class Foo3
{
    int a;
public:
    friend bool operator<(const Foo3 &a, const Foo3 &b) { return a.a < b.a; }
};

@Anastasia_Galax · 15.10.2021, 22:33 **[ТС]**

Дело в том, что нам сказали перегружать именно "скобки" operator()

@TheCalligrapher · 15.10.2021, 23:12

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

Дело в том, что нам сказали перегружать именно "скобки" operator()

Странно, что до сих пор в ответах не сообразили, о чем идет речь. Вам нужно написать класс-компаратор, вместо функции-компаратора

C++

class Greater
{
public:
  bool operator()(const NUMBER& a, const NUMBER& b) const
  {
    return a.value != b.value ? a.value > b.value : a.title < b.title;
  }
};

и далее использовать его как

C++

1	sort(begin(vecNUMBER), end(vecNUMBER), Greater());

@Anastasia_Galax 0 / 0 / 0 Регистрация: 27.02.2021 Сообщений: 89
	15.10.2021, 22:33 [ТС]	5
	Дело в том, что нам сказали перегружать именно "скобки" operator() 0

	15.10.2021, 23:12