shared_ptr и этот указатель

Question

ОК, я начал использовать общие указатели и передавать общие указатели как можно больше. Больше не конвертировать в исходные указатели. Это работает хорошо, но в данном конкретном случае:

Предположим, у нас есть класс, который также является наблюдателем от другого класса, например:

class MyClass : public IObserver 
    { 
    public: 
     MyClass (std::shared_ptr<SomeOtherClass> otherClass); 
     void DoSomethingImportant(); 
    private: 
     std::shared_ptr<SomeOtherClass> m_otherClass; 
    }; 

Этот класс используется, как это в моем приложении:

std::shared_ptr<MyClass> myInstance(new MyClass(otherInstance)); 
... 
myInstance->DoSomethingImportant(); 

MyClass получает общий указатель на другой класс и сохраняет его в своем элементе данных m_otherClass. В методе DoSomethingImportant экземпляр MyClass выполняет множество важных задач, включая регистрацию себя как наблюдателя на m_otherClass, например:

m_otherClass-> registerObserver (this);

Проблема заключается в том, что метод registerObserver определяется следующим образом:

недействительного registerObserver (станд :: shared_ptr наблюдателя);

Он ожидает общий указатель, но «это» - это необработанный указатель, а не общий.

Я вижу три пути решения этого:

• Найти трюк, чтобы преобразовать обычный указатель на общий указатель (см вопроса convert pointer to shared_ptr), но ответы на этот вопрос только предложить, чтобы скопировать общие-указатели , а не о том, как фактически преобразовать указатель в общий указатель.
• Передайте общий указатель на наш метод, например: «myInstance-> DoSomethingImportant (myInstance);» что кажется немного глупым.
• Поместите часть наблюдателя в отдельный класс. Это похоже на излишний, и может сделать класс труднее понять.

Эта проблема делает очевидным, что общие указатели - это просто дополнение к C++ (я не думаю, что у вас такая же проблема на других языках/средах, таких как C# (или .Net вообще) и Java).

Любые другие предложения или рекомендации по устранению этой ситуации?

Answer 1

Возможно, что вам нужно enable_shared_from_this и shared_from_this. Документы: here

Обратите внимание, что вы не можете использовать shared_from_this, пока конструктор не будет полностью завершен, а объект уже принадлежит другому shared_ptr.

struct test : boost::enabled_shared_from_this<test> 
{ 
    test() { 
     // shared_from_this(this); // error, still not owned by another shared_ptr 
    } 
    boost::shared_ptr<test> shared() { 
     return shared_from_this(this); 
    } 
}; 
int main() { 
    test * t = new test; 
    // boost::shared_ptr<test> p = t->shared(); // error, not yet owned by other shared_ptr 
    boost::shared_ptr<test> owner(t); 
    boost::shared_ptr<test> p = t->shared();  // [*] ok, "owner" owns the object 
} 

[*] Эта часть примера глупо, вы можете просто скопировать владельца в р, вместо вызова метода.Это просто замечается, когда это нормально или не называть shared_from_this внутри методов test.

Answer 2

Как о том, что конструктор приватным и имеющий статический метод строительства, как это:

class MyClass : public IObserver 
    { 
    public: 
     static std::shared_ptr<MyClass> createObserver(std::shared_ptr<SomeOtherClass> otherClass); 
     void DoSomethingImportant(); 
    private: 
     MyClass (std::shared_ptr<SomeOtherClass> otherClass); 
     std::shared_ptr<SomeOtherClass> m_otherClass; 
    }; 

Тогда вы могли бы создать экземпляр наблюдателя чисто в статическом методе и не придется беспокоиться о этом указателе на всех.

Answer 3

Можете ли вы перевести стадию регистрации в отдельный метод? :

shared_ptr<SomeOtherClass> other(new SomeOtherClass()); 
shared_ptr<MyClass> my(new MyClass()); 
// register myself to the observer 
other->registerObserver(my); 
my->DoSomethingImportant(); 

Хороший дизайн шаблона наблюдателя может быть реализован с буст :: сигнала и повышение :: связать библиотеки. Я призываю вас взглянуть.

С наилучшими пожеланиями,
Marcin

Answer 4

Для шаблона наблюдателя, наблюдаемый объект не принимает права собственности наблюдателя, почему бы не просто использовать сырой указатель? Жизненный цикл наблюдателя должен контролироваться самим наблюдателем.

Используя enable_shared_from_this, вы вводите зависимость цикла для наблюдателя и его наблюдаемого объекта. Это означает, что если не удалять явно, ресурс никогда не будет выпущен.