C ++优先级队列按特定方法按升序排列，以获取指向对象的唯一指针

Question

我有一个名为Foo 的类，每个Foo 对象都有一个名为yVal 的方法。我想要的是Foo对象的优先级队列，按Foo.yVal()的升序排列

我在Foo 中重载了 operator> 和 operator

   bool operator> (const Foo &f){
        return yVal() > f.yVal();
   }

   bool operator< (const Foo &f){
        return yVal() < f.yVal();
   }

所以我有以下代码：

priority_queue<unique_ptr<Foo>, vector<unique_ptr<Foo>>, greater<unique_ptr<Foo>> > Queue;

但这并不是按照Foo.yVal() 的升序对优先级队列进行排序。相反，它只是以某种未知的随机顺序对它们进行排序。我在 operator> 和 operator

auto cmp = [](Foo left, Foo right) {return left.xVal() > right.xVal();};
priority_queue<unique_ptr<Foo>, vector<unique_ptr<Foo>>, decltype(cmp) > Queue(cmp);

但这给了我“非静态数据成员声明为自动”的错误。

理想情况下，我希望通过更大的 和运算符重载来实现这一点。如果不能，请告诉我我在 lambda 上做错了什么（对 lambda 不太熟悉，如果可能的话希望避免使用它们）。

我也尝试过使用仿函数，但同样，仿函数上的 auto 给了我同样的“非静态数据成员声明为 auto”的错误。

Answer 1

你得到了未知的随机顺序，因为当你有greater<T> 执行以下操作

// pseudocode
cmp(T lhs, T rhs) {
  return lhs > rhs;
}

您的代码中的T 是什么？ T 是 unique_ptr<Foo>，C++ 库中有 operator>(unique_ptr<>,unique_ptr<>) 吗？ 没错，就是see here，这个算子在比较的时候使用unique_ptr::get方法：

// Psuedocode when greater used:
cmp (unique_ptr<Foo>& lhs, unique_ptr<Foo>& rhs)
{
  lhs.get () > rhs.get()
}

unique_ptr::get 返回什么？它返回指向Foo的指针， 所以您正在比较指向 Foo 实例的指针。结果是不可预测的。 代码可以编译并执行，但并没有达到您的预期。

---

如何修复你的 lambda：

您的队列中存储了哪些对象？ Foo 或 unique_ptr<Foo> ？ 您正在存储 unique_ptr<Foo>，因此您的 lambda 参数应该声明为采用这种类型。

auto cmp = [](const unique_ptr<Foo>& left, const unique_ptr<Foo>& right) 
{                   ^^^^^^^^^^^^^^^^
    return left->yVal() > right->yVal();
};             ^^
priority_queue<unique_ptr<Foo>, vector<unique_ptr<Foo>>, decltype(cmp) > Queue(cmp);

因为unique_ptr 的实例无法复制，您必须通过引用传递它们。 也可以使用-> 运算符来访问yVal 方法。

---

编辑：带有函数对象的版本。

 // comparator as function object with overloaded operator()
 struct Cmp {
   bool operator()(const std::unique_ptr<Foo>& left, const std::unique_ptr<Foo>& right) const {
     return left->xVal() > right->xVal();
   }
 };

class YourClass {
public:
   std::priority_queue<std::unique_ptr<Foo>, std::vector<std::unique_ptr<Foo>>, Cmp> Queue;