Go Premium for a chance to win a PS4. Enter to Win

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 1080
  • Last Modified:

calling "postmessage()" in C++ mfc

I'm use "Visual Studio 2005".  I'm  use C++  mfc (microsoft  foundation class).  I made a pointer  bottom of the  file, you can see it here.

};==========================>end of test11Dlg.h file  
extern Ctest11Dlg *g_MainDlg;

I use  pointer "g_MainDlg" to call "postmessage()" , you can see it below.

I want to know the  solution and  reason to the cause.

I mentioned the output.
 

#ifndef PORTSIP_CALLBACKEVENT_hxx
#define PORTSIP_CALLBACKEVENT_hxx
 
 
#include "stdafx.h"
#include "test11Dlg.h"
//#include <iostream.h>
 
 
 
// for SIP
 
struct SIPEvent : public SIPCallbackEvent 
{
	
 
	virtual void onCommand(SIPCallbackCommand * command)
	{
		if (!command)
		{
			return;
		}
 
 
 
		if (!g_MainDlg->PostMessage(WM_SIPEVENT, (WPARAM)command, 0))
		{
			
			delete command;
			command = NULL;
			
		}
	
 
 
 
 
 
	}
	
 
};
 
 
#endif

Open in new window

0
shaf81
Asked:
shaf81
1 Solution
 
waysideCommented:
> extern Ctest11Dlg *g_MainDlg;

You've declared an external reference to a global variable. Somewhere in your code you have to allocate storage for the global.

So in one of your source files you need to have a line that looks like

Ctest11Dlg *g_MainDlg;

0
 
itsmeandnobodyelseCommented:
You better would make the g_MainDlg a static member fo your dialog class:

// test11Dlg.h
...
class Ctest11Dlg : public CDialog
{
public:
       static Ctest11Dlg *g_MainDlg;
       ....
};


// test11Dlg.cpp

....
// somewhere (outside of functions)
Ctest11Dlg* Ctest11Dlg::g_MainDlg = NULL;   // always make an initialization


// in constructor

Ctest11Dlg::Ctest11Dlg(CWnd* pParent, IDD, ....)
: CDialog(pParent)
{
      ....
      Ctest11Dlg::g_MainDlg = this;          // here you set the pointer
}


Then your code changes to

          if (Ctest11Dlg::g_MainDlg != NULL && 
             !Ctest11Dlg::g_MainDlg->PostMessage(WM_SIPEVENT, (WPARAM)command, 0))
          {

0
 
suryaxchangeCommented:
Hi,

extern variable do not allocate memory for the specified. They only link to the existing storage.
In your case

extern Ctest11Dlg *g_MainDlg;

g_MainDlg pointer is not allocated but tries to link to the global pointer to Ctest11Dlg which is not created.

if the storage is already created with the pointer of the same name somewhere in the code externally then the extern keyword can link to this pointer, for this reason the linking got failed in your case.

e.g., create a global integer pointer as follows in any .cpp file

int g_nIntVal = 9;

In any other file you refernece the same global variable in this way
extern int g_nIntVal;

here the name and datatype of the variable need to be same. Here the second declaration will not create a storage but  try to link to the existing storage.

-Surya





0
 
shaf81Author Commented:
Thank you very much for your successful effort. Your  answer is perfectly correct.
0
 
shaf81Author Commented:
Hello Surya,

Thank you very much.
0

Featured Post

VIDEO: THE CONCERTO CLOUD FOR HEALTHCARE

Modern healthcare requires a modern cloud. View this brief video to understand how the Concerto Cloud for Healthcare can help your organization.

Tackle projects and never again get stuck behind a technical roadblock.
Join Now