为什么在声明 std::set 时需要重复排序子程序？

Question

在我的 C++ 程序中，我尝试按值而非键对地图进行排序。

从this question 看来，这样做的方法似乎很清楚，即创建一个集合，其元素是对的，并且按我自己的小于函数排序。

这是我尝试执行此操作的一些示例代码：

#include <map>
#include <set>
#include <iostream>
#include <string>

using namespace std;

bool compareCounts(const pair<string, size_t> &lhs, const pair<string, size_t> &rhs);

int main (int argc, char *argv[]) {
        map <string, size_t> counter = { {"A", 1}, {"B", 2}, {"C", 3} };
        set <pair<string, size_t>, decltype(compareCounts) *> sorted_counter;
        for (map<string, size_t>::iterator it = counter.begin(); it != counter.end(); ++it) {
                cout << "About to add: " << it->first << ":" << it->second << endl;
                auto ret = sorted_counter.insert(*it);
                if (! ret.second) {
                        cout << "ERROR adding this element!" << endl;
                } else {
                        cout << "Element added ok" << endl;
                }
                cout << "Set is of size: " << sorted_counter.size() << endl;
        }

        return 0;
}

bool compareCounts(const pair<string, size_t> &lhs, const pair<string, size_t> &rhs) {
        return lhs.second > rhs.second;
}

这是输出：

即将添加：A:1
元素添加好了
套装尺寸：1
即将添加：B:2
分段错误：11

我注意到，当我添加第二个元素时，事情就崩溃了。我发现这种情况正在发生，因为现在需要调用我的排序子例程compareCounts。

解决方法是更改​​此行：

set <pair<string, size_t>, decltype(compareCounts) *> sorted_counter;

到这里：

set <pair<string, size_t>, decltype(compareCounts) *> sorted_counter(compareCounts);

为什么我需要指定排序子程序compareCounts 两次？编译器不是已经从我的类型定义中知道了吗？

Answer 1

set <pair<string, size_t>, decltype(compareCounts) *> sorted_counter;

您从未指定set 应该实际使用的比较器。将上面的行改为

set <pair<string, size_t>, decltype(compareCounts) *> sorted_counter(compareCounts);

如果没有指定比较器，set 默认构造一个 (nullptr)，当它尝试使用比较器插入第二个元素时，您的代码会崩溃。

你应该只使用仿函数而不是函数指针

struct compareCounts
{
    bool operator()(const pair<string, size_t> &lhs, 
                    const pair<string, size_t> &rhs) const
    {
        return lhs.second > rhs.second;
    }
};

set <pair<string, size_t>, compareCounts> sorted_counter;