Errors with simple window handler

OK, so I'm trying to learn C++ and get into game programming in windows.... I've been using the book Tricks of the Windows Game Programming Gurus and I had some trouble compiling the second example. I'm using VC++ 6.0 Introductory Edition and am running Win XP.

This is my WINMAIN.... I used ^ and ^^ to indicate the lines that the errors occure on.

Code:

---


int WINAPI WinMain(HINSTANCE hinstance,
                    HINSTANCE hprevinstance,
                    LPSTR lpcmdline,
                    int ncmdshow)
{
    WNDCLASSEX winclass;    // this will hold the class we create
    HWND hwnd;              // generic window handle
    MSG msg;                // generic message
   
    // first fill in the window class structure
    winclass.cbSize        = sizeof(WNDCLASSEX);
    winclass.style          = CS_DBLCLKS | CS_OWNDC | CS_HREDRAW | CS_VREDRAW;
    winclass.lpfnWndProc    = WindowProc;
    winclass.cbClsExtra    = 0;
    winclass.cbWndExtra    = 0;
    winclass.hInstance      = hinstance;
    winclass.hIcon          = LoadIcon(NULL, IDI_APPLICATION);
    winclass.hCursor        = LoadCursor(NULL, IDC_ARROW);
  ^winclass.hbrBackground  = GetStockObject(BLACK_BRUSH);
    winclass.lpszMenuName  = NULL;
    winclass.lpszClassName  = WINDOW_CLASS_NAME;
    winclass.hIconSm        = LoadIcon(NULL, IDI_APPLICATION);
   
    // register the window class
    if (!RegisterClassEx(&winclass))
        return(0);
       
    // create the window
    if (!(hwnd = CreateWindowEx(NULL,                  // extended style
                                WINDOW_CLASS_NAME,      // class
                                "Your Basic Window",    // title
                              ^^WS_OVERLAPPEDWINDOW | WS_VISBLE,
                                0,0,                    // initial x,y
                                400,400,                // initial width, height
                                NULL,                  // handle to parent
                                NULL,                  // handle to menu
                                hinstance,              // instance of this application
                                NULL)))                // extra creation parms
        return(0);
       
    // enter main event loop
    while(GetMessage(&msg,NULL,0,0))
    {
        // translate any accelerator keys
        TranslateMessage(&msg);
       
        // sned the message to the window proc
        DispatchMessage(&msg);
    } // end while
   
    // return to Windows like this
    return(msg.wParam);
   

} // end WinMain

---

I get the following errors during compile...

^error C2440: '=' : cannot convert from 'void *' to 'struct HBRUSH__ *'
Conversion from 'void*' to pointer to non-'void' requires an explicit cast

^^error C2065: 'WS_VISBLE' : undeclared identifier
Error executing cl.exe.

If you need any more info to help me out just ask.

Thanks.

>> winclass.hbrBackground = GetStockObject(BLACK_BRUSH);

Change it to

winclass.hbrBackground = (HBRUSH) GetStockObject(BLACK_BRUSH);


>> ^^WS_OVERLAPPEDWINDOW | WS_VISBLE,

Change it to

WS_OVERLAPPEDWINDOW | WS_VISIBLE,

WS_VISABLE is not a paramater of dwExStyle so get rid of it
if you want to show your window do this after you create the window

Code:

---

ShowWindow(hwnd,SW_SHOWNORMAL);
UpdateWindow(hwnd);
//msg loop

---

Awesome Thanks guys.... I'm getting a linking error now though :(

--------------------Configuration: main - Win32 Debug--------------------
Compiling...
main.cpp
Linking...
LIBCD.lib(crt0.obj) : error LNK2001: unresolved external symbol _main
Debug/main.exe : fatal error LNK1120: 1 unresolved externals
Error executing link.exe.

main.exe - 2 error(s), 0 warning(s)

Again, if you need more info just ask.

Thanks

Can you post all your code?
[edit] Also a word of advice use some braces to begin and end your if statments also some indentation too, you will be happy you did this later.[/edit]

By request....:

Code:

---

//WINDOWS - A Complete Windows Program

// INCLUDES ///////////////////////////
#define WIN32_LEAN_AND_MEAN // just say no to MFC

#include <windows.h>  // include all the windows headers
#include <windowsx.h>  // include useful macros
#include <stdio.h>
#include <math.h>

// DEFINES ////////////////////////////
#define WINDOW_CLASS_NAME "WINCLASS1"

// GLOBALS ////////////////////////////

