Chrash at UpdateData(FALSE)

[bert_r]

Try this: the thread function:

DWORD WINAPI ThreadFunc( LPVOID lpParam )
{
CWnd * pDlg = (pDlg*)lpParam;
int a = 44;
int b = 45;
for (int i=40; i<100; i++)
{
pDlg->PostMessage(WMU_SETVALUE1, (WPARAM)i,(LPARAM)b);
Sleep(10);
}
return 0;

}

in the dialog.h file

#define WMU_SETVALUE1 (WM_USER+1)

in the message map add following line:

ON\_MESSAGE(WMU\_SETVALUE1,OnSetValue1)

And add following function:

LRESULT CSurveyUpdateDlg::OnSetValue1(WPARAM wParam, LPARAM lParam)
{
setValueAlt1((int)wParam, (int)lParam);
}

[jhwurmbach]

merh wrote:

Please, can somebody show me how I can do that?

Sure. Joseph M. Newcomer can: Threads and Processes Article Series[^]

Let's think the unthinkable, let's do the undoable, let's prepare to grapple with the ineffable itself, and see if we may not eff it after all.
Douglas Adams, "Dirk Gently's Holistic Detective Agency"

[merh]

First of all thanks a lot. It dosent work. I have the following interface you mentioned that I could use pDlg->PostMessage(WMU_SETVALUE1, (WPARAM)i,(LPARAM)b); where is comming pDlg? I have got error at CWnd * pDlg = (pDlg*)lpParam; and pDlg->PostMessage(WMU_SETVALUE1, (WPARAM)i,(LPARAM)b); I have copied all files maybe you can see my problem better Thanks M.h // ThreadDlg.cpp : implementation file // #include "stdafx.h" #include "Thread.h" #include "ThreadDlg.h" #ifdef _DEBUG #define new DEBUG_NEW #undef THIS_FILE static char THIS_FILE[] = __FILE__; #endif #ifndef GLOGAL #define GLOBAL int Globalvariable; #endif class CAboutDlg : public CDialog { public: CAboutDlg(); // Dialog Data //{{AFX_DATA(CAboutDlg) enum { IDD = IDD_ABOUTBOX }; //}}AFX_DATA // ClassWizard generated virtual function overrides //{{AFX_VIRTUAL(CAboutDlg) protected: virtual void DoDataExchange(CDataExchange* pDX); // DDX/DDV support //}}AFX_VIRTUAL // Implementation protected: //{{AFX_MSG(CAboutDlg) //}}AFX_MSG DECLARE_MESSAGE_MAP() }; CAboutDlg::CAboutDlg() : CDialog(CAboutDlg::IDD) { //{{AFX_DATA_INIT(CAboutDlg) //}}AFX_DATA_INIT } void CAboutDlg::DoDataExchange(CDataExchange* pDX) { CDialog::DoDataExchange(pDX); //{{AFX_DATA_MAP(CAboutDlg) //}}AFX_DATA_MAP } BEGIN_MESSAGE_MAP(CAboutDlg, CDialog) //{{AFX_MSG_MAP(CAboutDlg) // No message handlers //}}AFX_MSG_MAP END_MESSAGE_MAP() ///////////////////////////////////////////////////////////////////////////// // CThreadDlg dialog CThreadDlg::CThreadDlg(CWnd* pParent /*=NULL*/) : CDialog(CThreadDlg::IDD, pParent) { //{{AFX_DATA_INIT(CThreadDlg) m_Value1 = _T(""); //}}AFX_DATA_INIT // Note that LoadIcon does not require a subsequent DestroyIcon in Win32 m_hIcon = AfxGetApp()->LoadIcon(IDR_MAINFRAME); } void CThreadDlg::DoDataExchange(CDataExchange* pDX) { CDialog::DoDataExchange(pDX); //{{AFX_DATA_MAP(CThreadDlg) DDX_Text(pDX, IDC_VALUE1, m_Value1); //}}AFX_DATA_MAP } BEGIN_MESSAGE_MAP(CThreadDlg, CDialog) //{{AFX_MSG_MAP(CThreadDlg) ON_WM_SYSCOMMAND() ON_WM_PAINT() ON_WM_QUERYDRAGICON() ON_MESSAGE(WMU_SETVALUE1,OnSetValue1) //}}AFX_MSG_MAP END_MESSAGE_MAP() ///////////////////////////////////////////////////////////////////////////// // CThreadDlg message handlers BOOL CThreadDlg::OnInitDialog() { CDialog::OnInitDialog(); // Add "About..." menu item to system menu. // IDM_ABOUTBOX must be in the system comman

[bert_r]

You can see I didn't test this ... :sigh: That should have been:

CWnd * pDlg = (CWnd*)lpParam;

If that doesn't work, I'll test it myself! :)

[merh]

Hi Thanks, It still dosent work. I get the following error 'CSurveyUpdateDlg' : is not a class or namespace name 'WMU_SETVALUE1' : undeclared identifier If you have a email adress I can send you my workspace which you can test it easier. For your information I have VC++ 6.0 Thanks M.H

[bert_r]

I copy-pasted the OnSetValue1 function of a project of mine, and forgot to edit the class name, so CSurveyUpdateDlg::OnSetValue1 should be CThreadDlg::OnSetValue1 And don't forget to add the function prototype in the ThreadDlg.h header file too:

afx_msg LRESULT OnSetValue1(WPARAM wParam, LPARAM lParam);

The WMU_SETVALUE1 error is caused by a missing

#include "Thread.h"

in the file where you're calling PostMessage(WMU_SETVALUE1, ...) That should do it.

[merh]

Hi, have got the crash at BOOL AFXAPI AfxAssertFailedLine(LPCSTR lpszFileName, int nLine) { #ifndef _AFX_NO_DEBUG_CRT // we remove WM_QUIT because if it is in the queue then the message box // won't display MSG msg; BOOL bQuit = PeekMessage(&msg, NULL, WM_QUIT, WM_QUIT, PM_REMOVE); BOOL bResult = _CrtDbgReport(_CRT_ASSERT, lpszFileName, nLine, NULL, NULL); if (bQuit) Thanks Mehran

[David Crow]

merh wrote:

'CSurveyUpdateDlg' : is not a class or namespace name 'WMU_SETVALUE1' : undeclared identifier

If these basic errors are troubling you, perhaps multithreaded applications should be postponed for a while. They're not a trivial concept.

"Normal is getting dressed in clothes that you buy for work and driving through traffic in a car that you are still paying for, in order to get to the job you need to pay for the clothes and the car and the house you leave vacant all day so you can afford to live in it." - Ellen Goodman

"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne

[merh]

Hi, I need how I can use PostMessage for sending the message with some parameters to main thread(window) and then how I can read the data from It. Best regards Mehran

[merh]

Thanks a lot it works. Have a nice weekend M.H

[David Crow]

See here.

"Normal is getting dressed in clothes that you buy for work and driving through traffic in a car that you are still paying for, in order to get to the job you need to pay for the clothes and the car and the house you leave vacant all day so you can afford to live in it." - Ellen Goodman

"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne