C / C++ / MFC

12-May-03 23:24

Ok, I know this is really a back to basics question, but I would really appreciate some help.

I am developing a MFC application (mainly to teach myself the whole thing) and I am trying to show a modal dialog box when I click a toolbar button. (SDI program). I get response when I click the button, but the dialog show various errors when I try diffrent approaches I found in source codes on the net. Sometimes it hangs, sometimes it just doesn't show up, but hangs the rest of the app (because it's modal, yes I know). I haven't gotten to the point where I can actually interact with the dialog, as in pushing the command buttons, writing in the edit boxes and such.

The dialog box is a dialog resource template with it's own class CDataEditor, derived from CDialog. In my code, I am mostly trying DoModal() but as you understand, it's not going very well.

Hope you can get me out of this.... Big Grin | :-D

jhwurmbach12-May-03 23:35

12-May-03 23:35

OldViking wrote:
Hope you can get me out of this....

How? You did not show what you were doing when your call to DoModal() gave errors.

My opinions may have changed, but not the fact that I am right.

Jorgen E.12-May-03 23:40

12-May-03 23:40

Hmm... My bad. Just forgot about it.

void CMainFrame::OnDataedit()
{
// Displays the data Edior dialog box
CDataEditor * dlg;
dlg = new CDataEditor;
dlg->DoModal();
}

jhwurmbach12-May-03 23:55

12-May-03 23:55

Looks OK. What errors do you get?

Can you call your class on the stack instead of on the heap?

My opinions may have changed, but not the fact that I am right.

Jorgen E.13-May-03 0:04

13-May-03 0:04

I don't get any errors. That's the thing. When I click the toolbar button, the whole programs hangs and I have to end it with ctrl-alt-del.

I tried to use Create( IDD_TEMPLATEID, NULL) instead of DoModal(). The only thing I accomplished was being able to shut down the program in a normal way. The only thing that showed up was the cancel button in the dialog, somehow.

For your second question...remember that I'm a newbie. I don't understand what that means Unsure | :~

------------------------------
The great successful men of the world have used their imagination...they think ahead and create their mental picture in all its details, filling in here, adding a little there, altering this a bit and that a bit, but steadily building-steadily buildings. (Robert Collier)

jhwurmbach13-May-03 1:08

13-May-03 1:08

Answer to second question:

void CMainFrame::OnDataedit() 
{
// Displays the data Edior dialog box
CDataEditor dlg;
dlg.DoModal();
}

For the first question - IDD_TEMPLATEID is a valid resource (one that is actually visible with the resource editor)?

My opinions may have changed, but not the fact that I am right.

Jorgen E.13-May-03 1:19

13-May-03 1:19

Well, for the code you gave me... It showed the dialog in a split second, before it disappeared again, and the whole program freezed once again.

The IDD_TEMPLATEID is a valid resource, yes. I double-checked the properties and it's set to visible and disabled is NOT checked. It has the child-style set.

I'm using VC++ 6.0 if that has anything to say.

------------------------------
The great successful men of the world have used their imagination...they think ahead and create their mental picture in all its details, filling in here, adding a little there, altering this a bit and that a bit, but steadily building-steadily buildings. (Robert Collier)

David Crow13-May-03 2:32

David Crow

13-May-03 2:32

Do you have any odd code in the dialog's OnInitDialog() method (that might be causing it to go away)?

Jorgen E.13-May-03 2:39

13-May-03 2:39

Well, no. The dialog only has a ClassWizard-generated class connected to it. I haven't edited that class by hand in any way. In fact, there is no OnInitDialog string in any of the source code. I am assuming that means that MFC does all the required "standard" work?

------------------------------
The great successful men of the world have used their imagination...they think ahead and create their mental picture in all its details, filling in here, adding a little there, altering this a bit and that a bit, but steadily building-steadily buildings. (Robert Collier)

jhwurmbach13-May-03 2:51

13-May-03 2:51

Your CDataEdit is derived from CDialog? That is, it has

class CDataEdit : public CDialog
{
...stuff...
}

written in its header file?

My opinions may have changed, but not the fact that I am right.

Jorgen E.13-May-03 3:07

13-May-03 3:07

Yes, it's derived from CDialog.

If you want and got the time, I can give you the source code to see if you find anything wrong...

------------------------------
The great successful men of the world have used their imagination...they think ahead and create their mental picture in all its details, filling in here, adding a little there, altering this a bit and that a bit, but steadily building-steadily buildings. (Robert Collier)

Chris Richardson13-May-03 16:44

Chris Richardson

13-May-03 16:44

OldViking wrote:
It has the child-style set.

This is the problem I bet. If you are trying to show a modal dialog box, don't make it a Child, make it a Popup style. If your point was to embed the dialog on a different window, that's a whole different story.

So, for now, go change that Child style to Popup, and you don't need to create the dialog on the heap if it's a modal dialog. Just create it on the stack:

C_ICantRememberTheClassName_Dlg dlg;
dlg.DoModal();

Chris Richardson
Terrain Software