Member function not declared

[Ryan Jantz]

What is missing in this code?

I get a C2509 compile time error saying:" 'OnCopyData': member function not declared in 'CmfclDlg'"

BOOL CmfclDlg::OnCopyData( CWnd *pWnd, COPYDATASTRUCT *pReq)

{
	

	char				cMsg[256]	= { 0 };
	CString				cResp;
	COPYDATASTRUCT		resp;

	if ( pReq )
	{
		if( pReq->cbData && pReq->lpData)
		{
			strncpy( cMsg, (char*)pReq->lpData, pReq->cbData >= sizeof( cMsg ) ? sizeof ( cMsg ) - 1 : pReq->cbData );
			SetDlgItemText( IDC_REQUEST, cMsg);
		}
		if ( pReq->dwData )
		{
			GetDlgItemText( IDC_RESPONSE, cResp);
			strcpy( cMsg, cResp.GetBuffer( sizeof ( cMsg )));

			resp.dwData = (DWORD)GetSafeHwnd();
			resp.cbData = strlen( cMsg );
			resp.lpData = (PVOID)&cMsg[0];

			::SendMessage( (HWND)(pReq->dwData), WM_COPYDATA, (WPARAM)GetSafeHwnd(), (LPARAM)&resp );
			SetDlgItemText( IDC_STATUS, "Response sent.");

		}
		else
		{
			SetDlgItemText( IDC_STATUS, "No dwData found.");
		}
	}
	else
	{
		SetDlgItemText( IDC_STATUS, "No COPYDATASTRUCT found.");
	}
	return( TRUE );
}

I got this off of davidjwalling.com to try to help me understand how MFC and .NET interacted.

[Zhuge]

Is OnCopyData() declared in CmfclDlg? You should check the CmfclDlg class and see.

[vlad from moscow]

All class members shall be declared in class definition.

[Ryan Jantz]

Here is the whole cpp file. I am brand new to MFC and do not know where I should be looking.

// mfclDlg.cpp : implementation file
//

#include "stdafx.h"
#include "mfcl.h"
#include "mfclDlg.h"

#ifdef _DEBUG
#define new DEBUG_NEW
#endif


// CAboutDlg dialog used for App About

class CAboutDlg : public CDialog
{
public:
	CAboutDlg();

// Dialog Data
	enum { IDD = IDD_ABOUTBOX };

	protected:
	virtual void DoDataExchange(CDataExchange* pDX);    // DDX/DDV support

// Implementation
protected:
	DECLARE_MESSAGE_MAP()
};

CAboutDlg::CAboutDlg() : CDialog(CAboutDlg::IDD)
{
}

void CAboutDlg::DoDataExchange(CDataExchange* pDX)
{
	CDialog::DoDataExchange(pDX);
}

BEGIN_MESSAGE_MAP(CAboutDlg, CDialog)
END_MESSAGE_MAP()


// CmfclDlg dialog




CmfclDlg::CmfclDlg(CWnd* pParent /*=NULL*/)
	: CDialog(CmfclDlg::IDD, pParent)
{
	m_hIcon = AfxGetApp()->LoadIcon(IDR_MAINFRAME);
}

void CmfclDlg::DoDataExchange(CDataExchange* pDX)
{
	CDialog::DoDataExchange(pDX);
}

BEGIN_MESSAGE_MAP(CmfclDlg, CDialog)
	ON_WM_SYSCOMMAND()
	ON_WM_PAINT()
	ON_WM_QUERYDRAGICON()
	//}}AFX_MSG_MAP
	ON_BN_CLICKED(IDC_SEND1, &CmfclDlg::OnBnClickedSend1)
	ON_BN_CLICKED(IDC_CANCEL1, &CmfclDlg::OnBnClickedCancel1)
END_MESSAGE_MAP()


// CmfclDlg message handlers

#define SetDlgItemText
BOOL CmfclDlg::OnInitDialog()
{
	CDialog::OnInitDialog();

	// Add "About..." menu item to system menu.

	// IDM_ABOUTBOX must be in the system command range.
	ASSERT((IDM_ABOUTBOX & 0xFFF0) == IDM_ABOUTBOX);
	ASSERT(IDM_ABOUTBOX < 0xF000);

	CMenu* pSysMenu = GetSystemMenu(FALSE);
	if (pSysMenu != NULL)
	{
		CString strAboutMenu;
		strAboutMenu.LoadString(IDS_ABOUTBOX);
		if (!strAboutMenu.IsEmpty())
		{
			pSysMenu->AppendMenu(MF_SEPARATOR);
			pSysMenu->AppendMenu(MF_STRING, IDM_ABOUTBOX, strAboutMenu);
		}
	}

	// Set the icon for this dialog.  The framework does this automatically
	//  when the application's main window is not a dialog
	SetIcon(m_hIcon, TRUE);			// Set big icon
	SetIcon(m_hIcon, FALSE);		// Set small icon

	// TODO: Add extra initialization here

	SetDlgItemText( IDC_STATUS, "Enter data for response.");
	SetDlgItemText( IDC_RESPONSE, "ACK");

	return TRUE;  // return TRUE  unless you set the focus to a control
}

void CmfclDlg::OnSysCommand(UINT nID, LPARAM lParam)
{
	if ((nID & 0xFFF0) == IDM_ABOUTBOX)
	{
		CAboutDlg dlgAbout;
		dlgAbout.DoModal();
	}
	else
	{
		CDialog::OnSysCommand(nID, lParam);
	}
}

