Question

我有这样的代码：

#include <memory>
#include <map>

struct element {
  std::map<std::weak_ptr<int>, int> weights;
  bool operator<(const element &a) const { return this->weights < a.weights; }
};

int main() { return 0; }

我想比较这个类的两个实例，但是我遇到了编译器错误：

/usr/include/c++/4.8/bits/stl_pair.h: In instantiation of ‘constexpr bool std::operator<(const std::pair<_T1, _T2>&, const std::pair<_T1, _T2>&) [with _T1 = const std::weak_ptr<int>; _T2 = int]’:
/usr/include/c++/4.8/bits/stl_pair.h:221:24: error: no match for ‘operator<’ (operand types are ‘const std::weak_ptr<int>’ and ‘const std::weak_ptr<int>’)
     { return __x.first < __y.first
/usr/include/c++/4.8/bits/stl_pair.h:222:23: error: no match for ‘operator<’ (operand types are ‘const std::weak_ptr<int>’ and ‘const std::weak_ptr<int>’)
       || (!(__y.first < __x.first) && __x.second < __y.second); }
/usr/include/c++/4.8/bits/stl_pair.h:222:65: error: body of constexpr function ‘constexpr bool std::operator<(const std::pair<_T1, _T2>&, const std::pair<_T1, _T2>&) [with _T1 = const std::weak_ptr<int>; _T2 = int]’ not a return-statement
       || (!(__y.first < __x.first) && __x.second < __y.second); }

看到no match for operator，我添加了以下代码，但它没有帮助。

// from boost::weak_ptr
template<typename T, typename U>
bool operator<(const std::weak_ptr<T> &a, const std::weak_ptr<U> &b)
{
  return a.owner_before(b);
}

当我尝试以下操作时，错误仍然存在：

将constexpr添加到此运算符中的任何一个;
向地图添加自定义比较器，如下所示：std::map<std::weak_ptr<int>, int, std::owner_less<std::weak_ptr<int>>>。

我可以通过以下方式编译此代码：

将运算符返回语句替换为：return true;
将权重成员的类型更改为不使用std::weak_ptr，例如到std::map<int, int>;
将自定义比较运算符添加到不比较地图的类元素，但比较每个键和值。

选项1.和2.只是测试而不是选项; 3.有可能，但我想了解为什么我会收到此错误并尽可能使用标准库。根据我的理解，它应该编译：{{1}}有一个std::map，它比较内部树，它应该比较operator<，它比较对中的第一个和第二个元素，它应该至少起作用如果提供了pairs<key, data>的{{1}}。

但它不起作用（至少不适用于g ++ 4.8。{1,2}），因此我的问题：

为什么它不起作用，为什么我会收到此错误消息？
如何将两个operator<与weak_ptr作为关键字进行比较？

更新，使用KerrekSB建议的maps。

我正在尝试比较两个不同的地图。在下面的示例中，两个映射m1和m2都具有相同的键，但使用此键存储不同的值。如果比较这两个地图，它们不应该相等，应该先排在另一个之前。

weak_ptr

显示的输出表明两者相等：

std::lexicographical_compare

但它们不是，并且通过使用评论的比较，结果变为：

#include <memory>
#include <map>
#include <iostream>

typedef std::owner_less<std::weak_ptr<int>> wp_less;
typedef std::map<std::weak_ptr<int>, int, wp_less> wp_map;
bool map_cmp(const wp_map &a, const wp_map &b)
{
  return std::lexicographical_compare(
    a.begin(), a.end(),
    b.begin(), b.end(),
    []( std::pair<std::weak_ptr<int> const, int> const & p, 
        std::pair<std::weak_ptr<int> const, int> const & q) -> bool 
      { return wp_less()(p.first, q.first); });
      //{ return wp_less()(p.first, q.first) 
      //    || ( ! (wp_less()(q.first, p.first)) && p.second < q.second); });
}

int main() 
{
  std::shared_ptr<int> sp_int(std::make_shared<int>(5));
  std::weak_ptr<int> wp_int(sp_int);
  wp_map m1, m2;
  m1[wp_int] = 1;
  m2[wp_int] = 2;
  std::cout << "m1 < m2=" << map_cmp(m1, m2) << "\nm2 < m1=" << map_cmp(m2, m1);
  return 0; 
}

所以这让我失望：

我需要做些什么来使默认的词典比较做我想要的比较这些对？
从问题的原始部分开始，为什么我会收到此错误，尤其是导致constexpr错误的原因？

Answer 1

不要使用裸<，而是使用std::lexicographical_compare算法，您可以使用自定义谓词（例如std::owner_less）提供该算法。

裸<运算符使用默认版本的词典比较与std::less谓词，这对弱指针不起作用。

它有点满口，所以让我为你拼出一个例子：

std::map<std::weak_ptr<int>, int, std::owner_less<std::weak_ptr<int>>> a, b;

return std::lexicographical_compare(
    a.begin(), a.end(),
    b.begin(), b.end(),
    [](std::pair<std::weak_ptr<int> const, int> const & p,
       std::pair<std::weak_ptr<int> const, int> const & q)
    -> bool { return std::owner_less<std::weak_ptr<int>>()(p.first, q.first); });

在此示例中，表达式a < b与：

相同

std::lexicographical_compare(a.begin(), a.end(), b.begin(), b.end())

这不起作用，因为这会尝试比较对，而这又会尝试将弱指针与std::less<std::weak_ptr<int>>()进行比较。（问题当然是算法使用迭代器并且不知道相应容器中的比较器对象。一般来说，没有理由为什么两个映射具有相同的容器值类型甚至应该使用相同的比较器。）

如果你愿意的话，你可以用owner_before来写类似的东西。 std::owner_less的美妙之处在于它在同一个版块中比较了弱和共享指针。

如何将两个std :: maps与std :: weak_ptr进行比较？

1 个答案: