Pergunta

Veja também: pergunta homóloga

O código abaixo é obviamente perigoso. A questão é: como você faz faixa de manter a referência a * isso?

using namespace boost;

// MyClass Definition
class MyClass {

public:
   shared_ptr< OtherClass > createOtherClass() {
      return shared_ptr< OtherClass > OtherClass( this ); // baaad
   }
   MyClass();
   ~MyClass();
};

// OtherClass Definition
class OtherClass {

public:
   OtherClass( const *MyClass myClass );
   ~OtherClass();
};

// Call; pMyClass refcount = 1
shared_ptr< MyClass > pMyClass( new MyClass() );

// Call; pMyClass refcount = 1 => dangerous
pMyClass->createOtherClass();

Eu tenho a resposta (publicado abaixo), eu só quero que seja em stackoverflow (onde todos podem me corrija se eu estiver errado).

Foi útil?

Solução

A chave é estender enable_shared_from_this<T> e usar o método shared_from_this() para obter um shared_ptr para *this

Para obter informações detalhadas

using namespace boost;    

// MyClass Definition
class MyClass : public enable_shared_from_this< MyClass > {

public:
   shared_ptr< OtherClass> createOtherClass() {
      return shared_ptr< OtherClass > OtherClass( shared_from_this() );
   }
   MyClass();
   ~MyClass();
};

// OtherClass Definition
class OtherClass {

public:
   OtherClass( shared_ptr< const MyClass > myClass );
   ~OtherClass();
};

// Call; pMyClass refcount = 1
shared_ptr< MyClass > pMyClass( new MyClass() );

// Call; pMyClass refcount = 2
pMyClass->createOtherClass();

Outras dicas

Um par de problemas:
Seu código não compila !!

Seu código não é projetado de uma maneira que ele pára de abuso de uso / incorreto:

MyClass            x;
SharedPtr<MyClass> y = x.createOtherClass();

Agora o que?
Este parece ser um perfeitamente bom caso de uso!

Licenciado em: CC-BY-SA com atribuição
Não afiliado a StackOverflow
scroll top