Question

请考虑以下事项。

我有两个std::set，想要按照排序顺序将它们合并到std::vector。

哪种方法最有效？

我做了类似的事情，但我认为必须有更好的方法。

std::set<int> S1;
std::set<int> S2;
// ....
// Initialization of sets
// ....

std::vector V;

std::set<int>::iterator iter;

for(iter = S1.begin(); iter!=S1.end(); ++iter)
{
    V.push_back(*iter);
}

for(iter = S2.begin(); iter!=S2.end(); ++iter)
{
    V.push_back(*iter);
}

std::sort(V.begin(), V.end());

这是我的代码，有更有效的方法吗？提前谢谢。

Answer 1

std :: merge应该可以解决问题，因为S1和S2已经排序了：

// Initialization of sets
// ....
std::vector V;

std::merge(S1.begin(), S1.end(), S2.begin(), S2.end(), std::back_inserter(V));

//而不是V.begin（） - 感谢您的纠正。

Answer 2

我相信您可以使用std::copy()：

来简化您的代码

std::copy(S1.begin(), S1.end(), std::back_inserter(V));
std::copy(S2.begin(), S2.end(), std::back_inserter(V));
std::sort(V.begin(), V.end());

Answer 3

这些集已经排序，所以最后再次对它进行重新排序似乎很浪费。

这样的事情怎么样（留下几个TODO作为读者的练习！）：

  std::set<int>iterator S1iter = S1.begin();
  std::set<int>iterator S1iter = S2.begin();

  while( S1iter != S1.end() && S2iter != S2.end() ) {

    if( S1iter == S1.end() ) {
       //TODO:  S1 is finished, so push back range S2iter->S2.end()
       //       onto the vector
       break;
    }
    else if( S2iter == S2.end() ) {
       //TODO:  S2 is finished, so push back range S1iter->S1.end()
       //       onto the vector
       break;
    }
    else if( *S1iter < *S2iter ) {
      V.push_back( *S1iter );
      ++S1iter;
    }
    else {
      V.push_back( *S2iter );
      ++S2iter;
    }

  }

Answer 4

您可以使用std :: merge（）：http://en.cppreference.com/w/cpp/algorithm/merge

它的实现类似于你的。

将std :: sets合并到std :: vector中

4 个答案: