C / C++ / MFC

19-Jun-03 4:38

Use the following functions from <ctype.h>
isalpha
isdigit
ispunct
isspace

Re: help in C++

David Crow19-Jun-03 10:19

David Crow

19-Jun-03 10:19

CString strLine;
CStdioFile file(...);
int nLineCount = 0,
nAlphaCount = 0;
while (file.ReadString(strLine) == TRUE)
{
nLineCount++;

for (x = 0; x < strLine.GetLength(); x++)
{
if (isalpha(strLine[x]))
{
nAlphaCount++;
}
else if (isdigit(strLine[x]))
{
nDigitCount++;
}
...
}
}

// to count characters (or count them above as each character is analyzed)
int nCharCount = file.GetLength();

Thread

Gandilf19-Jun-03 4:09

Gandilf

19-Jun-03 4:09

Hi everyone,

I have a little question about thread. I have read the faq and managed to program threads but now I'm facing a newbie challenge : I need to access to my dialog controls and the Document. I copy this code from this forum to access the Document from anywhere in my program.

//Declaration
static CTestSDIDoc * GetDoc();

//Implementation
CTestSDIDoc * CTestSDIDoc::GetDoc()
{
CFrameWnd * pFrame = (CFrameWnd *)(AfxGetApp()->m_pMainWnd);
return (CTestSDIDoc *) pFrame->GetActiveDocument();
}

In my dialog (my own progress window with static text, a progress bar, 6 bitmaps and an OK button), I want to show a green check mark when a process has done is job. The process is created with CreateProcess method. The CreateProcess is writed in the TreadFunc of my dialog class. Before calling createprocess method, I must do some initialization in the Document so I must do this :

metprof *metProfPtr = static_cast<metprof *="">(pvParam);
CTestSDIDoc *pDoc = static_cast<ctestsdidoc *="">(pvParam);

pDoc=CTestSDIDoc::GetDoc();

The bold text code works everywhere except in the TreadFunc method (I've got an assertion when running in debug mode). I want to know what Am I doing wrong ?

Here my complete ThreadFunc function:

UINT metprof::ThreadFunc(LPVOID pvParam)
{

metprof *metProfPtr = static_cast<metprof *="">(pvParam);
CTestSDIDoc *pDoc = static_cast<ctestsdidoc *="">(pvParam);

CString workingDirectory,strTemp,appArguments,strTemp2;

pDoc=CTestSDIDoc::GetDoc();
TRACE("vis: %d\n",pDoc->m_VIS);

workingDirectory=pDoc->m_modulesDirectory;
workingDirectory.Insert(workingDirectory.GetLength(),"ATMOSPHERIC\\\\");
pDoc->m_SL_COND=1;
appArguments=pDoc->m_dosPrompt;
pDoc->createMetDataFile();
pDoc->createMetProfInputFiles();

STARTUPINFO siStartupInfo;
PROCESS_INFORMATION piProcessInfo;

memset(&siStartupInfo, 0, sizeof(siStartupInfo));
memset(&piProcessInfo, 0, sizeof(piProcessInfo));

siStartupInfo.cb = sizeof(siStartupInfo);
strTemp2=pDoc->m_binDirectory;
strTemp2.Replace("\\\\","\\");
strTemp2.Delete(strTemp2.GetLength()-1,1);
strTemp.Format(" /c \"metprof.exe < metprof.rep -L -P \"%s\"\"",strTemp2);
appArguments.Insert(appArguments.GetLength(),strTemp);

char *str = appArguments.GetBuffer(appArguments.GetLength());

if(CreateProcess(NULL, // Application name
str,//"cmd /c \"metprof.exe < metprof.rep -L\"",// Application arguments
0,
0,
FALSE,
CREATE_NO_WINDOW,
0,
workingDirectory,//"C:\\Program Files\\Irblempkg\\IRBLEM4.1\\modules\\atmospheric\\", // Working directory
&siStartupInfo,
&piProcessInfo)){}

::WaitForSingleObject(piProcessInfo.hProcess,INFINITE);
metProfPtr->GetDlgItem(IDC_CHECKMETEO)->ShowWindow(TRUE);
pDoc->moveMetProfOutputFiles();

return 0;
}

I think I'm missing something with my code but I don't know what.

Thanks for your help,

Rene

AlexO19-Jun-03 4:18

19-Jun-03 4:18

Yeah, you are missing the bold text Smile | :)

. Which part of the code does not work?

AlexO19-Jun-03 4:22

19-Jun-03 4:22

Could you explain little bit more what is the following code does?

