Класс настроек

[deMax]

Как правильно реализовать сабж?
Настройки должны отображаться в окне настроек, их используют разные модули программы. Singletone? MVC?

[vizir.vs]

Вариант с синглтоном мне нравится больше. Что за MVC - Model Viewer Controller? а как он здесь связан? Второй вариант через наблюдателя. Но тогда надо все классы наследовать от наблюдателя и определять соответствующие методы.

P.S. если у тебя настройки меняются и тебе потом надо рассылать обновления, то это либо через наблюдателя, либо синглтоно, но нужно слать сигналы всем окнам, чтобы они обновили инфу о настройках.

[navrocky]

Делаешь класс на базе QObject с сигналом changed(), с сохранением в реестр/ini по методу commit(). Далее, зависит от архитектуры твоего приложения, или делаешь синглтоном объект этого класса или раздаешь указатель на него всем заинтересованным окнам, объектам.

В общем-то, если у тебя будет отдельное приложение - то скорее всего можно использовать синглтон. Если разделяемая библиотека, то однозначно надо тянуть указатель.

[kai666_73]

Если разделяемая библиотека, то однозначно надо тянуть указатель.

Чёоо
В библиотеках нельзя использовать синглтоны? Или объявлять?

[Alexu007]

Сделать переменные настроек глобальными, меню настройки пусть их меняет, а разные модули программы только читают. В чём проблема? 

[vizir.vs]

Сделать переменные настроек глобальными, меню настройки пусть их меняет, а разные модули программы только читают. В чём проблема? 

Задача синглтона сделать класс легкодоступным всем и при этом гарантировать единственность этого класса. А если ты сделаешь просто глобальную переменную кто угодно может ее создать или скопировать, но настройки то должны быть одни для всей программы! И не должно быть возможность создать еще одну переменную настроек или скопировать существующие.

[Igors]

Сделать переменные настроек глобальными, меню настройки пусть их меняет, а разные модули программы только читают. В чём проблема? 

В какой-то момент приходит неприятное ощущение "бардака", обнаруживается что разрозненных переменных уже слишком много и надо с ними что-то делать. Ну это конечно когда проект разрастется.

[navrocky]

Если разделяемая библиотека, то однозначно надо тянуть указатель.

Чёоо
В библиотеках нельзя использовать синглтоны? Или объявлять?

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

[Bepec]

Несогласен с последним оратором.
Синглтон подразумевает, что архитектура приложения будет фиксированной.

Это приводит к тому, что библиотека будет пользоваться ТОЛЬКО так, как и задумывалось.
Что с одной стороны правильно - неправильно её использовать будет почти невозможно
С другой стороны - приложение, её использующее должно соответствовать определённым требованиям.

Правильный синглтон не обращает внимание на такие мелочи как "многопоточность". Но это уже заслуга его автора и продуманности архитектуры

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

[kai666_73]

Если разделяемая библиотека, то однозначно надо тянуть указатель.

Чёоо
В библиотеках нельзя использовать синглтоны? Или объявлять?

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

Так значит все-таки объявлять...
Аргументы либо притянуты, либо далеки от контектста:
> Плохой дизайн
  если только неграмотная реализация синглтона...
> в многопоточной среде
  настройки в отдельном потоке? оно надо?
> Синглтон накладывает серьезное ограничение на способ использования
  так ведь в этом его предназначение. Одиночка она и в приложении и в библиотеке одиночка

ЗЫ. Ой, Верес опередил.

[Igors]

Ну давайте копнем, что ли (а то много общих слов  ). Корректна ли такая реализация ?

Код

C++ (Qt)
MySingle * MySingle::mInstance = 0;
 
MySingle * MySingle::Instance( void )
{
 static QMutex mutex;
 if (!mInstance) {
  QMutexlocker lock(&mutex);
  mInstance = new MySingle;
 }
 return mInstance;
}

[OKTA]

Хм, я бы поставил QMutexLocker перед if. А так, подозрений не вызывает)

[Bepec]

++ убрать перед if. На первый взгляд нормальная архитектурка.

[kai666_73]

Ага и в результате получать фризы при каждом вызове Instance, даже когда физически инстанс создан. Но деваться некуда если хотим потокобезопасный синглтон... при данной реализации...

А если так?

Код:

QScopedPointer<MySingle> MySingle::m_instance(new MySingle());
MySingle *MySingle::instance() {
    return m_instance.data();
}

[m_ax]

А про стандарт с++11 кто-нить вообще слышал?