初始化unique_ptr会导致AN＆quot“错误：使用已删除函数”即使它s＆quot s＆quot std :: move”

Question

我正在编写代码，该代码通过看起来不好的几层传递std :: unique_ptr，但是我别无选择，但现在一直通过它。

问题在于，当我尝试将std :: unique_ptr传递给提供者类的构造函数时，我会遇到错误。在Child :: function1（）的点，child已使用inpland> impl＆amp;和std的参数进行了初始化/构造:: Unique_ptr＆lt; retriever＆gt;来自其他地方。

有人可以告诉我为什么我会遇到这个错误，以及如何获得此工作？我想知道这是否是因为继承发生了，但是除了搬家以获得这项工作之外，我还不知道该怎么办与相同的构造函数...大声笑）

// child.cpp
Child::Child(Impl& child_impl, std::unique_ptr<Retriever> child_retriever_up)
: Parent(child_impl, std::move(child_retriever_up))
{}

T::Y Child::function1() const
{
    **Provider provider(d_impl, d_retriever_up);** 
    // these d_impl and d_retriever are in Parent class.
    ...
    return Y;
}

// child.h
class Child : public Parent {
public:
    // creators
    explicit Child(Impl& child_impl, std::unique_ptr<Retriever> child_retriever_up);
    ~Child() override = default;
    // accessors
    T::Y Child::function1() const;
private:
    Child(const Child&) = delete;
    Child& operator=(const Child&) = delete;

// parent.cpp
Parent::Parent(Impl& impl, std::unique_ptr<Retriever> retriever_up)
: d_impl(impl), d_retriever_up(std::move(retriever_up))
{}

// parent.h
class Parent {
public:
    // creators
    explicit Parent(Impl& impl, std::unique_ptr<Retriever> retriever_up);
    virtual ~Parent() = default;
    // copy constructor
    Parent(const Parent& parent) 
    : d_context(parent.d_impl)
    , d_retriever_up(std::move(*parent.d_retriever_up))
    {
    }
    // data
    Impl& d_impl;
    std::unique_ptr<Retriever> d_retriever_up;
private:
    //Parent(const Parent&) = delete;
    Parent& operator=(const Parent&) = delete;

// provider.cpp
Provider::Provider(Impl& impl, std::unique_ptr<Retriever> retriever_up)
: d_impl(impl)
, d_retriever(std::move(retriever_up))
{}

// provider.h
class Provider {
public:
    Provider(Impl& context, std::unique_ptr<Retriever> retriever_up);
    //copy contstructor
    Provider(const Provider& provider) 
    : d_impl(provider.d_impl)
    , d_retriever(std::move(*provider.d_retriever))
    {
    }

private:
    Impl& d_impl;
    std::unique_ptr<Retriever> d_retriever;
}

Answer 1

在Child :: function1（）中，您正在将d_retriever_up by Value 传递给提供者构造函数。这需要制作d_retriever_up的副本，这是不可能的，因为它是unique_ptr具有delete> delete'd的复制构造函数，因此错误。

您需要使用std :: move（）将d_retriever_up移动到提供者中。您声称这样做，但您不是，至少不是在需要的任何地方。将呼叫者的unique_ptr移动到提供商 constructor的retriever_up参数是一个单独的操作，而不是将retriever_up参数移动到<<代码>提供商's d_retriever类成员。

但是，解决此问题后，您将遇到另一个问题。 child :: function1（）被标记为const，它可以访问d_retriever_up be const> const在该上下文中，因此，function1（）无法修改d_retriever_up，包括移动（因为移动unique_ptr将其固定指针设置为nullptr）。

因此，要使代码正常工作，您需要删除const并添加std :: move（）：

T::Y Child::function1()
{
    Provider provider(d_impl, std::move(d_retriever_up));
    ...
    return Y;
}

也就是说，移动d_retriever_up在您的设计中值得怀疑。您可能需要考虑使用std :: shared_ptr。