metprof *metProfPtr = static_cast(pvParam);
CTestSDIDoc *pDoc = static_cast(pvParam);

P.S. I assume "<" and ">" are lost due to the editor.

Gandilf19-Jun-03 4:48

Gandilf

19-Jun-03 4:48

Hi AlexO,

metprof *metProfPtr = static_cast<<metprof*>>(pvParam);
CTestSDIDoc *pDoc = static_cast<<ctestsdidoc*>>(pvParam);

These are pointers to metProf dialog class to access dialog controls like editBox etc...
The second one is a pointer too but to the Document.

If I run my program, I got an assertion but if I click on ignore button the program continues to work correctly.

Thanks for your help,

Rene

AlexO19-Jun-03 5:35

19-Jun-03 5:35

that is what I thought ... But it is the same pointer is it not?
As for pointers, just an idea, would it not be safer to pass HWND of the dialog and use PostMessage for notification from the worker thread?

Ryan Binns19-Jun-03 4:22

Ryan Binns

19-Jun-03 4:22

You can't use MFC windows in multiple threads. Since CFrameWnd::GetActiveDocument() is MFC specific (not a Win32 function) you're pretty much screwed. You'll need to either pass the document as a parameter to your thread function, or find another way of doing it. MFC is just no good for applications where multiple threads manipulate the GUI.

Ryan

Being little and getting pushed around by big guys all my life I guess I compensate by pushing electrons and holes around. What a bully I am, but I do enjoy making subatomic particles hop at my bidding - Roger Wright (2^nd April 2003, The Lounge)

Punctuality is only a virtue for those who aren't smart enough to think of good excuses for being late - John Nichol "Point Of Impact"

Jonathan Craig19-Jun-03 5:09

Jonathan Craig

19-Jun-03 5:09

Pass all the data the thread will need as the parameter:

struct ThreadData
{
    CTestSDIDoc *pDoc;
    metprof *metProfPtr;
    .
    . //Your data here.
    .
    HWND hWnd;  //Pass CWnd objects by HWND.
};
 
.
.
.
ThreadData *td = new ThreadData
 
//Do this before starting thread.
td.pDoc = CTestSDIDoc::GetDoc();
td.metProfPtr = ... //Set this too.
 
//Thread function must delete td!
CWinThread *pThread = AfxBeginThread(ThreadProc, (LPVOID)td);
.
.
.
 
UINT ThreadProc(LPVOID lpParam)
{
    ThreadData *td = (ThreadData *)lpParam;
    .
    . //Your code here.
    .
    delete td; //Delete memory so no leaks.
    return 0;
}

Also, you don't need a thread just to create a proccess.

Hope this helps... Smile | :)

Jonathan Craig
www.mcw-tech.com

Jonathan Craig20-Jun-03 3:37

Jonathan Craig

20-Jun-03 3:37

Gandilf emailed me:
Your struct trick works perfectly but now I got an exception 0xC0000005 about metProfPtr. metProfPtr is a pointer to the current dialog (my own progress bar and bitmap, etc...). I need it to unhide one different bitmap at each process completion (total of processes: 6). When you talk about initializing pointer metprofPtr like td.pDoc = CTestSDIDoc::GetDoc(); for pDoc, what can I do ? (td.metProfPtr = 0 ?) Maybe you have another suggestion.

Thanks a lot,

Rene

There are two different ways to do it.

1.) Add the pointer to the dialog to the ThreadData struct.

struct ThreadData 
{ 
 . 
 CMyDialog *pDlg; 
 . 
}; 
 
td.pDlg = this;

This may or may not cause problems when calling methods on td->pDlg.

2.) If you know which object you wish to hide or show, you can pass a HWND handle to it. If your bitmap object is a CStatic.

struct ThreadData 
{ 
 . 
 HWND hWnd; 
 . 
}; 
 
td.hWnd = m_bitmap.GetSafeHwnd();

In your thread you would call.

::ShowWindow(td->hWnd, SW_HIDE);

::ShowWindow(td->hWnd, SW_SHOW);

This is the correct way to pass window objects to a thread.

Good Luck Smile | :)

Jonathan Craig
www.mcw-tech.com

Brian Delahunty19-Jun-03 7:57

Brian Delahunty

19-Jun-03 7:57

Have a look at Jonathan Craig's post above this one. If that doesn't help drop me an email [click on the email button at the bottom of this post] and I'll help you out.

Regards,
Brian Dela Smile | :)

David Crow19-Jun-03 10:22

David Crow

19-Jun-03 10:22

When the secondary thread finishes up a work unit, it simply needs to post a message to the primary thread (i.e., the one that owns the UI).

