Не могу получить копии / назначить конструкторы работают

Question

Не могу получить копии / назначить конструкторы работают

0

Я делаю сборку MFC как класс на WinAPI как exersise. Теперь я сталкиваюсь с проблемой, что я получаю ошибку выполнения во время выполнения кода.

В моей главной строке присутствуют следующие строки:

void MyApp::Init(int nCmd)
{
    main.Create(L"This is the MainFrame Window");
    main.ActivateFrame(nCmd);   
    GWnd* p= main.GetActiveView();//Get pointer to the first view of the FrameWnd
    //p->GetID();
}

Записанная строка вызывает ошибку. Конструкторы класса GWnd:

//Standard classname
GWnd::GWnd() :m_ClassName(L"Wnd"), BaseWnd(), m_pOwnerWnd(NULL)
{
    GWnd::Init();
}
//Userdefined Classname
GWnd::GWnd(LPCWSTR ClassName) : m_ClassName(ClassName), BaseWnd(), m_pOwnerWnd(NULL)
{
    GWnd::Init();
}
//Copy constructor
GWnd::GWnd(const GWnd& Wnd) : BaseWnd(GetHwnd()), m_pOwnerWnd(NULL)
{
    m_ClassName = L"Wnd";
}

Функция назначения:

//Operators
operator HWND() { return BaseWnd<GWnd>::GetHwnd(); }
GWnd operator=(GWnd Wnd) 
{
    GWnd tmp(Wnd);
}

Я надеюсь, что строк кода достаточно для ваших ответов. Кто-нибудь понимает проблему, с которой я сталкиваюсь? И мог бы дать мне подсказку, куда смотреть дальше?

gedysan 05 нояб. 2014, в 23:15

Источник

0

Я предполагаю, что ошибка в том, что P равен NULL, а вы разыменовываете указатель NULL? Нигде в приведенном выше коде вы на самом деле не создаете экземпляр (по крайней мере, в Init)
KirkSpaziani 05 нояб. 2014, в 21:59
0

Почему ваш оператор присваивания не возвращает значение, если вы специально указали, что он возвращает объект GWnd ? Кроме того, знаете ли вы цель конструктора копирования и оператора присваивания? Его цель - сделать копию переданного объекта - я не вижу, что происходит в вашем конструкторе копирования.
PaulMcKenzie 05 нояб. 2014, в 21:59
0

Таким образом, p равен NULL ? Читали ли вы документацию ( msdn.microsoft.com/en-us/library/zs0t7t34.aspx ) о том, как это может иметь место?
AndyG 05 нояб. 2014, в 22:01
0

Вы, вероятно, должны сделать это некопируемым и не присваиваемым; вам нужно создать окна, используя CreateWindow или другую функцию WinAPI
M.M 05 нояб. 2014, в 22:06
0

Я понимаю, что я мог бы дать слишком мало кода в приведенных выше фрагментах. @KirkSpaziani: указатель не возвращает NULL. Значение является правильным адресом экземпляра MyView (это то, что должна возвращать функция GetActiveView () в этом случае. Вызов GetID () использует :: GetDlgCtrlID (). Он использует член базового класса GWnd для получить HWND этого окна. Когда я проверяю это, это не то же самое, что HWND Active View. Я чувствую, что это вызывает проблему. Поэтому я пришел к выводу, что что-то не так с назначением или копированием экземпляров.
gedysan 06 нояб. 2014, в 08:24
0

@AndyG: смотри выше
gedysan 06 нояб. 2014, в 08:25
0

@PaulMcKenzie: Итак, вы говорите, что я забыл ключевое слово return .... Я попытался использовать конструктор copy для создания нового экземпляра GWnd с вводом экземпляра Wnd. Я также подумал, что конструктор копирования назначает членов скопированного экземпляра новому. Члены: m_hwnd (устанавливается GetHwnd () в BaseWnd, для Owner задано значение NULL, а для m_ClassName установлено значение «Wnd». Не могли бы вы намекнуть, если я принял совершенно неправильный подход к этой проблеме?
gedysan 06 нояб. 2014, в 08:27
0

@MattMcNabb: я хочу назначить указатель (p) с адресом Active View. И использовать этот указатель для доступа к функциям экземпляра активного представления. Сначала я подумал, что мне не нужны никакие копии или задания, но это тоже не сработало. Конечно, я не копирую окна, класс использует вызовы WinApi, и это прекрасно работает, кроме закомментированного вызова с указателем p
gedysan 06 нояб. 2014, в 08:33
0

Глядя снова на мой код, я понимаю, что я думаю, что GetActiveView () возвращается в указатель на тип GWnd. отладка показывает, что адрес правильный (я записал адрес указателя «this» при создании). Мой оператор присваивания: 1) не хватает возврата и 2) не принимает указатель в качестве параметра. Я думаю, что то же самое верно для конструктора копирования. Я не уверен, разрешено ли здесь использовать указатели .... Я также подумал, что не нужно будет что-то делать, потому что все, что я хочу, это указатель на существующий экземпляр.
gedysan 06 нояб. 2014, в 09:12
0

