初始化boost :: shared_ptr <std::vector<T> &gt;使用boost :: shared_ptr <std::list<T> &gt;

Question

我想用boost::shared_ptr<std::list<std::string> > list在构造函数初始化列表中初始化boost::shared_ptr<std::vector<std::string> > vec ？

可能吗？

我试过这个：

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())
{
}

部分错误消息：

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

更换：

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

有：

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

您的构造函数应如下所示：

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

请注意，您正在将数据从std::list复制到std::vector 。

如果您想要更便宜的解决方案，可以使用std::make_move_iterator移动它们。 但是，由于您仍然使用boost智能指针，我认为您无法访问它。

编辑：

如果它不起作用试试这个：

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

编辑2：

为了覆盖@Maxim Egorushkin提到的nullptr案例：

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

作为旁注：不清楚为什么构造函数采用shared_ptr<X>但不使用shared_ptr<X> 。 它应该采取X&而不是。 尽可能避免使用智能指针。

理想情况下，您的代码应如下所示：

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

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