从priority_queue弹出时出现排序问题,这是 std::p riority_queue 的错误吗?

Ordering issue while popping from priority_queue, Is this a bug with std::priority_queue

本文关键字:queue riority 错误 std 问题 priority 排序 这是      更新时间:2023-10-16
#include <functional>
#include <queue>
#include <vector>
#include <iostream>
struct Temp
{
int p;
std::string str;
};
struct TempCompare
{
bool operator()(Temp const & a, Temp const & b)
{
return a.p > b.p;
}
};
int main() {
std::priority_queue<Temp, std::vector<Temp>, TempCompare> pq;
//Enable and Disable the following line to see the different output
//{Temp t; t.p=8;t.str="str1";pq.push(t);} 
{Temp t; t.p=8;t.str="str2";pq.push(t);}
{Temp t; t.p=9; t.str="str1";pq.push(t);}
{Temp t; t.p=9; t.str="str2";pq.push(t);}
while(!pq.empty())
{
std::cout << pq.top().p << " " << pq.top().str << std::endl;
pq.pop();
}
}

运行上述程序,启用和禁用主中的第四行;禁用时获得的输出是

8 str2
9 str1
9 str2

而当它启用时,你会得到

8 str1
8 str2
9 str2
9 str1

行为不应该是一致的吗?

No. 行为没有理由保持一致。 根据比较函数,Temp{9, "str1"}Temp{9,"str2"}相等,因此它们以任意顺序返回。 向队列中添加不同的元素很可能会改变该顺序。

如果您希望它们以一致的顺序返回,则需要扩展比较函数。 最简单的方法是

bool operator()(Temp const & a, Temp const & b)
{
return std::tie(a.p,a.str) > std::tie(b.p,b.str);
}

如果你想"p下降,但str上升",你必须自己做。