2012-05-10 15 views
7

que estoy tratando de pasar 'esto' al constructor usando std :: make_sharedC++ 11 Passing 'esto' como paramenter para std :: make_shared

Ejemplo:

// headers 
class A 
{ 
public: 
    std::shared_ptr<B> createB(); 
} 


class B 
{ 
private: 
    std::shared_ptr<A> a; 

public: 
    B(std::shared_ptr<A>); 
} 


// source 
std::shared_ptr<B> A::createB() 
{ 
    auto b = std::make_shared<B>(this); // Compiler error (VS11 Beta) 
    auto b = std::make_shared<B>(std::shared_ptr<A>(this)); // No compiler error, but doenst work 
    return b; 
} 

Sin embargo, esta no funciona correctamente, ¿alguna sugerencia de cómo puedo pasar esto como argumento?

Respuesta

13

Creo que lo que probablemente quiera aquí es shared_from_this.

// headers 
class A : std::enable_shared_from_this<A> 
{ 
public: 
    std::shared_ptr<B> createB(); 
} 


class B 
{ 
private: 
    std::shared_ptr<A> a; 

public: 
    B(std::shared_ptr<A>); 
} 


// source 
std::shared_ptr<B> A::createB() 
{ 
    return std::make_shared<B>(shared_from_this()); 
} 

actualización para incluir comments from David Rodriguez:

Tenga en cuenta que debe shared_from_this() no se recurrirá a un objeto que no esté gestionado por un shared_ptr. Esto es válido:

shared_ptr<A> a(new A); 
a->createB(); 

Mientras que los siguientes conduce a un comportamiento indefinido (intentando llamar a delete en a):

A a; 
a.createB(); 
+0

Muchas gracias Andrés! – RdR

+7

Es importante tener en cuenta que 'shared_from_this()' requiere que el objeto actual ya esté administrado en un 'shared_ptr'. Es decir. 'int main() {A a; a.createB(); } 'es un comportamiento indefinido, mientras que' int main() {shared_ptr a (nuevo A); a-> createB(); } 'es correcto. –

+0

@ DavidRodríguez-dribeas Absolutamente. He actualizado la respuesta en consecuencia. Gracias. –

Cuestiones relacionadas