1

Как я реализовал:

#pragma once
class Singletone {
private: 
Singletone(): instance(nullptr) {}
Singletone(const Singletone&);
const Singletone& operator=(const Singletone&);
static Singletone * instance;
~Singletone() { delete instance; }
public:

  static const Singletone& getInstance();

}

singletone.cpp:

#include "Singletone.h"
Singletone* Singletone::instance = nullptr;
const Singletone& Singletone::getInstance() {
     if(instance==nullptr)
           instance = new Singletone();
    return *instance;
}   

Скажите, насколько правильна такая реализация?
Например, я не могу вернуть ссылку т.к. запрещен конструктор копирования (который должен быть запрещен, верно?)
Что тогда делать, возвращать указатель на объект, или?...
И какой лучший способ создания однопоточного синглтона в C++?

7
  • > Например, я не могу вернуть ссылку > т.к. запрещен конструктор копирования > (который должен быть запрещен, верно?) при создании ссылок и указателей конструктор ненужен (объект ведь не создаётся) подробнее о возврате ссылок [Как работает возврат ссылки? ][1] [1]: hashcode.ru/questions/178009#180400 8 янв 2013 в 20:50
  • Еще могу порекомендовать чтение groups.google.com/forum/?fromgroups=#!topic/… (большей частью из-за комментариев по теме). 8 янв 2013 в 23:35
  • А мне одному кажется, что строка ~Singletone() { delete instance; } вызовет UB? Если есть только один объект instance, зачем при его удалении. пытаться удалить его еще раз?
    – nnesterov
    9 янв 2013 в 5:53
  • Я наверное что-то не понимаю. В данном случае строка delete instance вызовет деструктор ~Singletone(), который опять выполнит строку delete instance, которая опят вызовет деструктор и так далее. Или я что-то упускаю?
    – nnesterov
    9 янв 2013 в 9:25
  • 2
    @nnesterov Сейчас еще раз прочитал код топикстартера и деструктор ~Singletone() вообще не будет вызван никогда, поскольку статическим членом класса является raw указатель. Поэтому того, о чем вы говорите, не произойдет, но деструктор, очевидно, бессмысленен. Извините, что ввел в заблуждение предыдущим комментарием :) 9 янв 2013 в 9:31

3 ответа 3

6

Есть же классические реализации синглтонов, почему не воспользоваться? Заодно узнаете типовые ошибки:

1
  • спасибо .
    – PaulD
    8 янв 2013 в 20:29
4

1.

if(instance==nullptr)
       instance = new Singletone();

А с чего Вы решили, что неинициализированный указатель будет равен nullptr ?

2.

Например, я не могу вернуть ссылку т.к. запрещен конструктор копирования (который должен быть запрещен, верно?)

А какая связь между возвращаемой ссылкой и конструктором копирования?

3
  • mikillskeg, посмотрите пожалуйста еще раз мой вопрос, я там добавил что в .cpp instance инициализируется nullptr'ом. 2. оно не может сконструировть объект при его возвращении.
    – PaulD
    8 янв 2013 в 20:17
  • 2
    А как и где будет инициироваться указатель, если он объявлен как private? А если его сделать public, то последнее будет хуже первого. Лучше прислушайтесь к ответу @KoVadim'а и изучите синглтон Мейерса. Он похож на Ваше решение, но не совсем. Удачи.
    – skegg
    8 янв 2013 в 20:39
  • 1
    Во-первых, указатель обявлен как static, т.е. он гарантированно будет изначально nullptr. Во-вторых, он даже явно инициализирован. 5 июл 2017 в 9:27
4

У вас лишний деструктор. Фактически, вы не должны никому разрешать уничтожить синглтон. Деструктор нужно оставить приватным, но пустым.

Других проблем в однопоточном случае не вижу.

Для запрета конструктора копирования, оператора присваивания и деструктора можно воспользоваться конструкцией = delete, если ваш компилятор поддерживает C++11. Но это вопрос вкуса.

Ссылку вы вернуть можете потому, что ссылка на объект представляет тот же самый объект, копирования не происходит.

ЗЫ: на английском вроде бы Singleton, без "e".

ЗЗЫ: да, у вас ещё нету точки с запятой после объявления класса.

ЗЗЗЫ: А синглтон Майерса таки гораздо изящнее.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.