Проблема реализации класса Singleton

Question

Существует статический указатель aSingletonClass * instance_ и статическая функция void deleteInstance (void), объявленная в коде 1.

Я реализовал Код 2 без использования статического указателя и статической функции (я думал, что Singleton может быть достигнут без него, но я не уверен !!!)

Пожалуйста, сообщите мне, может ли это изменение быть опасным, и в этом случае моя программа может аварийно завершить работу.

Код 1:

class aSingletonClass
{
public:

  static aSingletonClass *getInstance( void )
  {

    if(!instance_)
      instance_ = new aSingletonClass;
    return instance_;
  }

  static void deleteInstance()
  {
    if(instance_)
      delete instance_;
    instance_ = NULL; //important as this can create dead reference problems
  }

private:

  static aSingletonClass *instance_;

  aSingletonClass() {};

  ~aSingletonClass() {};
};

int main()
{
  aSingletonClass *someVar = NULL;

  someVar = aSingletonClass::getInstance();

  aSingletonClass::deleteInstance();

  return 0;
}

Код 2:

class A
{
    int x;
    A(int a)
    { x=a;  }

    ~A()
    { }

public:
     static A* start()
     {
       A* ptr1 = new A(3);
       return (ptr1);
     }


     void end()
     {
       delete this;
     }
};

int main()
{
  A* ptr=A::start();

  ptr->end();
}

Answer 1

Тьфу. Не очень хороший способ делать одиночные игры!

Я бы сделал что-то вроде следующего ..

class A
{
private:
  A() {}

public:

  static A& instance()
  {
    static A _inst;
    return _inst;
  }

};

Для использования,

A& inst = A::instance()

Всегда будет возвращать единственный экземпляр!

РЕДАКТИРОВАТЬ: у этого подхода есть несколько преимуществ - вам не нужно беспокоиться об очистке, но он загружен (или инициализирован), и вы работаете со ссылками! Черт, у тебя даже может быть постоянный синглтон (почему, я уверен, ты можешь подумать о каком-то использовании!;))

Answer 2

Ваша вторая версия не является одноэлементной, потому что у вас нет статического A* указателя и вы просто возвращаете новый при каждом вызове start (), поэтому в вашем коде действует следующее:

int main()
{
    A* ptr = A::start();
    A* ptr2 = A::start();
    A* ptr3 = A::start();
    // etc
    ptr->end();
    ptr2->end();
    ptr3->end();
    return 0;
}; // eo main

Answer 3

Код 2 не будет одноэлементным.В функции start () вы возвращаете указатель на новый объект A.Но в функции end () вы удаляете этот объект.Так что не получится.

Answer 4

Ваш код должен быть таким:

class A    {
  public:
    static A *getInstance( void )
    {
       if(!instance_)
          instance_ = new A();
       return instance_;
    }
    static void deleteInstance()
    {
        if(instance_)
           delete instance_;
        instance_ = NULL; //important as this can create dead reference problems
    }
    static A* start()
    {
        A* ptr1 = A::getInstance();
        ptr1 -> started = true; // check here if A had been started already !!!!
        return (ptr1);
    }

    static void end()
    {
        A* ptr1 = A::getInstance();
        ptr1 -> started = false; // check here if A had ever been started !!!!
        //A::deleteInstance();      // may be it will be useful here
    }
  private:
    bool started = false;
    static A *instance_;
    A() {};
    ~A() {};
};

int main()
{
  A *someVar = NULL;
  someVar = aSingletonClass::getInstance();
  A::deleteInstance();

  return 0;
}