std :: unique_ptr和templates:为什么不能编译这段代码?

时间:2015-09-08 19:58:59

标签: c++ templates c++14 unique-ptr

我正在研究游戏的模板化组件系统。它将项目保存在向量中,并具有两个部分特化:一个用于POD类型,另一个用于std :: unique_ptr。 std :: unique_ptr的向量的模板化特化不会编译,但在非模板化代码中使用std :: unique_ptr的向量可以正常工作。我已经找到了解决这个问题的方法,但遗憾的是我对C ++ 11 / C ++ 14的了解并不够完善;我做错了什么?

最小示例(剥离到问题区域):

#include <vector>
#include <memory>

template<typename T>
class component 
{
    public:
    class handle
    {
    friend class component;
    protected:
        std::size_t inner;
    };

    component<T>::handle add(const T& t)
    {
        items.push_back(t);
        handle h;
        h.inner = items.size() - 1;
        return h;
    }

    protected:
        std::vector<T> items;
};


template<typename T>
class component<std::unique_ptr<T>> 
{
    public:
    class handle
    {
    friend class component;
    protected:
         std::size_t inner;
    };

    component<std::unique_ptr<T>>::handle add(const std::unique_ptr<T>& t)
    {
        items.push_back(std::move(t));
        handle h;
        h.inner = items.size() - 1;
        return h;
    }

    protected:
    std::vector<std::unique_ptr<T>> items;
};

这是测试代码

int main()
{
    // This works fine.
    component<int> pod_component;
    pod_component.add(5);

    // This works, too!
    std::vector<std::unique_ptr<int>> pointer_vector;
    pointer_vector.push_back(std::make_unique<int>(5));

    component<std::unique_ptr<int>> pointer_component;
    // Why doesn't this compile?
    pointer_component.add(std::make_unique<int>(5));

    return 0;
 }

这是gcc(4.9.3 Gentoo)的错误输出:

 error: use of deleted function ‘std::unique_ptr<_Tp, _Dp>::unique_ptr(const std::unique_ptr<_Tp, _Dp>&) [with _Tp = int; _Dp = std::default_delete<int>]’
{ ::new((void *)__p) _Up(std::forward<_Args>(__args)...); }

我希望C ++比我更好的人可以帮助我和未来的读者理解这个棘手的问题。

1 个答案:

答案 0 :(得分:4)

问题在于:

component<std::unique_ptr<T>>::handle add(const std::unique_ptr<T>& t)   
{                                 //        |
    items.push_back(std::move(t));// <------+
    // ...
应用于std::move左值的

const会返回const右值参考。因此,它受push_back(const value_type&)重载而不是push_back(value_type&&)重载的约束,这意味着它会尝试复制作为参数传递的unique_ptr

正确的声明应如下所示:

component<std::unique_ptr<T>>::handle add(std::unique_ptr<T>&& t)
//                                                         ~^^~