我正在尝试通过执行简单的链表程序来学习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);
有什么想法吗?
答案 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>
。您尝试使用的constructor将Node*
作为第一个参数:
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
,它将首先删除所有节点。