// If you add a minimize button to your dialog, you will need the code below
//  to draw the icon.  For MFC applications using the document/view model,
//  this is automatically done for you by the framework.

void CmfclDlg::OnPaint()
{
	if (IsIconic())
	{
		CPaintDC dc(this); // device context for painting

		SendMessage(WM_ICONERASEBKGND, reinterpret_cast<WPARAM>(dc.GetSafeHdc()), 0);

		// Center icon in client rectangle
		int cxIcon = GetSystemMetrics(SM_CXICON);
		int cyIcon = GetSystemMetrics(SM_CYICON);
		CRect rect;
		GetClientRect(&rect);
		int x = (rect.Width() - cxIcon + 1) / 2;
		int y = (rect.Height() - cyIcon + 1) / 2;

		// Draw the icon
		dc.DrawIcon(x, y, m_hIcon);
	}
	else
	{
		CDialog::OnPaint();
	}
}

// The system calls this function to obtain the cursor to display while the user drags
//  the minimized window.
HCURSOR CmfclDlg::OnQueryDragIcon()
{
	return static_cast<HCURSOR>(m_hIcon);
}



void CmfclDlg::OnBnClickedSend1()
{
	// TODO: Add your control notification handler code here
	COPYDATASTRUCT cds;
	SetDlgItemText(IDC_STATUS,"Sending Message ...");
	cds.dwData = 1;
	cds.cbData = 5;
	cds.lpData = (LPVOID)"Test\0";

	SendMessage ( WM_COPYDATA, (WPARAM)GetSafeHwnd(), (LPARAM)&cds);
	SetDlgItemText( IDC_STATUS,"Message Sent.");
}

void CmfclDlg::OnBnClickedCancel1()
{
	// TODO: Add your control notification handler code here
	SetDlgItemText(IDC_REQUEST,"");
	SetDlgItemText(IDC_RESPONSE,"");
	SetDlgItemText(IDC_STATUS,"Enter data for response.");
	
	SendMessage( WM_SETFOCUS, IDC_RESPONSE, 0L);

}

BOOL CmfclDlg::OnCopyData( CWnd *pWnd, COPYDATASTRUCT *pReq)

{
	

	char				cMsg[256]	= { 0 };
	CString				cResp;
	COPYDATASTRUCT		resp;

	if ( pReq )
	{
		if( pReq->cbData && pReq->lpData)
		{
			strncpy( cMsg, (char*)pReq->lpData, pReq->cbData >= sizeof( cMsg ) ? sizeof ( cMsg ) - 1 : pReq->cbData );
			SetDlgItemText( IDC_REQUEST, cMsg);
		}
		if ( pReq->dwData )
		{
			GetDlgItemText( IDC_RESPONSE, cResp);
			strcpy( cMsg, cResp.GetBuffer( sizeof ( cMsg )));

			resp.dwData = (DWORD)GetSafeHwnd();
			resp.cbData = strlen( cMsg );
			resp.lpData = (PVOID)&cMsg[0];

			::SendMessage( (HWND)(pReq->dwData), WM_COPYDATA, (WPARAM)GetSafeHwnd(), (LPARAM)&resp );
			SetDlgItemText( IDC_STATUS, "Response sent.");

		}
		else
		{
			SetDlgItemText( IDC_STATUS, "No dwData found.");
		}
	}
	else
	{
		SetDlgItemText( IDC_STATUS, "No COPYDATASTRUCT found.");
	}
	return( TRUE );
}

Thanks for your answers already though!

[Volatile Pulse]

Ryan Jantz wrote:

I get a C2509 compile time error saying:" 'OnCopyData': member function not declared in 'CmfclDlg'"

This basically tells you that the OnCopyData doesn't exist as a member of CmfclDlg. There is a way to tell if it exists the way you have it by opening up the class declaration which should be located in the mfclDlg.h header. This should contain all of the class members that are valid. It's possible the OnCopyData exists in the class as a function, but the call could be incorrect. Is it possible that you're using an old version of the header file? Did you get the header file from the same place you got the example from?

I know that MFC is a Microsoft thing, and that they have a wizard for the console projects, but if you happened to copy your code from somewhere, it's possible it uses an older/updated version of mfclDlg.h than the one that is supplied with your compiler.

My only suggestion is to try to narrow down the possibilities. Make sure the files came from the same place and haven't been updated since.

[Ryan Jantz]

I thought that OnCopyData was part of the MFC initial setup. Intellesense provided that function when I was typing. That is why I am not sure where to look. OnCopyData is part of <afxwin.h> that is included in the <stdafx.h>. Any other ideas?

[Volatile Pulse]

I'm sorry, a lot of us don't use the MFC since it's a mix between console apps and the windows gui, a microsoft specific implementation, and only supported by the MSVS suites.

[Ryan Jantz]

@Volatile Pulse,

Your idea about looking in the header was correct. I added

BOOL OnCopyData( CWnd *pWnd, COPYDATASTRUCT *pReq);

in the .h file and that worked. Thanks for your help.

[Volatile Pulse]

That's not quite the way to do it...but if you have no issues with it, it's quite possible something got lost between one of the versions? But it's really not supposed to work correctly just by adding one line like that.

Glad you got it to work regardless though.