0

我试图编写一个简短的实用函子，它接受两个 std::pair 项目并测试它们的相等性，但忽略元素的顺序。此外（这是我遇到麻烦的地方）我编写了一个函数来获取这些 std::pair 项目的容器并测试容器中给定对参数的成员资格。

/* A quick functor way to check the identity of the two items of a pair to see if each pair contains the same items regardless of order */
template <class T>
class EqualPairs : public std::binary_function<T,T,bool> {
  T arg2;

  public:
  explicit EqualPairs (const T& x) : arg2(x) { }

  bool operator() (const T& arg1) { 
    bool same = false;
    if (arg1 == arg2 || (arg1.first == arg2.second && arg1.second == arg2.first))
      same = true;
    return same;
  }
};

/* checks to see if the give pair p is a member of the list of pairs l. The pairs are compared disregarding the order of the pair elements (i.e. (4,2) == (2,4)) */
template <class P>
bool PairListMember (const P& p, const std::vector<P>& l)
{
  std::vector<P>::iterator it;
  it = find_if (l.begin(), l.end(), EqualPairs<P>(p));
  bool member_of_list = (it != l.end()) ? true : false;
  return member_of_list;
}

我想不出一种允许通用容器选择的干净方法，所以我现在将 std::vector 硬编码为容器类型。对使容器类型通用的帮助也将不胜感激，但现在我只想让上面的内容编译和工作。我得到的错误是：

In function ‘bool PairListMember(const P&, const std::vector<P, std::allocator<_CharT> >&)’:

    error: expected `;' before ‘it’
    error: ‘it’ was not declared in this scope

In function ‘bool PairListMember(const P&, const std::vector<P, std::allocator<_CharT> >&) [with P = std::pair<int, int>]’:

    error: dependent-name ‘std::vector<P,std::allocator<_CharT> >::iterator’ is parsed as a non-type, but instantiation yields a type
    note: say ‘typename std::vector<P,std::allocator<_CharT> >::iterator’ if a type is meant

按照建议通过添加“类型名称”来更改代码只会导致以下错误：

error: no match for ‘operator=’ in ‘it = std::find_if [with _InputIterator = __gnu_cxx::__normal_iterator<const std::pair<int, int>*, std::vector<std::pair<int, int>, std::allocator<std::pair<int, int> > > >, _Predicate = EqualPairs<std::pair<int, int> >](((const std::vector<std::pair<int, int>, std::allocator<std::pair<int, int> > >*)l)->std::vector<_Tp, _Alloc>::begin [with _Tp = std::pair<int, int>, _Alloc = std::allocator<std::pair<int, int> >](), ((const std::vector<std::pair<int, int>, std::allocator<std::pair<int, int> > >*)l)->std::vector<_Tp, _Alloc>::end [with _Tp = std::pair<int, int>, _Alloc = std::allocator<std::pair<int, int> >](), EqualPairs<std::pair<int, int> >(((const std::pair<int, int>&)((const std::pair<int, int>*)p))))’

/usr/include/c++/4.2/bits/stl_iterator.h:637: note: candidates are: __gnu_cxx::__normal_iterator<std::pair<int, int>*, std::vector<std::pair<int, int>, std::allocator<std::pair<int, int> > > >& __gnu_cxx::__normal_iterator<std::pair<int, int>*, std::vector<std::pair<int, int>, std::allocator<std::pair<int, int> > > >::operator=(const __gnu_cxx::__normal_iterator<std::pair<int, int>*, std::vector<std::pair<int, int>, std::allocator<std::pair<int, int> > > >&)
4

2 回答 2

5

对于您的编译器错误，您需要使用typename关键字。

typename std::vector<P>::iterator it;

iterator是一个类型名，即它指的是 std::vector 中的嵌入类型。当您::在模板中使用运算符访问类型名时，您需要使用 typename 关键字，以便编译器知道它是类型的名称，而不是类中某些变量或函数的名称。

编辑：另外，您需要使用 a const_iterator，因为在这种情况下您的向量是 const 。

typename std::vector<P>::const_iterator it;
于 2009-11-20T23:05:46.250 回答
3

EqualPairs您的模板有几个问题。它派生自 binary_function 但实际上不是 abinary_function因为operator()只接受一个参数。您可以（并且应该）制作operator()const 因为它不会修改EqualPairs对象。

我认为你可以稍微简化一下。

template<class T>
struct EqualPairs : public std::binary_function<T, T, bool>
{
    bool operator()(const T& lhs, const T& rhs) const
    {
        return lhs == rhs || lhs.first == rhs.second && lhs.second == rhs.first;
    }
};

然后您可以使用std::bind1st(或std::bind2nd) 从您的二元函数和输入参数中创建一个谓词。此外，通过使函数成为“单线”，您实际上不需要为迭代器声明临时变量，因此获取const和typename纠正不是问题。

template <class P>
bool PairListMember (const P& p, const std::vector<P>& l)
{
    return l.end() != std::find_if(l.begin(), l.end(), std::bind1st(EqualPairs<P>(), p));
}

您可以通过将迭代器类型作为模板参数来使此模板更通用。这消除了您对std::vector.

template <class Iter>
bool PairListMember(const typename std::iterator_traits<Iter>::value_type& p, Iter first, Iter last)
{
    return last != std::find_if(first, last, std::bind1st(EqualPairs<typename std::iterator_traits<Iter>::value_type>(), p));
}
于 2009-11-21T00:03:01.707 回答