移动std :: unique_ptr时出错

Question

我尝试运行以下代码以将unique_ptr的值从第一个向量移动到副本构造函数中的另一个向量：

class Text
{
    struct paragraph
    {
        int index;
        string text;
    };

    vector<unique_ptr<paragraph>> paragraphs; 

public:

    Text()
    {
        paragraphs.push_back(unique_ptr<paragraph>(new paragraph));
    }

    Text(const Text & t)
    {
        for(int i = 0; i < (int)t.paragraphs.size(); i++)
        {
            paragraphs.push_back(move(t.paragraphs[i]));
        }
    }
};

我得到了这个错误：

1>c:\program files\microsoft visual studio 10.0\vc\include\xmemory(208): error C2248: 'std::unique_ptr<_Ty>::unique_ptr' : cannot access private member declared in class 'std::unique_ptr<_Ty>'
1>          with
1>          [
1>              _Ty=Text::paragraph
1>          ]
// Etc.

Answer 1

您的代码有几处错误。

首先，您不能从const&移开，这也适用于const&成员。 运动具有破坏性； 您只能从&&移开。 你的复制构造器应该复制 ; 如果您不想要复制构造函数，则= delete它或编译器允许的任何形式。

其次，假设您为适当的move构造函数进行了Test && ，则不应该这样移动每个元素。 而是将向量移到新的向量中，如下所示：

Text(Text && t) : paragraphs(std::move(t.paragraphs)) {}

第三，您只应在假设您选择的编译器不支持自动生成的move构造函数（即Visual Studio）的情况下编写此函数。 如果它支持它，那么您根本不应该编写一个 。 让编译器完成任务。

Answer 2

Text(const Text & t)
{
    for(int i = 0; i < (int)t.paragraphs.size(); i++)
    {
        paragraphs.push_back(move(t.paragraphs[i]));
    }
}

在此构造函数中， t为const，因此t.paragraphs[i]给出了对unique_ptr的const左值引用。

move(t.paragraphs[i])将其转换为const rvalue引用，但仍为const。 unique_ptr move构造函数需要一个非常量右值引用，因此不可行，因此删除的副本构造函数是最佳匹配。 您不应该尝试在副本构造函数中移动t的内容，这就是move构造函数的作用。

您还应该说std::move不只是move ，以防止ADL找到错误的动作。