Question

我需要在std::vector中旋转元素，以便它在第一个重复元素的开头保留。为了更清楚，如果我有：

1 2 3 4 5 6 3 7 8

然后我想：

3 4 5 6 3 7 8 1 2

好的，让我们使用std::rotate()，并构建一个函数来获得第一个重复的位置：

int main() {
   std::vector<int> v = { 1,2,3,4,5,6,3,7,8 };
   auto it = FindFirstDuplicate(v);
   cout << "first dupe is " << *it << " at " << it - v.begin() << endl;
   std::rotate( v.begin(), it, v.end() );
}

该任务的想法是逐个存储新向量中的元素，直到我发现我要添加的元素已经存在于这个新向量中。

但是我偶然发现了迭代器问题，因此无法编译。我对不同迭代器的类型以及const / non const迭代器问题感到困惑。

以下是我的代码，try it online here（为了便于阅读，跳过了std::）：

template<typename T>
typename vector<T>::iterator FindFirstDuplicate( const vector<T>& v )
{
   vector<T> nv; // new temp vector
   nv.reserve( v.size() );
   for( auto it1 = v.cbegin(); it1 != v.cend(); it1++ )
   {
      auto it2 = find( nv.begin(), nv.end(), *it1 ); // search elem in new vector
      if( it2 == nv.end() )                          // if not present,
         nv.push_back( *it1 );                       //   add in new vector
      else                                           // else, we found a dupe
         return v.begin() + (it2-nv.begin());
   }
   return v.begin();
}

为了保存你的眼睛，我不发布错误信息，但对我来说似乎编译器抱怨最后一行上不同类型的迭代器。我也试过使用非{const}迭代器来it1，但我仍然遇到了这个问题。

在此迭代器问题上有任何帮助，但也有任何关于算法的建议。

Answer 1

问题是您尝试从iterator引用返回const vector。您可以在此最小示例中看到问题：

#include <iostream>
#include <vector>

template <typename T>
typename std::vector<T>::iterator foo(const std::vector<T>& v)
{
    return v.begin(); // const overload returns const_iterator
}

int main() {
    std::vector<int> v(42);
    foo(v); // Instantiate function template: ERROR
}

您可以做的是将非const引用传递给您的finder函数：

template<typename T>
typename vector<T>::iterator FindFirstDuplicate( vector<T>& v )
{
   ...

Answer 2

为了记录，并且根据@James Kanze的建议（信用证给他），这是一个更短，更正和完整的版本。这个也返回找到的副本的位置。

template<typename T>
typename vector<T>::iterator
FindFirstDuplicate( vector<T>& v, typename vector<T>::iterator& dupe )
{
   auto current = v.begin();
   while (
      current != v.end() &&
      (dupe = std::find( std::next( current ), v.end(), *current )) == v.end()
   )
   {
       ++current;
   }
   return current;
}

// test case
int main()
{   
   vector<int> v = { 1,2,3,4,5,6,5,4,3 };
   auto dupe = v.begin();
   auto it = FindFirstDuplicate(v,dupe);
   cout << "first dupe is " << *it << " at pos " << it - v.cbegin() << endl;
   cout << "dupe is " << *dupe << " at pos " << dupe - v.cbegin() << endl;
}

可用here进行试验。为了完整，我使用的版本实际上返回两个迭代器的std::pair。

C ++：在迭代器之间计算距离上的类型混淆

2 个答案: