C++ 11 unique_ptrをカスタムディテクタで使用

Question

私は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);  

ある任意のアイデア？

Answer 1

あなたは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を参照してください。

Answer 2

unique_ptrの2つの引数のコンストラクタは、スマートポインタとデリゲータではなく、生ポインタとデリータを受け取ります。

例を安全にするために、createListはおそらく最初にすべてのノードを削除するunique_ptrを返すべきです。

Answer 3

createList()は、std::unique_ptr<Node>を返します。

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

そこでエラー：constructorあなたが使用しようとしているが、最初の引数としてNode*を取ります。

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

をかstd::unique_ptr<Node, decltype(del1)>を返すためにcreateList()自体を変更する：あなたはmain()にローカルなカスタムデリータを維持したい場合は

、あなただけのcreateListの外にポインタを引っ張っする必要があります。

Answer 4

代替設計は、... `、なぜあなたは右辺値を`ヘッド（のstd ::移動（createList（））では

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; 
}