在std :: list中保存std :: set

Question

我是c ++编程的新手,有一些问题.

我有一套:

std::set<Proc*> finalProc = getFinalProc();

我只是想将这个集合中的所有元素复制到一个列表中.我以为我必须遍历集合并保存列表中的元素？我想我犯了一个重大错误,但我无法找到解决方案:

std::list<Proc*> firstLevel;
for(std::set<Processor*>::iterator it = endProcessors.begin(); it != endProcessors.end(); ++it){
    firstLevel.push_back( ???? );
}

想法是push_back所有迭代元素？

谢谢

Answer 1

有一个更好的方法来做到这一点:

std::list<Proc*> firstLevel(finalProc.begin(), finalProc.end());

在你原来的代码,你应该已经取代????与*it和endProcessors与finalProc

正如@ luk32所说,可能你有一个错误(如果你想复制实际数据,而不是指针).在这种情况下,您可以看到@ luk32解决方案,或者我会去std::unique_ptr这里(智能指针是在现代C++中管理内存的首选方式):

std::list<std::unique_ptr<Proc>> firstLevel;
for(const auto& ptr: finalProc){
    firstLevel.push_back(std::make_unique<Proc>(*ptr));
}

超出firstLevel可见范围后,firstLevel中指针指向的所有内存将自动释放.