Использование собственного компаратора std :: set - PullRequest
80 голосов
/ 12 апреля 2010

Я пытаюсь изменить порядок по умолчанию элементов в наборе целых чисел, чтобы они были лексикографическими, а не числовыми, и я не могу получить следующее для компиляции с g ++:

file.cpp:

bool lex_compare(const int64_t &a, const int64_t &b) 
{
    stringstream s1,s2;
    s1 << a;
    s2 << b;
    return s1.str() < s2.str();
}

void foo()
{
    set<int64_t, lex_compare> s;
    s.insert(1);
    ...
}

Я получаю следующую ошибку:

error: type/value mismatch at argument 2 in template parameter list for ‘template<class _Key, class _Compare, class _Alloc> class std::set’
error:   expected a type, got ‘lex_compare’

что я делаю не так?

Ответы [ 5 ]

133 голосов
/ 12 апреля 2010

Вы используете функцию, где по мере необходимости следует использовать функтор (класс, который перегружает оператор (), поэтому его можно вызывать как функцию).

struct lex_compare {
    bool operator() (const int64_t& lhs, const int64_t& rhs) const {
        stringstream s1, s2;
        s1 << lhs;
        s2 << rhs;
        return s1.str() < s2.str();
    }
};

Затем вы используете имя класса в качестве параметра типа

set<int64_t, lex_compare> s;

Если вы хотите избежать шаблонного кода функтора, вы также можете использовать указатель на функцию (при условии, что lex_compare - это функция).

set<int64_t, bool(*)(const int64_t& lhs, const int64_t& rhs)> s(&lex_compare);
42 голосов
/ 09 сентября 2017

1. Современное решение C ++ 11

auto cmp = [](int a, int b) { return ... };
std::set<int, decltype(cmp)> s(cmp);

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

Демо онлайн

2. Аналогично первому решению, но с функцией вместо лямбды

Сделать компаратор как обычную булеву функцию

bool cmp(int a, int b) {
    return ...;
}

Тогда используйте это

std::set<int, decltype(&cmp)> s(&cmp);

Демоверсия

3. Старое решение с использованием структуры с оператором ()

struct cmp {
    bool operator() (int a, int b) const {
        return ...
    }
};

// ...
// later
std::set<int, cmp> s;

Демоверсия

4. Альтернативное решение: создать структуру из логической функции

Взять булеву функцию

bool cmp(int a, int b) {
    return ...;
}

И сделать из него структуру, используя std::integral_constant

#include <type_traits>
using Cmp = std::integral_constant<decltype(&cmp), &cmp>;

Наконец, используйте структуру в качестве компаратора

std::set<X, Cmp> set;

Демоверсия

16 голосов
/ 12 апреля 2010

Ответ Якоби вдохновляет меня на написание адаптера для инкапсуляции функторной матрицы.

template< class T, bool (*comp)( T const &, T const & ) >
class set_funcomp {
    struct ftor {
        bool operator()( T const &l, T const &r )
            { return comp( l, r ); }
    };
public:
    typedef std::set< T, ftor > t;
};

// usage

bool my_comparison( foo const &l, foo const &r );
set_funcomp< foo, my_comparison >::t boo; // just the way you want it!

Ух ты, думаю, это стоило того!

6 голосов
/ 03 апреля 2017

Вы можете использовать компаратор функций, не оборачивая его так:

bool comparator(const MyType &lhs, const MyType &rhs)
{
    return [...];
}

std::set<MyType, bool(*)(const MyType&, const MyType&)> mySet(&comparator);

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

0 голосов

std::less<> при использовании пользовательских классов с operator<

Если вы имеете дело с набором своего пользовательского класса, для которого определено operator<, то вы можете просто использовать std::less<>, например:

Как уже упоминалось в http://en.cppreference.com/w/cpp/container/set/find C ++ 14 добавил два новых find API:

template< class K > iterator find( const K& x );
template< class K > const_iterator find( const K& x ) const;

, которые позволяют вам:

#include <cassert>
#include <set>

class Point {
    public:
        // Note that there is _no_ conversion constructor,
        // everything is done at the template level without
        // intermediate object creation.
        //Point(int x) : x(x) {}
        Point(int x, int y) : x(x), y(y) {}
        int x;
        int y;
};
bool operator<(const Point& c, int x) { return c.x < x; }
bool operator<(int x, const Point& c) { return x < c.x; }
bool operator<(const Point& c, const Point& d) {
    return c.x < d;
}

int main() {
    std::set<Point, std::less<>> s;
    s.insert(Point(1, -1));
    s.insert(Point(2, -2));
    s.insert(Point(0,  0));
    s.insert(Point(3, -3));
    assert(s.find(0)->y ==  0);
    assert(s.find(1)->y == -1);
    assert(s.find(2)->y == -2);
    assert(s.find(3)->y == -3);
    // Ignore 1234, find 1.
    assert(s.find(Point(1, 1234))->y == -1);
}

Протестировано на Ubuntu 16.10, g++ 6.2.0, с:

g++ -std=c++14 -Wall -Wextra -pedantic -o main.out main.cpp
./main.out

Более подробную информацию о std::less<> можно найти по адресу: Что такое прозрачные компараторы?

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...