0

Есть некоторый класс, который удобно реализовать как singleton. Как лучше инициализировать его поля(информация о том как их заполнять приходит извне)?

Адекватно ли решение заставляющее клиентский код вызывать некий метод init(parameters) перед первой попыткой получить инстанс данного singlton-a?

Если да, то рентабельнее ли при этом инстанцировать лишь в том самом методе init или это совершенно не критично?

13
  • Неважно когда именно инициализируются поля. Важно чтобы это произошло до их использования.
    – Sheridan
    Commented 26 авг. 2019 в 20:22
  • Если поля как то запрашиваются извне, то можно запросить их изнутри синглтона при его инициализации.
    – Sheridan
    Commented 26 авг. 2019 в 20:23
  • class Singleton { private: int m_field; Singleton() { m_field = get_field_from_somewhere(); } }
    – Sheridan
    Commented 26 авг. 2019 в 20:35
  • @Sheridan не, это какое-то извращение, мне для этого придётся данные хранить в каком-то, скорее всего тоже, синглтоне. Степень связанности только увеличивается... Вопрос относится скорее к тому нормально ли вообще описанное и альтернативам
    – Eikthyrnir
    Commented 26 авг. 2019 в 20:43
  • А, то есть данные как то сами приходят? Синглтон может инициализироваться как до прихода данных, так и после?
    – Sheridan
    Commented 26 авг. 2019 в 20:49

1 ответ 1

0

Я бы, во избежание использования не инициализированного класса, сделал бы так.

class Singleton
{
  private:
    static std::unique_ptr<Singleton> instance;
    // Конструкторы и оператор присваивания недоступны клиентам
    Singleton(параметры конструктора) {...}
    Singleton( const Singleton& );  
    Singleton& operator=( Singleton& );
public:
    static Singleton* init(параметры конструктора) {
        assert(instance == nullptr);
        instance = std::make_unique<Singleton>(параметры конструктора);
        return instance.get();
    } 
    static Singleton* getInstance() {
        assert(instance != nullptr);
        return instance.get();
    }
}
4
  • 1
    Простите, но тут умный указатель совершенно не нужен. Владение никому не передаётся, время жизни - известно. Только ради того чтобы можно было создать экземпляр класса через вызов .get()?
    – Sheridan
    Commented 27 авг. 2019 в 5:07
  • Ради того, чтобы вызвался деструктор. Commented 27 авг. 2019 в 8:24
  • А, чтобы пару лишних строк не писать - будем оверхедить в рантайме :)
    – Sheridan
    Commented 27 авг. 2019 в 10:12
  • Нет там никакого оверхеда в рантайме. Commented 27 авг. 2019 в 20:12

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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