// FUNCTIONS //////////////////////////
LRESULT CALLBACK WindowProc(HWND hwnd,
                            UINT msg,
                            WPARAM wparam,
                            LPARAM lparam)
{
    // this is the main message handler of the system
    PAINTSTRUCT ps;    // used in WM_PAINT
    HDC hdc;            // handle to a device context
   
    // what is the message
    switch(msg)
    {
        case WM_CREATE:
        {
            // do initialization stuff here

            // return success
            return(0);
        } break;
       
        case WM_PAINT:
        {
            // simply validate the window
            hdc = BeginPaint(hwnd,&ps);

            // you would do all your painting here
           
            EndPaint(hwnd,&ps);
           
            // return success
            return(0);
        } break;
       
        case WM_DESTROY:
        {
            // kill the application, this sends a WM_QUIT message
            PostQuitMessage(0);
           
            // return success
            return(0);
        } break;
       
        default: break;
       
    }  // end switch
   
    // process any messages that we didn't take care of
    return (DefWindowProc(hwnd, msg, wparam, lparam));
   
}  //end WinProc

// WINMAIN ///////////////////////////////

int WINAPI WinMain(HINSTANCE hinstance,
                    HINSTANCE hprevinstance,
                    LPSTR lpcmdline,
                    int ncmdshow)
{
    WNDCLASSEX winclass;    // this will hold the class we create
    HWND hwnd;              // generic window handle
    MSG msg;                // generic message
   
    // first fill in the window class structure
    winclass.cbSize        = sizeof(WNDCLASSEX);
    winclass.style          = CS_DBLCLKS | CS_OWNDC | CS_HREDRAW | CS_VREDRAW;
    winclass.lpfnWndProc    = WindowProc;
    winclass.cbClsExtra    = 0;
    winclass.cbWndExtra    = 0;
    winclass.hInstance      = hinstance;
    winclass.hIcon          = LoadIcon(NULL, IDI_APPLICATION);
    winclass.hCursor        = LoadCursor(NULL, IDC_ARROW);
    winclass.hbrBackground  = (HBRUSH) GetStockObject(BLACK_BRUSH);
    winclass.lpszMenuName  = NULL;
    winclass.lpszClassName  = WINDOW_CLASS_NAME;
    winclass.hIconSm        = LoadIcon(NULL, IDI_APPLICATION);
   
    // register the window class
    if (!RegisterClassEx(&winclass))
        return(0);
       
    // create the window
    if (!(hwnd = CreateWindowEx(NULL,                  // extended style
                                WINDOW_CLASS_NAME,      // class
                                "Your Basic Window",    // title
                                WS_OVERLAPPEDWINDOW | WS_VISIBLE,
                                0,0,                    // initial x,y
                                400,400,                // initial width, height
                                NULL,                  // handle to parent
                                NULL,                  // handle to menu
                                hinstance,              // instance of this application
                                NULL)))                // extra creation parms
        return(0);
       
    // enter main event loop
    while(GetMessage(&msg,NULL,0,0))
    {
        // translate any accelerator keys
        TranslateMessage(&msg);
       
        // sned the message to the window proc
        DispatchMessage(&msg);
    } // end while
   
    // return to Windows like this
    return(msg.wParam);
   
} // end WinMain

///////////////////////////////////////////////////////

---

This is basically copied from the book, so this isn't my code formatting. I usually use braces and indentation in my IF statements. I just need to get this to work to really be able to learn from it.

Code:

---

#define WIN32_LEAN_AND_MEAN // just say no to MFC

---

win32 has no idea about MFC so this is tottally wrong. This is just saying don't include unnecessary headers
umm it does compile for me i just get some warnings about NULL being passed here

Code:

---

if (!(hwnd = CreateWindowEx(NULL,                  // extended style
                                WINDOW_CLASS_NAME,      // class
                                "Your Basic Window",    // title
                                WS_OVERLAPPEDWINDOW | WS_VISIBLE,
                                0,0,                    // initial x,y
                                400,400,                // initial width, height
                                NULL,                  // handle to parent
                                NULL,                  // handle to menu
                                hinstance,              // instance of this application
                                NULL)))

---

But easily fixed by passing 0 as the first parameter instead of NULL.
What book are you using? Maybe you should invest in a better one or you could check out these good online tuts http://www.winprog.org/tutorial/

I mentioned earlier that I was using the book "Tricks of the Windows Game Programming Gurus". Thats interesting that it compiles fine for you other than some warnings. I wonder if its just some settings in my compiler then?

I'll definately read through those tutorials you mentioned.

So, no suggestions for fixing the linking error then??

Create a Win32 Project, not a Win32 Console project.

Thanks Dante... I guess the default is a Win32 Console project; I never actually made a project before I just loaded the .cpp file and then made a workspace.

So it compiles fine now. But it still tells me it can't execute the file... Are there some settings I have to set up first??

Quote:

---

But it still tells me it can't execute the file

---

Strange. Is there an error message? Could you copy and paste it?

It simply says...

Quote:

---

Cannot Execute Program

---

Thats all it says....