Типизированный синглтон расширение, потенциальные проблемы??? Опции Подписка на тему Сообщить другу Версия для печати Скачать тему Подписка на этот форум Режимы отображения Древовидный Переключить на: Стандартный Переключить на: Линейный

[ViGOur]

Подумалось мне тут, что если взять типизированный синглтон:

template <class T>
class CTypedSingleton;

который создает-возвращает указатель на класс типа T, то при вызове:

delete p; // указатель на класс типа T

вызываться декремент счетчика ссылок у нас не будет и его нужно вызывать в ручную, что чревато ошибками!
Следовательно нужно создать некий класс обертку, который открыто унаследован от класса T и закрыто от нашего типизированного синглтона, а в нем уже переопределить operator delete, в котором собственно и вызывать наш CTypedSingleton<T>::free();

Что я и сделал в коде ниже:

собственно сам код

#include <iostream>

class CClass
{
public:
    virtual ~CClass(){  }
};

template <class T>
class CTypedSingleton;

template<class T>
class CTypedWrapper : public T, private CTypedSingleton<T>
{
public:
    void operator delete(void *p)
    {
        std::cout << "CTypedWrapper::operator delete" << std::endl;
        CTypedSingleton<T>::free();
    }
};

template <class T>
class CTypedSingleton
{
    static T* m_self;
    static int m_refcount;

protected:
    CTypedSingleton(){}
    CTypedSingleton(const CTypedSingleton&){}
    virtual ~CTypedSingleton(){ m_self = 0; }
    CTypedSingleton &operator=(const CTypedSingleton&){}

public:
  static T *init()
  {
      std::cout << "init" << std::endl;
      if(!m_self)
          m_self = new CTypedWrapper<T>;
      m_refcount++;
      return m_self;
  }

  static void free()
  {
      std::cout << "free" << std::endl;
      if( m_refcount > 0)
      {
          --m_refcount;
          if( m_refcount == 0)
          {
              delete m_self;
              m_self = 0;
          }
      }
  }
};

template <class T>
T *CTypedSingleton<T>::m_self = 0;

template <class T>
int CTypedSingleton<T>::m_refcount = 0;

int main(int , char **)
{
    CClass *p = CTypedSingleton<CClass>::init();
    delete p;

    return 0;
}

В принципе потенциальных проблем я не виже, но может это из-за того, что я пока грипую и возможно не так внимателен...
Что вы скажите?

[ViGOur]

А тогда как за нимим следить проще чтобы не плодить?

[lanz]

А тогда как за нимим следить проще чтобы не плодить?

Сделать один синглтон(а лучше просто класс), который будет их плодить и владеть ими, по запросу либо возвращать уже созданный, либо создавать новый.
При этом можно приделать счетчик ссылок, но мне кажется это не очень хорошо - заставлять пользователей синглтона удалять его/их (это получается уже фабрика а не синглтон).

Думается мне, что не правильно, когда к закрытым счетчикам класса, которые инкрементируются-декрементируются в двух его методах, имеют доступ все кому не лень сделать дочерний класс!

Кому очень хочется, тот и #define private public сделает, его не остановить