Ah, I see now that you are deleting the pointer in the message handler.
Could it go out of scope even if I use:
CString * ptrToData = new CString(buffer) ;
and passed the pointer to the message handler??
I was under the impression that this would stay on the heap until I
deleted it. And, in this case, the "new" CString is deleted in the message
handler function. If it can still go out of scope in this situation, I
would need to create a global variable/array to hold the messages.
Do I need to be concerned about it going out of scope and what would you
suggest?
Thanks,
KW
"Tom Serface" <tom@nospam.camaswood.com> wrote in message
news:D6EE5666-61C3-4148-9BD3-692BE41E8627@microsoft.com...
I would probably use a function to set the msg window, but this would work
OK. Since you are using PostMessage you may have trouble with the data
since it may go out of scope before it's actually used. You may want to
use SendMessage instead.
You will also want to make sure the window is a valid window before
posting to it.
Tom
"Discovery" <phikaa@126.com> wrote in message
news:%23r4J%23u3oJHA.5228@TK2MSFTNGP02.phx.gbl...
Mark~
"KW" <KW@there.com>
????????????:OZcqxPmoJHA.1252@TK2MSFTNGP03.phx.gbl...
I have a MDI application with several child windows and a TCPIP server.
I want to pass the handle (of one child) to a server socket class so it
can post a message to the child window when data is received. When the
data is received by the child window, a listbox will display the data.
I have it working, but I wonder if there is a better way of passing (or
getting) the handle. I have to pass it twice to get it into the
OnReceive() Handler of CServer.cpp.
// Click a button and start the listener - couldn't get it to work in
constructor (handle not created yet?) - "SocketWindow" is UI class
void SocketWindow::OnButton1()
{
sockServ = new MySocket(CWnd::m_hWnd); //Is this the best way to
get/pass the handle?
sockServ->Create(1001);
sockServ->Listen();
}
void SocketWindow::OnSockMSG(UINT wParam, long lParam)
{
CString * ptr = (CString *)wParam;
m_ListBoxAdder.AddString(*ptr);
delete(ptr);
}
// These 4 lines are from MySocket.h
public:
MySocket(HWND);
virtual ~MySocket();
CServer client;
HWND ChildWindowHandle;
// Constructor MySocket.cpp
MySocket::MySocket(HWND MyHandle)
{
ChildWindowHandle = MyHandle;
}
// MySocket.cpp
void MySocket::OnAccept(int nErrorCode)
{
if (nErrorCode == 0)
{
this->Accept(client);
// IS THIS A PROBLEM? "client" is declared in CServer.h
client.HandleToChld = ChildWindowHandle ;
}
}
// CServer.cpp
void CServer::OnReceive(int nErrorCode)
{
char buffer[1024] ;
memset(buffer, 0, sizeof(buffer)) ;
this->Receive(buffer, sizeof(buffer)) ;
CString result(buffer);
CString * ptrToData = new CString(result) ;
// "WM_SCKTMSG_RECV" is defined in stdAfx.h (#define
WM_SCKTMSG_RECV WM_USER + 3), executes "OnSockMSG" above
::PostMessage(HandleToChld, WM_SCKTMSG_RECV, (WPARAM)ptrToData,
(LPARAM)0);
//CAsyncSocket::OnReceive(nErrorCode);
}
Although this code currently works, it does not feel right (passing the
handle in the constructor??).
There has to be a better way of doing this. Any suggestions?
Thanks,
KW