@gedysan - Во-первых, не возвращать значение из функции, которая возвращает значение, является undefined behavior , поэтому сразу же ваш оператор присваивания неверен. Во-вторых, цель конструктора копирования и оператора присваивания состоит в том, чтобы сделать точную копию переданного объекта. Вы должны просто присвоить значение m_ClassName какому-либо значению в переданном вам объекте - не думайте, что это всегда «Wnd». Наконец, вам нужно решить, что значит «сделать копию». Если нет четкого определения того, что это значит, сделайте ваш объект не подлежащим копированию.
PaulMcKenzie 06 нояб. 2014, в 09:29
0

@gedysan - я считаю, что ваша проблема заключается просто в этом - это не проблема кодирования, это проблема в том, что вы не имеете четкого представления о том, что означает копирование вашего объекта. Вы манипулируете указателями, вызываете функции и выполняете всякую «бизнес-логику». Но это не цель конструктора копирования и оператора присваивания. Это должно быть просто, но это не так. Поэтому, возможно, лучший совет для вас - «не делай этого» - просто сделай свой объект не подлежащим копированию.
PaulMcKenzie 06 нояб. 2014, в 09:37
0

@PaulMcKenzie: Я чувствую, как покраснели мои щеки .... Я займусь этим и попытаюсь сообщить о том, что я изменил. Спасибо за ответ!
gedysan 06 нояб. 2014, в 09:38
0

Я подумал, что нужно использовать его из-за следующего утверждения: конструктор копирования вызывается всякий раз, когда объект инициализируется из другого объекта того же типа, который включает инициализацию, T a = b; или T a (b) ;, где b относится к типу T. Но я думаю, что это не работает с указателями?
gedysan 06 нояб. 2014, в 09:56
0

Ваши конструкторы назначения / копирования не вызываются при назначении указателя, правильно.
KirkSpaziani 06 нояб. 2014, в 14:52

Показать ещё 12 комментариев

Теги:

c++

windows

mfc

1 ответ

Ещё вопросы

