使用 std::make_move_iterator 将 std::list<std::unique_ptr>> 插入另一个时出错

Error when using std::make_move_iterator to insert a std::list<std::unique_ptr>> into another one

我一直在阅读 this answer 关于使用 std::make_move_iterator 将元素从 std::vector<std::unique_ptr<T>> 移动到另一个的内容。它完美运行:

std::vector<std::unique_ptr<int>> c1;
std::vector<std::unique_ptr<int>> c2;
c1.push_back(std::unique_ptr<int>(new int));
c2.insert(c2.end(), std::make_move_iterator(c1.begin()), std::make_move_iterator(c1.end()));

现在我尝试将其更改为使用不同的容器 (std::list):

std::list<std::unique_ptr<int>> c1;
std::list<std::unique_ptr<int>> c2;
c1.push_back(std::unique_ptr<int>(new int));
c2.insert(c2.end(), std::make_move_iterator(c1.begin()), std::make_move_iterator(c1.end()));

但编译失败:

error C2248: 'std::unique_ptr<_Ty>::unique_ptr' : cannot access private member declared in class 'std::unique_ptr<_Ty>'

但是,如果列表包含其他对象(如 std::string),它会起作用:

std::list<std::string> s1;
std::list<std::string> s2;
s2.insert(s2.end(), std::make_move_iterator(s1.begin()), std::make_move_iterator(s1.end()));

现在,如果我使用来自同一个问题的 another answer,它使用 std::back_inserterstd::move,它适用于 std::vectorstd::list

std::list<std::unique_ptr<int>> c1;
std::list<std::unique_ptr<int>> c2;
c1.push_back(std::unique_ptr<int>(new int));
std::move(c1.begin(), c1.end(), std::back_inserter(c2));

据我了解,此解决方案基本上是在向后插入第二个容器时单独移动每个项目。

我的问题是,为什么在 std::unique_ptrstd::list 上使用 std::make_move_iterator 不起作用,但它对 std::vector 起作用,或者如果列表元素是不同类型?

注:我用的是Visual Studio2010.

核心问题是 Visual Studio 2010 年 std::list::insert 的实现不支持移动语义。在内部,std::list::insert 调用一个名为 _Insert 的方法,该方法在 VS 2010 中声明如下:

void _Insert(const_iterator _Where, const _Ty& _Val)

在 VS 2017 中(未检查其他版本),它调用相同的方法但声明为:

void _Insert(_Unchecked_const_iterator _Where, _Valty&&... _Val)

正如所见,VS 2017 使用右值引用,因此在进行插入时调用 std::unique_ptr 的移动构造函数。另一方面,VS 2010 改为使用左值引用,因此在某些时候调用复制构造函数,它被声明为 std::unique_ptr 的私有,从而产生编译错误。


结论

在 VS 2010 中无法使用基于 std::make_move_iterator 的解决方案,因为 std::list::insert 是如何实现的。将 std::list<std::unique_ptr<T>> 附加到另一个的选项包括:

  • 使用back_inserter:

    std::move(c1.begin(), c1.end(), std::back_inserter(c2));
    
  • Using std::list::splice (credit to ),这非常有用,因为它可以直接与函数返回的列表一起使用:

    c2.splice(c2.end(), c1);