Шаблон проектирования Singleton (одиночка), подбираю подходящий для моей задачи Опции

[Litkevich Yuriy]

Подбираю шаблон проектирования для моей задачи.
Задача такая:
Есть программа типа MDI, хочу сделать в ней многопользовательскую работу, в одно время работает только один пользователь.
В зависимости от группы пользователя ему становится доступным определенный, для данной группы, набор пунктов меню.

Я это так прикинул:
1-Нужно хранить переменную, например, uid, в которой указан код текущего пользователя/группы.
2-Нужен диалог ввода имени и пароля, который будет где-нибудь искать есть ли такая пара логин/пароль. Если есть, то устанавливать найденное значение в переменной uid. Если нет - ругнутся на пользователя.
3-Основное окно программы получает от диалога результат: ОК-пользователь найден, надо обновить меню, НЕ ОК - пользователь не найден, ничего не делать.

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

[ViGOur]

Для твоей задачи можно использовать как глобальную переменную, так и Singleton. Singleton правда "красивше".
Немогу понять, что в чем именно заключается вопрос? Если в реализации Singleton'а, то это достаточно просто:

class CSingleton
{
private:
   static CSingleton *m_pThis;
   static int m_nCounter;

private:
   CSingleton(){}
   CSingleton( const CSingleton &){}
   ~CSingleton(){}
   operator=( const CSingleton&){}

public:
   static CSingleton *InitInstance();
   static void FreeInstance();
};

CSingleton *CSingleton::m_pThis=0;
int CSingleton::m_nCounter=0;

CSingleton *CSingleton::InitInstance()
{
   if( !m_pThis)
      m_pThis = new CSingleton();

   m_nCounter++;
   
   return m_pThis;   
}

void CSingleton::FreeInstance()
{
   if( m_nCounter>0)
      m_nCounter--;
   else
      return;

   if( m_nCounter==0)
   {
      delete m_pThis;
      m_pThis = 0;
   }
}

Примерно так, на наличие ошибок завтра проверю, а то сейчас уже голова не варит и студии под рукой нет.

Сообщение отредактировал ViGOur - 5.5.2008, 9:12

[Litkevich Yuriy]

Singleton правда "красивше". Немогу понять, что в чем именно заключается вопрос?

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

[Litkevich Yuriy]

слушай а функции InitInstance() и FreeInstance(); разве не должны быть объявлены как static, ведь они имеют дело со статической переменной?

[ViGOur]

Угу, именно так, только не потому, что статические переменные члены использует, а чтобы не создавать объект, который запрещено создавать вне класса.

Поправил.

[Litkevich Yuriy]

еще вопрос по теме задачи:
В голове крутится вариант без "Одиночки", т.е. класс прсто содержащий переменную uid обявленную как static, т.е. у всех классов она общая, краем уха слышал, что такой вариант чреват последствиями, но какими не знаю, может прояснишь?

[ViGOur]

Да никаких последствий вроде не должно быть.

[Litkevich Yuriy]

положим "одиночку" я кладу в два файла singleton.h и singleton.cpp, заголовочный подстегиваю к разным файлам, которым он может понадобиться, а в каком месте программы объявлять статические переменные:

CSingleton *CSingleton::m_pThis=0;
int CSingleton::m_nCounter=0;

[ViGOur]

Принципиальной разницы не вижу, я обычно это делаю в *.cpp.

[Litkevich Yuriy]

просто я подумал, что если в хидере будет, то он несколько раз будет скормлен компилеру, и в последствии линкер может подавиться, или не подавится?