Boost initialisieren :: shared_ptr <std :: vector <T>> mit boost :: shared_ptr <std::list<T>>

Question

Ich möchte eine boost::shared_ptr<std::vector<std::string> > vec in der Konstruktor initialisieren Liste mit einem boost::shared_ptr<std::list<std::string> > list initialisieren?

Ist es möglich?

Ich versuchte dies:

Test.hpp

class Test 
{ 
public: 
    Test(boost::shared_ptr<std::list<std::string> > list); 

private: 
    boost::shared_ptr<std::vector<std::string> > vec; 
}; 

Test.cpp

Test::Test(boost::shared_ptr<std::list<std::string> > list) : vec(list->begin(), list->end()) 
{ 
} 

Teil der Fehlermeldung:

Test.cpp: In constructor ‘Test::Test(boost::shared_ptr<std::list<std::basic_string<char> > >)’: 
Test.cpp:6:85: error: no matching function for call to ‘boost::shared_ptr<std::vector<std::basic_string<char> > >::shared_ptr(std::list<std::basic_string<char> >::iterator, std::list<std::basic_string<char> >::iterator)’ 

Answer 1

ersetzen:

vec(list->begin(), list->end()) 

mit:

vec(boost::make_shared(list->begin(), list->end())) 

Ihr Konstruktor sollte wie folgt aussieht:

Test::Test(const boost::shared_ptr<std::list<std::string> >& list) : 
    vec(boost::make_shared(list->begin(), list->end())){ 
} 

Beachten Sie, dass Sie die Daten aus dem std::list zum std::vector kopieren.

Wenn Sie eine günstigere Lösung wünschen, können Sie sie mit std::make_move_iterator verschieben. Da Sie jedoch immer noch Smart-Pointer verwenden, habe ich keinen Zugriff darauf.

Edit:

wenn es nicht versuchen, diese funktioniert:

vec(boost::make_shared<std::vector<std::string>>(list->begin(), list->end())) 

Edit 2:

Um den nullptr Fall abzudecken, wie es von @Maxim erwähnt wurde Egorushkin:

class Test{ 
public: 
    Test(const boost::shared_ptr<std::list<std::string> >& list); 

private: 
    boost::shared_ptr<std::vector<std::string> > convert_to_vec(const boost::shared_ptr<std::list<std::string> >& lst) const; 
    boost::shared_ptr<std::vector<std::string> > vec; 
}; 

//in .cpp 
Test::Test(const boost::shared_ptr<std::list<std::string> >& list): 
    vec(convert_to_vec(list)){ 
} 
boost::shared_ptr<std::vector<std::string> > Test::convert_to_vec(const boost::shared_ptr<std::list<std::string> >& lst) const{ 
    if(lst!=nullptr){ 
     return boost::make_shared<std::vector<std::string>>(list->begin(), list->end()); 
    } 
    return nullptr; 
} 

Answer 2

Als eine Randnotiz: Es ist nicht klar, warum der Konstruktor shared_ptr<X>, aber nicht die shared_ptr<X> verwendet. Es sollte stattdessen X& dauern. Vermeiden Sie möglichst Smart-Pointer.

Idealerweise sollte der Code wie folgt aussehen:

class Test 
{ 
public: 
    Test(std::list<std::string> const& list) 
     : vec(list.begin(), list.end()) 
    {} 

private: 
    std::vector<std::string> vec; 
};