Я предполагаю, что ошибка в том, что P равен NULL, а вы разыменовываете указатель NULL? Нигде в приведенном выше коде вы на самом деле не создаете экземпляр (по крайней мере, в Init)
Почему ваш оператор присваивания не возвращает значение, если вы специально указали, что он возвращает объект GWnd ? Кроме того, знаете ли вы цель конструктора копирования и оператора присваивания? Его цель - сделать копию переданного объекта - я не вижу, что происходит в вашем конструкторе копирования.
Таким образом, p равен NULL ? Читали ли вы документацию ( msdn.microsoft.com/en-us/library/zs0t7t34.aspx ) о том, как это может иметь место?
Вы, вероятно, должны сделать это некопируемым и не присваиваемым; вам нужно создать окна, используя CreateWindow или другую функцию WinAPI
Я понимаю, что я мог бы дать слишком мало кода в приведенных выше фрагментах. @KirkSpaziani: указатель не возвращает NULL. Значение является правильным адресом экземпляра MyView (это то, что должна возвращать функция GetActiveView () в этом случае. Вызов GetID () использует :: GetDlgCtrlID (). Он использует член базового класса GWnd для получить HWND этого окна. Когда я проверяю это, это не то же самое, что HWND Active View. Я чувствую, что это вызывает проблему. Поэтому я пришел к выводу, что что-то не так с назначением или копированием экземпляров.
@PaulMcKenzie: Итак, вы говорите, что я забыл ключевое слово return .... Я попытался использовать конструктор copy для создания нового экземпляра GWnd с вводом экземпляра Wnd. Я также подумал, что конструктор копирования назначает членов скопированного экземпляра новому. Члены: m_hwnd (устанавливается GetHwnd () в BaseWnd, для Owner задано значение NULL, а для m_ClassName установлено значение «Wnd». Не могли бы вы намекнуть, если я принял совершенно неправильный подход к этой проблеме?
@MattMcNabb: я хочу назначить указатель (p) с адресом Active View. И использовать этот указатель для доступа к функциям экземпляра активного представления. Сначала я подумал, что мне не нужны никакие копии или задания, но это тоже не сработало. Конечно, я не копирую окна, класс использует вызовы WinApi, и это прекрасно работает, кроме закомментированного вызова с указателем p
Глядя снова на мой код, я понимаю, что я думаю, что GetActiveView () возвращается в указатель на тип GWnd. отладка показывает, что адрес правильный (я записал адрес указателя «this» при создании). Мой оператор присваивания: 1) не хватает возврата и 2) не принимает указатель в качестве параметра. Я думаю, что то же самое верно для конструктора копирования. Я не уверен, разрешено ли здесь использовать указатели .... Я также подумал, что не нужно будет что-то делать, потому что все, что я хочу, это указатель на существующий экземпляр.
@gedysan - Во-первых, не возвращать значение из функции, которая возвращает значение, является undefined behavior , поэтому сразу же ваш оператор присваивания неверен. Во-вторых, цель конструктора копирования и оператора присваивания состоит в том, чтобы сделать точную копию переданного объекта. Вы должны просто присвоить значение m_ClassName какому-либо значению в переданном вам объекте - не думайте, что это всегда «Wnd». Наконец, вам нужно решить, что значит «сделать копию». Если нет четкого определения того, что это значит, сделайте ваш объект не подлежащим копированию.
@gedysan - я считаю, что ваша проблема заключается просто в этом - это не проблема кодирования, это проблема в том, что вы не имеете четкого представления о том, что означает копирование вашего объекта. Вы манипулируете указателями, вызываете функции и выполняете всякую «бизнес-логику». Но это не цель конструктора копирования и оператора присваивания. Это должно быть просто, но это не так. Поэтому, возможно, лучший совет для вас - «не делай этого» - просто сделай свой объект не подлежащим копированию.
@PaulMcKenzie: Я чувствую, как покраснели мои щеки .... Я займусь этим и попытаюсь сообщить о том, что я изменил. Спасибо за ответ!
Я подумал, что нужно использовать его из-за следующего утверждения: конструктор копирования вызывается всякий раз, когда объект инициализируется из другого объекта того же типа, который включает инициализацию, T a = b; или T a (b) ;, где b относится к типу T. Но я думаю, что это не работает с указателями?
Ваши конструкторы назначения / копирования не вызываются при назначении указателя, правильно.

Samer · Answer 1 · 2014-11-05T20-00-00.000Z

0

попробуйте добавить это:

GWnd* p= (GWnd*)main.GetActiveView();

вместо этого:

GWnd* p= main.GetActiveView();

и затем используйте это

p->GetDlgCtrlID()

Samer 05 нояб. 2014, в 20:00

0

Не используйте C-стиль в C ++
Ryan Haining 05 нояб. 2014, в 22:07
0

@RyanHaining: почему бы и нет?
Deduplicator 05 нояб. 2014, в 22:13
1

@Deduplicator: Не бросайте в стиле C, это хороший совет в целом, IMO. Используйте dynamic_cast и const_cast когда это более уместно. reinterpret_cast более reinterpret_cast описывает намерение
AndyG 05 нояб. 2014, в 22:27
0

@AndyG: На самом деле вы хотите static_cast там. И я просто возразил, потому что есть много ситуаций, в которых не используется приведение в стиле c (или в стиле функции) более многословно без какой-либо компенсации.
Deduplicator 05 нояб. 2014, в 22:37
0

@Deduplicator, что вы подразумеваете под: без предоставления какой-либо компенсации
Samer 05 нояб. 2014, в 22:43
1

Что лучше: (void*)0 или static_cast<void*>(0) ? Второй в три раза длиннее и не яснее, лишний текст скрывает, что это просто нулевой указатель определенного типа. Это ситуация, когда «новые» броски более читабельны, потому что есть намного больше, чтобы прочитать ». подходит.
Deduplicator 05 нояб. 2014, в 22:46
0

@ Сэмер: Я действительно попробовал это. Я не имею никакого значения. Я думаю, что оператор присваивания должен сделать "приведение". И это не так, как я ожидал. Указатель p в обоих случаях имеет правильный тип с правильным адресом.
gedysan 06 нояб. 2014, в 08:41
0

@Deduplicator: Ой, я определенно хотел сказать это вместо const_cast . Я на самом деле действительно не люблю const_cast
AndyG 06 нояб. 2014, в 13:41

Показать ещё 6 комментариев