Ces constructeurs sont-ils corrects?

Question

Ces constructeurs sont-ils corrects? Sont-ils bien formés et correctement surchargés à la fois pour les valeurs et les valeurs?

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

Cela me semble bon.

Dans le constructeur de mouvement, std::move(p_data) est nécessaire car p_data est une lvalue.

Answer 2

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

Etes-vous sûr que vous voulez implicites conversions de std::string-Obj? Si pas, pensez à faire le constructeur explicit:

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