Question

I need to make a few programs in C and i cannot get the window to work. Its come up with about 30 errors mostly saying ; is expected when there is one there, no storage class or type specifier, and declaration expected, not sure what these mean. I have looked at two turtorials and they both look extremely similar and mine looks the same, so not sure what these missing things are.

Heres my code

#include <windows.h>

LRESULT CALLBACK WindowFunc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
{
    switch(msg)
    {
        case WM_CLOSE:
            DestroyWindow(hwnd);
        break;
        case WM_DESTROY:
            PostQuitMessage(0);
        break;
        default:
            return DefWindowProc(hwnd, msg, wParam, lParam);
    }
    return 0;
}

int WINAPI WinMain(HINSTANCE hThisInst, 
                         HINSTANCE hPrevInst, 
                         LPSTR lpszArgs, 
                         int nWinMode);

{
WNDCLASS wcls;
HWND hwnd;
MSG msg;

LPCWSTR szClassName = L"ThreadsProgram";
LPCWSTR szWinName = L"My Threads Program"

    //Register Class
  wcls.style         =0; 
  wcls.lpfnWndProc   =WindowFunc;
  wcls.cbClsExtra    =0;
  wcls.cbWndExtra    =0;
  wcls.hInstance     =hThisInst;
  wcls.hIcon         =LoadIcon(NULL, IDI_APPLICATION);
  wcls.hCursor       =LoadCurser(NULL, IDC_ARROW);
  wcls.hbrBackground =(HBRUSH)GetStockObject(WHITE_BRUSH);
  wcls.lpszMenuName  =NULL;
  wcls.lpszClassName =szClassName;

      if(!RegisterClass(&wcls))
    {
        MessageBox(NULL, "Window Registration Failed!",
            MB_ICONEXCLAMATION | MB_OK);
        return 0;
    }

//Make Window 
hwnd = CreateWindow(szClassName,
szWinName,
WS_OVERLAPPINGWINDOW,
100,
100,
400,
400,
HWND_DESKTOP,
NULL,
hThisInst,
NULL);

//Show Window

if(hwnd == NULL)
    {
        MessageBox(NULL, "Window Failed!",
            MB_ICONEXCLAMATION | MB_OK);
        return 0;
    }

ShowWindow(hwnd, nWinMode);
UpdateWindow(hwnd);

//Main Message Loop
while(GetMessage(&msg, NULL, 0, 0))
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
    return (int)msg.wParam;
}

Answer 1

A lot of typos there.

1. semicolon after the WinMain

2. MessageBox() function taking 3 instead of 4 params.

3. LPWCSTR params

4. ShowWindow() with nCmdShow doesn’t … show

5. WS_OVERPLAPPEDWINDOW (not WS_OVERLAPPINGWINDOW)

6. LoadCursor instread of LoadCurser

Should work now. Next time type carefully

#include <windows.h>

LRESULT CALLBACK WindowFunc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
{
     switch(msg)
     {
         case WM_CLOSE: DestroyWindow(hwnd); break;
         case WM_DESTROY: PostQuitMessage(0); break;
         default: return DefWindowProc(hwnd, msg, wParam, lParam);
     }
     return 0;
}

int WINAPI WinMain(HINSTANCE hThisInst,  HINSTANCE hPrevInst,  LPSTR lpszArgs,  int nWinMode)
{
    WNDCLASS wcls;
    HWND hwnd;
    MSG msg;

    LPCSTR szClassName = "ThreadsProgram";
    LPCSTR szWinName = "My Threads Program";

    //Register Class
    wcls.style         =0; 
    wcls.lpfnWndProc   =WindowFunc;
    wcls.cbClsExtra    =0;
    wcls.cbWndExtra    =0;
    wcls.hInstance     =hThisInst;
    wcls.hIcon         =LoadIcon(NULL, IDI_APPLICATION);
    wcls.hCursor       =LoadCursor(NULL, IDC_ARROW);
    wcls.hbrBackground =(HBRUSH)GetStockObject(WHITE_BRUSH);
    wcls.lpszMenuName  =NULL;
    wcls.lpszClassName =szClassName;

     if(!RegisterClassA(&wcls))
    {
        MessageBoxA(NULL, 0, "Window Registration Failed!", MB_ICONEXCLAMATION | MB_OK);
        return 0;
    }

    //Make Window 
    hwnd = CreateWindowA(szClassName, szWinName,
       WS_OVERLAPPEDWINDOW,
       100, 100, 400, 400,
       HWND_DESKTOP,
       NULL, hThisInst, NULL);

    //Show Window

    if(hwnd == NULL)
    {
        MessageBoxA(NULL, 0, "Window Failed!", MB_ICONEXCLAMATION | MB_OK);
        return 0;
    }

    ShowWindow(hwnd, SW_SHOW/*nWinMode*/);
    UpdateWindow(hwnd);

    //Main Message Loop
    while(GetMessage(&msg, NULL, 0, 0))
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
    return (int)msg.wParam;
}