window wont show on menu command!?

ok.........the title explains it all.....on a menu command, i can't get a widnow to display...heres the code:

mainprog.cpp (section):

Code:

---

case WM_COMMAND:
switch(LOWORD(wParam))
{
case ID_EXIT:
if(MessageBox (NULL, "Are you sure you want to close the program?\nAll un-saved data will be lost." , "Exiting...", 0 + MB_YESNO + MB_DEFBUTTON2 + MB_ICONQUESTION)==IDYES)
{
PostQuitMessage( 0 );
}
break;

case ID_LVLEDIT:
BOOL LvlEdit(HWND hWnd, HWND level_edit, HINSTANCE hInstance );
break;
return 0;
}

---

the window create function is in the 'ID_LVLEDIT' case...heres the function code:

windowcall.h:

Code:

---

BOOL LvlEdit(HWND hWnd, HWND level_edit, HINSTANCE hInstance )
{
level_edit = CreateWindow(
"LVLEDIT", "Level Editor",
WS_CHILD | WS_CAPTION | WS_VISIBLE,
200, 0, 592, 516,
hWnd, NULL, hInstance, NULL );
}

---

thanks ahead of time
-psychopath

OK, you seem to be confused between the syntax for a function prototype and the syntax for a function call.

Code:

---

BOOL LvlEdit(HWND hWnd, HWND level_edit, HINSTANCE hInstance );

---

This is a function prototype. A function prototype declares a function and the types of arguments it takes. You will typically find function prototypes in header files or at the top of the file if the function is only used in the current file. A function prototype does not call the function.
--

Code:

---

LvlEdit(hWnd, level_edit, hInstance);

---

This is a function call. It executes that function. To call a function in C++, there must be a function prototype or declaration above that point. For this reason, function prototypes typically go at the top of the file or in header files.

did that, but now it says that hWnd, level_edit, and hInstance are undeclared....?

does anyone have any idea whats wrong???

>>does anyone have any idea whats wrong???<<

Please don't bump your threads. Most people read through the bulk of stuff and will help out where and when they feel like it. :)

hWnd seems to be the parent window handle - use the HWND parameter of your main window procedure for that. level_edit seems to be a window handle passed to LvlEdit function as its second parameter and is set as the return value of CreateWindowEx within that function - you'll need to ensure you've declared it with suitable scope prior to passing it to the function. hInstance is, presumably, the application instance in which case GetModuleHandle(0) is the quickest, least painful method of getting hold of that. Something like:

Code:

---

LRESULT CALLBACK WndProc(HWND hWnd,UINT uMsg,WPARAM wParam, LPARAM lParam)
{
static HWND level_edit; /*this may not have sufficient scope.*/
                        /*If not redeclare as global or whatever's appropriate*/
case WM_COMMAND:
  /*whenever condition is met to call fn:*/
  LvlEdit(hWnd, level_edit, GetModuleHandle(0));
}

---

should eliminate those errors but may not necessarily produce the results you expect.

sorry about the bumping :)

and thanx, works perfectly now!

-psychopath