C ++ 11使用unique_ptr和自定义删除器

时间:2016-07-20 16:41:11

标签: c++ c++11 std smart-pointers

我正在尝试通过执行简单的链表程序来学习C ++ 11 unique_ptr用法。对于我的生活,我无法弄清楚为什么我在使用自定义删除器时遇到编译错误。

#include <cstdio>
#include <limits>
#include <memory>
#include <cstdlib>
#include <iostream>

using namespace std;

struct node {
    int value;
    struct node* next;
};

typedef struct node Node;

std::unique_ptr<Node> createList()
{

    std::unique_ptr<Node> head(new Node);
    Node* temp=head.get();
    temp->value=0;
    for(int i=1;i<8;i++) {
        if(temp->next==nullptr) {
            temp->next=new Node();
            temp=temp->next;
            temp->value=i;
            temp->next=nullptr;
        }
    //temp=temp->next;
    }
    return head;
}

int main()
{
    auto del1 = [](Node* p) { while(p) {std::cout << "Deleting value is : " << p->value;struct node* n=p->next;delete p; p=n;} return; };

    std::unique_ptr< Node, decltype(del1) > head(std::move(createList()),del1);
}

这是编译错误

sh-4.3$ g++ -std=c++11 -o main *.cpp                                                                                                                   
main.cpp: In function 'int main()':                                                                                                                    
main.cpp:38:82: error: no matching function for call to 'std::unique_ptr<node, main()::<lambda(Node*)> >::unique_ptr(std::remove_reference<std::unique_
ptr<node> >::type, main()::<lambda(Node*)>&)'                                                                                                          
         std::unique_ptr< Node, decltype(del1) > head(std::move(createList()),del1);                                                                   
                                                                                  ^                                                                    
In file included from /usr/include/c++/5.3.1/memory:81:0,                                                                                              
                 from main.cpp:3:                                                                                                                      
/usr/include/c++/5.3.1/bits/unique_ptr.h:228:2: note: candidate: template<class _Up, class> std::unique_ptr<_Tp, _Dp>::unique_ptr(std::auto_ptr<_Up>&&)
  unique_ptr(auto_ptr<_Up>&& __u) noexcept;                                                                                                            
  ^                                                                                                                                                    
/usr/include/c++/5.3.1/bits/unique_ptr.h:228:2: note:   template argument deduction/substitution failed:                                               
main.cpp:38:82: note:   'std::remove_reference<std::unique_ptr<node> >::type {aka std::unique_ptr<node>}' is not derived from 'std::auto_ptr<_Up>'     
         std::unique_ptr< Node, decltype(del1) > head(std::move(createList()),del1);                                                                   
                                                                                  ^                                                                    
In file included from /usr/include/c++/5.3.1/memory:81:0,                                                                                              
                 from main.cpp:3:                                                                                                                      
/usr/include/c++/5.3.1/bits/unique_ptr.h:220:2: note: candidate: template<class _Up, class _Ep, class> std::unique_ptr<_Tp, _Dp>::unique_ptr(std::uniqu
e_ptr<_Up, _Ep>&&)                                                                                                                                     
  unique_ptr(unique_ptr<_Up, _Ep>&& __u) noexcept                                                                                                      
  ^                                                                                                                                                    
/usr/include/c++/5.3.1/bits/unique_ptr.h:220:2: note:   template argument deduction/substitution failed:                                               
main.cpp:38:82: note:   candidate expects 1 argument, 2 provided                                                                                       
         std::unique_ptr< Node, decltype(del1) > head(std::move(createList()),del1);    

有什么想法吗?

4 个答案:

答案 0 :(得分:4)

您应该从createList返回正确的类型:

#include <cstdio>
#include <limits>
#include <memory>
#include <cstdlib>
#include <iostream>

using namespace std;

struct node {
    int value;
    struct node* next;
};

typedef struct node Node;

auto createList()
{
    auto del1 = [](Node* p) { while(p) {std::cout << "Deleting value is : " << p->value;struct node* n=p->next;delete p; p=n;} return; };
    std::unique_ptr< Node, decltype(del1) > head(new Node,del1);

    Node* temp=head.get();
    temp->value=0;
    for(int i=1;i<8;i++) {
        if(temp->next==nullptr) {
            temp->next=new Node();
            temp=temp->next;
            temp->value=i;
            temp->next=nullptr;
        }
    //temp=temp->next;
    }
    return head;
}

int main()
{
    auto node = createList();
 }

否则,在问题中显示的代码中,您应该获取内部数据的所有权并移动它们,作为不同类型的指针:

int main()
{
    auto del1 = [](Node* p) { while(p) {std::cout << "Deleting value is : " << p->value;struct node* n=p->next;delete p; p=n;} return; };

    std::unique_ptr< Node, decltype(del1) > head(createList().release(),del1);
}

请注意.release()的来电 有关详细信息,请参阅here

答案 1 :(得分:2)

createList()返回std::unique_ptr<Node>。您尝试使用的constructorNode*作为第一个参数:

unique_ptr( pointer p, /* see below */ d1 ); (3)    
unique_ptr( pointer p, /* see below */ d2 ); (4)    

因此错误。

如果您想将自定义删除工具保持为main()的本地删除,则必须将指针从createList中拉出来:

std::unique_ptr<Node, decltype(del1)> head(
    createList().release(), // NB release(), not get()!
    del1);

或更改createList()本身以返回std::unique_ptr<Node, decltype(del1)>

答案 2 :(得分:2)

另一种设计是

struct Node {
    Node(int value) : value(value) {}

    int value;
    std::unique_ptr<Node> next;
};

std::unique_ptr<Node> createList()
{
    std::unique_ptr<Node> head = std::make_unique<Node>(0);

    Node* node = head.get();
    for(int i = 1; i < 8; i++) {
        node->next = std::make_unique<Node>(i);
        node = node->next.get();
    }
    return head;
}

答案 3 :(得分:1)

unique_ptr的双参数构造函数接受原始指针和删除器,而不是智能指针和删除器。

为了使您的示例安全,createList应该返回一个unique_ptr,它将首先删除所有节点。