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

есть класс

Код

class CSession { public:    CSession();    ~CSession();    bool LoadSession();    bool AddVariable(const CString& VarName, const CString& VarValue);    bool SaveSession(); protected: private:    std::map<CString, CString> m_SessionVariables; }

и класс

Код

class CParams { public:    CParams();    ~CParams();    const CString& operator [] (CString& ParamName) const; protected: private:    std::map<CString, CString> m_Params; }

И еще один

Код

class CPage { public:    CPage()    {        m_pSession = 0;    };    ~CPage()    {        if ( m_pSession )           m_pSession->SaveSession();               delete m_pSession;    }    CSession* InitSession() {        if ( !m_pSession )        {           m_pSession = new CSession;           m_pSession->LoadSession();        }        return pSession;    } private:    CSession* m_pSession; }

В классе CPage можно только добавлять в сессию переменные, а читать их нужно из CParams.
примерно так

Код

void OnSomething(const CParams& Params) {    //Params["ID"] не существует    InitSession()->AddVariable("ID", "1");    //Params["ID"] должен быть равен 1 }

Как спроектировать иерархию классов, кого с кем подружить..есть идеи конечно, но как говорится одна голова хорошо а две как понимаете лучше..а тут их побольше будет

Думаю сделать класс CParams другом CSession чтобы он мог использовать map из CSession. Но тогда встает другой вопрос, откуда он получит указатель на CSession? Можно CParams -у при инициализации передать указатель на CPage но криво как то получается...
Есть идеи?
Добавлено @ 14:25
Или же пихать InitSession в CParams, это на первый взгляд самый легкий способ, но он логически туда не подходит..

Это сообщение отредактировал(а) azesmcar - 22.12.2005, 14:28

Как вариант, могу предложить такой:

В класс CSession допиши метод который принимает как параметры CParams. В методе AddVariable передаёш параметп ниже в СSession. Только чёто непойму, какая то странная инициализация.
Очень похожа на синглтон, но не то.

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

Это HTTP сессия, для CGI
Добавлено @ 15:08
Только что прошло в мыслях что то вроде этого...
Сделать CSession в CPage статической переменной, при создании CParams -а передать ему ссылку на CSession.

class CPage;

Код

class CParams { public:    CParams(const CPage& Page)    {       m_pPageRef = &Page;    };    ~CParams();    const CString& operator [] (CString& ParamName) const    {       return m_pPage->m_pSession->m_SessionVariables[ParamName];    }; protected: private:    std::map<CString, CString> m_Params;    CPage *m_pPage; }

В этом случае operator [] будет читать из m_pPage->m_pSession->m_SessionVariables а CParams -а сделать другом CSession.

Но все равно не очень нравится, наверняка можно придумать что нибудь поэлегантнее..

А что мешает создать переменую типа CParams в классе CSession?

Цитата

А что мешает создать переменую типа CParams в классе CSession?

То что CParams связан не только с сессией, он читает множество параметров из разных мест, и один из этих место сессия..опять таки логически не подходит..т.е. событие
OnSomething получает не CParams а CSession в которой хранятся множество параметров в том числе и сессия...

но он должен где то создаваться

Цитата(azesmcar @ 22.12.2005, 14:35)

CParams

где

Цитата(azesmcar @ 22.12.2005, 14:35)

OnSomething получает не CParams а CSession

Цитата(azesmcar @ 22.12.2005, 13:24)

void OnSomething(const CParams& Params)

В функции main()

Код

int main() {   CParams params;   params.ReadParameters(ReadParams_GET|ReadParams_POST|ReadParams_SESSION);   CPage *pPage = new CMyPage;   pPage->InitPage( params );   return 0; }

Добавлено @ 15:55

Код

void CMyPage::InitPage(const CParams& params) {    if ( params["SOMETHING"] == "1" )    {        OnSomething(params);    } }

Ну например так
Добавлено @ 15:57

Код

class CPage { public:   //Отпускаем детали   virtual void InitPage(const CParams& params) = 0; } class CMyPage: public CPage { public:   virtual void InitPage(const CParams& params) }

Я понимаю, что тебе не нравится - слишком много связей - все знают почти обо всех.
Может, можно убрать связь Session-Page? Т.е. пусть страница знает про параметры, а параметры - про сессию.
Собственно, ты похоже так и сделал - в последнем пимере, да?
А дружить никого ни с кем не надо, просто объяви
typedef map<CString,CString> CVariables;
в public-части класса Params (по-моему, самое логичное место), и сделай методы для чтения.

Цитата(azesmcar @ 22.12.2005, 14:54)

OnSomething(params);

'это отдельная ф-ция или чейто метод?
ЗЫ если честно я чет основного вопроса не понял (уже) !!!

Цитата

OnSomething(params);

это функция метод класса CMyPage

Цитата

Может, можно убрать связь Session-Page? Т.е. пусть страница знает про параметры, а параметры - про сессию.

Собственно вчера вечером я так и сделал, подружил сессию и параметры. После чего параметры читают данные напрямую из map<CString, CString> класса сессии..
Сессия обьявлены static. Она на всех одна. Используется примерно так..

Код

void CMyPage::InitPage(const CParams& params) {    CSession *pSess = params.InitSession();    pSess->AddVariable("Login", params["LOGIN"]); }

в любое время можно вызвать.

Код

params.GetSession()->DestroySession();

Earnest Ну если никаких добавок по этому поводу нет тогда наверное так и оставлю..

Цитата(azesmcar @ 23.12.2005, 08:59)

Сессия обьявлены static. Она на всех одна

Тогда реализуй ее как синглетон. Ибо тогда выражение params.GetSession() как-то бессмысленно выглядит (вроде как у каждого экземпляра параметров может быть своя сессия).

Цитата

Тогда реализуй ее как синглетон. Ибо тогда выражение params.GetSession() как-то бессмысленно выглядит (вроде как у каждого экземпляра параметров может быть своя сессия).

Ну можно заменить GetSession() на что то вроде GetSessionPtr()

А что такое синглетон..не встречал что-то такого...если не сложно пример приведи...

Цитата(azesmcar @ 23.12.2005, 17:54)

Ну можно заменить GetSession() на что то вроде GetSessionPtr()

Да какая разница, я и не думала, что ты объект по значению возвращаешь.
Плохо то, что функция эта как бы говорит: "вот вам моя сессия", а сессия эта вовсе не их (параметров), а общая. Понимаешь?

Синглетон - это паттерн проектирования, одиночка то есть. По сути, все как у тебя, только более ясно выражено + принимаются меры, чтобы объект действительно был один.
Я обычно пользуюсь следующей парой макросов, чтобы код ручками не писать:
(Знаю-знаю, отцы макросы использовать не велят, но я привыкла, мне так кажется удобнее ).

Код

// singleton.h: все, кто хочет быть синглетоном, его включают  // это используем внутри определения класса #define DECLARE_SINGLETON(Class) \ private: \    Class (); \ public:       \    ~Class(); \    static Class& Instance(); // а это - в файле реализации класса #define IMPLEMENT_SINGLETON(Class) \    Class& Class::Instance() \    { \       static Class theInst;  \       return theInst;          \    }

Использование:

Код

// session.h #include "singleton.h" class CSession {    DECLARE_SINGLETON(CSession); public:    // конструктор-деструктор уже объявлены в DECLARE_SINGLETON     ..... }; // ну и еще один макрос, чтобы покороче и редактор подсвечивал #define   theSession   CSession::Instance()

Код

// sesion.cpp #include "session.h" // это нам функцию Instance нарисовали IMPLEMENT_SINGLETON(CSession) CSession() {    ... } ~CSession() {    ... } ...и прочая

Далее, везде, где нам нужен объект-сессия, пишем:

Код

theSession.ДайЧегоНибудь();

Этот синглетон очень простой, умеет создаваться только без параметров, не является потокозащищенным, момент разрушения неуправляем и т.д. Если что-то не устраивает - дорабатываем. Возможные варианты обсуждаются, по-моему, у Майера, во 2м эффективном програмировании.

Цитата

Да какая разница, я и не думала, что ты объект по значению возвращаешь.

Не по значению возврашаю я в том смысле чтобы функция как бы говорила вот вам указатель на сессию...а не вот вам моя сессия..возвращять она будет все равно тоже самое... CSession *

Я просто кое чего не понял..на кой черт все это нужно...ну макросами так макросами...я и сам правила дядюшки Страуструпа иногда нарушаю...только зачем все это??
Есть класс...

Код

class CParams { public:    static CSession *Session; }

Код

void OnSomething(const CParams& Params) {    Params.Session->RegisterVariable(...); }

Знаю, опять не подходит...верно, пишем макрос

Код

#define Session CParams::Session

Earnest,

обожаю изящные и удобные конструкции (решения).

off: жаль, не могу репутацию добавить. не дорос ишо ;)

Это сообщение отредактировал(а) blackofe - 23.12.2005, 19:17