这些构造函数是否正确？

Question

这些构造函数是否正确？

它们是否形成良好并正确地重载了​​左值和右值？

std::shared_ptr<std::string> data; 

// lvalue constructor 
Obj(std::string const & p_data) 
    : data {std::make_shared<std::string>(p_data)} 
{} 

// rvalue constructor 
Obj(std::string && p_data) 
    : data {std::make_shared<std::string>(std::move(p_data))} 
{} 

Answer 1

看起来不错。

在移动构造函数中，std::move(p_data)是必需的，因为p_data是一个左值。

Answer 2

Obj(std::string const & p_data) 
    : data {std::make_shared<std::string>(p_data)} 
{} 

你确定你想从std::string隐转换到Obj？如果不，考虑使构造explicit：

// Prevent *implicit* conversions from std::string to Obj 
explicit Obj(std::string const& p_data) 
    : data{std::make_shared<std::string>(p_data)} 
{}