CSpinButtonCtrl

#realJSOP19-Jun-03 3:54

#realJSOP

19-Jun-03 3:54

I have a CSpinButtonCtrl control in a dialog box, and a corresponding CEdit control.

If I set the spin button to be "attached" to the CEdit control, the program asserts if I defined a variable associated with the attached CEdit control.

// Dialog Data
	//{{AFX_DATA(ViaSatChannelOptionsDlg)
	enum { IDD = IDD_MYDIALOG };
	CSpinButtonCtrl	m_ctrlSpinner;
	CEdit m_ctrlEdit; // defining this causes my program to assert in wincore.cpp
	//}}AFX_DATA

I looked all over MSDN and didn't see anything about this. Anyone have any clues/experience with this?

------- signature starts

"...the staggering layers of obscenity in your statement make it a work of art on so many levels." - Jason Jystad, 10/26/2001

Please review the Legal Disclaimer in my bio.

------- signature ends

David Chamberlain19-Jun-03 4:05

David Chamberlain

19-Jun-03 4:05

I've had to do this exact thing very recently, and I didn't have any problems. I have both a spin control and and edit control declared very much like you have them. I don't see any reason for that to assert.

I did find, through a roundabout way, that the edit control needs to follow the spin control in the tabbing order on the dialog. I'm not sure if this documented or not, but I hadn't seen it before. But, apparently, this is how the "attachment" works, by the edit immediately following the spin in tabbing order.

You might check that, and you might make sure exactly why the assertion is failing. I suspect the cause of the assertion isn't simply because of the edit declaration.

Dave

"You can say that again." -- Dept. of Redundancy Dept.

#realJSOP19-Jun-03 4:09

#realJSOP

19-Jun-03 4:09

The spin control has to immediately follow the associated edit control. If I set "autobuddy" to true, the program asserts. If I set it to false, the program, doesn't assert, but I have to handle updting the field myself.

------- signature starts

"...the staggering layers of obscenity in your statement make it a work of art on so many levels." - Jason Jystad, 10/26/2001

Please review the Legal Disclaimer in my bio.

------- signature ends

David Chamberlain19-Jun-03 4:19

David Chamberlain

19-Jun-03 4:19

In my code, I have the DoDataExchange with both the spin and edit controls included with

DDX_Control (pDX, IDC_SpinControlName, var_name1)<br />
DDX_Control (pDX, IDC_EditControlName, var_name2)

but then also have a member variable in the class that creates the dialog for the edit control value. The DoDataExchange goes to/from the controls to the control variables, but then the owning class needs to go from the conrtol variable to a member variable to be used further (?).

Dave

"You can say that again." -- Dept. of Redundancy Dept.

#realJSOP19-Jun-03 4:35

#realJSOP

19-Jun-03 4:35

That second part made no sense to me at all. Smile | :)

In order to do keep the program from asserting, I have to turn "autobuddy" off in the resource editor, and then call the spin button control "SetBuddy()" function to associate it with the desired edit control.

BTW, I guess I should mention that this is in PocketPC 2002... Smile | :)

------- signature starts

"...the staggering layers of obscenity in your statement make it a work of art on so many levels." - Jason Jystad, 10/26/2001

Please review the Legal Disclaimer in my bio.

------- signature ends

David Chamberlain19-Jun-03 4:56

David Chamberlain

19-Jun-03 4:56

John Simmons / outlaw programmer wrote:
BTW, I guess I should mention that this is in PocketPC 2002...

Oh, great!! Next you're going to tell me you're using VB!! OMG | :OMG:

I don't have any calls to SetBuddy(), but in the .rc file, the line for the spin control is

CONTROL "Spin1",IDC_ELEV_SPIN,"msctls_updown32", UDS_SETBUDDYINT | UDS_AUTOBUDDY | UDS_ARROWKEYS | UDS_NOTHOUSANDS, 143, 212, 11, 14

The magic of dialog editors and wizards, I guess. To me, the connection between spin and edit controls comes from the tab order, and the use of UDS_AUTOBUDDY establishes the connection.

I know this probably isn't much help, and I'm sure it will turn out to be something pretty simple, but getting all those little details worked out is always a PITA.

Good luck.

Dave

"You can say that again." -- Dept. of Redundancy Dept.

Popup Menu

modular19-Jun-03 3:20

modular

19-Jun-03 3:20

I display popup menu when right clicked on list control. I do not want it in main menu. How to hide it.
I want to display toolbar on dialog box whose buttons map to popup menu . How to do this?

Re: Popup Menu

AlexO19-Jun-03 3:34