Как правильно передвать между окнами MFC сложный тип

[Boriska]

Привет народ. Имею к класс

Код: (C++)

class CBox public CDescription,public CGeometric,public CNodes

В классе CNodes объявлены три указателя на вектора. Весктора содержат указатели на классы

Код: (C++)

BOX_NODE_COORDINATES_PNTS_VECTOR                *m_p_boxDestinNodesVector;
BOX_NODE_COORDINATES_PNTS_VECTOR                *m_p_boxDeltasNodesVector;
BOX_PLACES_PNTS_VECTOR                  *m_p_placesVector;

Сделал диалог главное окно.  В классе окна переменную

Код: (C++)

CBox m_activeBox;

Заполняю переменную и для теста вывожу

Код: (C++)

char buff[500];
        unsigned int indx;

        unsigned int size = (*m_activeBox.m_p_placesVector).size();
        for(indx = 0; indx < size; indx++){
                sprintf_s(buff,sizeof(buff),"MnD=%d MxD=%d Row = %d Colm = %d  XPos = %d  YPos = %d PosRow = %d PosColm = %d",
                        m_activeBox.GetMnD(),
                        m_activeBox.GetMxD(),
                        (*m_activeBox.m_p_placesVector)[indx]->Row(),
                        (*m_activeBox.m_p_placesVector)[indx]->Colm(),
                        (*m_activeBox.m_p_boxDestinNodesVector)[indx]->XPos(),
                        (*m_activeBox.m_p_boxDestinNodesVector)[indx]->YPos(),
                        (*m_activeBox.m_p_boxDestinNodesVector)[indx]->Row(),
                        (*m_activeBox.m_p_boxDestinNodesVector)[indx]->Colm()
                        );
                MessageBox(buff,"");   
        }

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

Код: (C++)

void CColorTuneDlg::OnBnClickedRightFrontBtn()
{
        if(m_pCD->CheckCurrentUserPermitoins() != SUFFICIENT_PERMISSIONS)return;

//      m_pCD->m_activeBox.SetRx(m_HorizFront.GetPos());
//      m_pCD->//m_pf_TransferBoxBottleParams(&(m_pCD->m_activeBox));
//      m_pCD->m_activeBox.UpdateBox(m_pCD->m_pLocDb);
        m_pCD->UpdateDestinNodes(m_pCD->m_activeBox);

m_pCD - указатель на главный диалог.

Код: (C++)

void CTestDlg::UpdateDestinNodes(CBox & box)
{
        char buff[500];
        CBottleBox b;
        b = box;
        unsigned int indx;
        unsigned int size = (*b.m_p_boxDestinNodesVector).size();

        for(indx = 0; indx < size; indx++)
        {
                sprintf_s(buff,sizeof(buff),"UPDATE Nodes SET Xpos = %d, Ypos = %d WHERE Row = %d AND Col = %d AND boxId = %d",
                        (*m_activeBox.m_p_boxDestinNodesVector)[indx]->XPos(),
                        (*m_activeBox.m_p_boxDestinNodesVector)[indx]->YPos(),
                        (*m_activeBox.m_p_boxDestinNodesVector)[indx]->Row(),
                        (*m_activeBox.m_p_boxDestinNodesVector)[indx]->Colm(),
                        m_activeBox.GetBoxId());
                m_pLocDb->ExecuteSQL(buff);
        }      
}

и тут обнаруживается что переменная size содержит жуть большое число. Ну соотвественно и все другие данные в вектрах съезжают и крах.
В то же время если вызвать эту функцию в диалоге где определена переменная m_activeBox там все ОК.

[Dimka]

Правильнее вынести модель в отдельные классы и передавать их объекты, а не сами окна.

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

[Boriska]

Я окна не передаю. У меня в классе диалога главного окна определена переменная. А в другие окна я передаю указатель на класс этого окна и таким образом из классов диалогов других окон добираюсь до главного. Я же написал. И функция которая сохраняет данные тоже находится в классе диалога главного окна. Так вот если ее вызывать внутри своего класса тогда все ОК а если по указателю из другого окна то баг.

[Dimka]

У меня в классе диалога главного окна определена переменная. А в другие окна я передаю указатель на класс этого окна и таким образом из классов диалогов других окон добираюсь до главного.

"Передать указатель на объект окна" - это и есть коротко "передать окно".

[Boriska]

А как это делать правильно?

[Джон]

Нууу как Димка уже сказал: отделить данные от окон. Т.е. твоя CBox m_activeBox; не должна быть членом CTestDlg, а общедоступным объектом, указатель на который получают обработчики/манипуляторы.
А так ты сам себя запутал, ведь в UpdateDestinNodes(CBox & box) box и есть m_activeBox.

А непосредственно по проблеме, похоже что у тебя что-то не проинициаллизировано. Не совсем понятно КОГДА ты заполняешь переменную для теста, и КОГДА жмёшь кнопку вызова диалога CColorTuneDlg.

[Boriska]

Да вот странное дело. Создал тестовый проект. Симитировал эту ситуацию и вроде все работает.
Создал в рабочем проекте другую переменную и не поверите с нее все вывожу в другом окне.
Значит где то я лезу в мою переменную. Погляжу.

Кстати насчет отделения мух от котлет. Это вы делаете глобальные переменные а в окнах храните только указатели на них?

[Dimka]

lkote, не обязательно глобальные, их можно передавать через параметры методов и хранить в членах классов.

[Джон]

... и не обязательно хранить в окнах указатели. Можно сделать глобальный доступ. Например, в MFC класс производный от CWinApp напрашивается сам собой. Если добавить в него метод типа:

CBox * GetActiveBox() { return например &m_activeBox; }

то тогда везде в программе можно будет пользоваться theApp.GetActiveBox();

[Boriska]

Народ а подскажите еще. Вот у меня sql размазан по коду. По идее это не красиво. На С# пробовал
там это можно в xml файлах. Так сказать все подготовлено для этого. С++ программистов вокруг практически
нет чтоб спросить. Как в этом случае правильно поступать, где можно набраться техники?

[Dimka]

Во-первых, SQL и XML - вещи разные. SQL - язык запросов к базе данных, а XML - язык разметки для описания структурированных данных.

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

В случае .NET последних версий речь может идти об использовании LINQ - интегрированном в язык подъязыке запросов, который позволяет единообразно работать с любыми источниками данных (реляционные базы данных, XML, плоские файлы, внутренние массивы программы и т.п.).