Почему вызывается деструктор?

[MrGukk]

Здравствуйте,

Попытка создать синглтон привела к следующему коду:

class Single
{
public:
    static Single* CreateSingle();
    ~Single() {std::cout << "\nDestructor addr = " << addr; delete addr;};
    void show();
private:
    static Single* addr;
    Single();
    static bool flag;
    int a;
};

void Single::show()
{
    std::cout<<"\na = "<<a<<endl;
}

bool Single::flag = true;
Single* Single::addr = NULL;

Single* Single::CreateSingle()
{
    if(flag)
    {
        Single();
        addr = new Single;
        std::cout<<"\nConstructor addr = " << addr;
        return addr;
    }
    else
        return NULL;
}

Single::Single()
{
    flag = false;
    a = 66;
}

int _tmain(int argc, _TCHAR* argv[])
{
    Single* s = Single::CreateSingle();
    Single* ss = Single::CreateSingle();
    s->show();
    return 0;
}

Всё работает нормально, вот только после вызова конструктора в функции CreateSingle, вызывается деструктор.
Вывод программы:
Destructor addr = 00000000
Constructor addr = 0036F438
a = 66

Для какого объекта вызывается деструктор?

Заранее спасибо.

[counter]

if(flag)
    {
        Single();
        addr = new Single;
        std::cout<<"\nConstructor addr = " << addr;
        return addr;
    }

[MrGukk]

Извиняюсь за рудемент - Single() убрал из CreateSingle().
Теперь всё работает правильно.
Причина ошибки ясна, но не ясно что-же тогда конструирует Single()?

[MrGukk]

И раз уж зашла речь о синглтонах. Покритикуйте плиз такой вариант реализации. Буду благодарен.

[waleri]

Сообщение от MrGukk

не ясно что-же тогда конструирует Single()?

new Single;

[waleri]

Сообщение от MrGukk

Покритикуйте плиз такой вариант реализации. Буду благодарен.

flag излишен. Вполне можно использовать addr для той же цели.

[pproger]

2MrGukk
плохо. flag не нужен, CreateSingle с какого-то перепуга возвращает NULL, потоконебезопасно.

class Singleton() {
public:
    static Singleton &instance();

private:
    Singleton() {};
    Singleton(const Singleton &);
    Singleton &operator=(const Singleton &);
};

Singleton &Singleton::instance()
{
    static Singleton inst;
    return inst;
}

...

Singleton &s = Singleton::instance();

[MrGukk]

Сообщение от waleri

new Single;

Т.е. вы хотите сказать что при вызове конструктора явно в методе класса Single() эквивалентен new Single?
Конструктор ведь ничего не возвращает, т.е. после выделения памяти указатель на выделенное место в куче теряется и память должна утекать.
Тем не менее деструктор вызывается, но для какого "адреса"?
Посмотрел гугл - не нашёл. Не могли бы вы дать ссылку на разъяснение?

[MrGukk]

Спасибо за критику)

[waleri]

Сообщение от MrGukk

Т.е. вы хотите сказать что при вызове конструктора явно в методе класса Single() эквивалентен new Single?
Конструктор ведь ничего не возвращает, т.е. после выделения памяти указатель на выделенное место в куче теряется и память должна утекать.

Я хочу сказать, что когда мы создаем объект через new вызывается его конструктор. Ваш вопрос был "что конструирует Single", я и ответил. Да, конструктор ничего не возрващает, но вы то НЕ вызываете конструкор а оператор new, а вот он возвращает.

В С++ нельзя вызывать конструкторы и деструкторы напрямую, явно (деструкторы можно, но ето скорее изключение и надо знать, что и зачем делается.).

В вашем случае деструктор вообще не вызывается (или по крайней мере я не вижу где удаляется addr).