hi, i've been reading this book that i've been determined to finish for a very long time now. all through the book, i've encountered syntax errors that i had to fix. i don't know if this is good or bad, but it's an extra part of the learning process for me. it teaches me how to get rid of errors.
so i'm in the last chapter of the book where you make a game using windows programming and directx. i'm reading the code and typing it in and as i get to the end of the code, the final
return( 0 );
}
is outside the rest of the code. the last curly brace doesn't enclose return(0). it's totaly outside of the WndProc(); function. so i look up and down the code in my book and my editor, counth the braces and everything. but i can't seem to find out where to put this return. so i'm thinking ok i'll edit it to put it in myself. and then i think, is this return even necessary since the case statement will return default?
very lost. if you want to looks at the code, i don't know which part to send. so i'll send the whole thing. but the problem is most noticable at the bottom(to a newb such as myself that is).
//pirates.cpp
#include "globals.h"
int WINAPI WinMain(/*Win32 entry-poin routine */
HINSTANCE hInst,
HINSTANCE hPreInst,
LPSTR lpszCmdLine,
int nCmdShow)
{
//calll initApp to do initialization
InitApp(hInst, nCmdShow);
//call DoEventLoop to handle the event loop
return DoEventLoop();
}
/*callback procedure*/
LRESULT CALLBACK WndProc( HWND hWnd, UINT messg, WPARAM wParam, LPARAM lParam)
//this function processes user input
//and the timers used to run the game
{
static bool inCity = false; //stores if the player is in a city
switch(messg)//figure out which message is being sent
{
case WM_CREATE://if the window is being CreateD
//do directx initianlization
initializeDirectX(hWnd);
CreateTowns();//create all of the town objects
//the title screen is displayed for
//5 seconds (5000 milliseconds)
//after this we enter the main part of the game
//set the opening screen timer
SetTimer(hWnd, OpeningTimer, 5000, NULL);
//display the opening Screen
DisplayScreen( OpeningScreen);
break //end case WM_CREATE
//this event is used to get rid of the mouse
case WM_SETCURSOR:
SetCursor(NULL); //set it to NULL
break;//end case WM_SETCURSOR
case WM_KEYDOWN://if a key is pressed
//if layer is in city then
//only the enter key works
if(inCity)
{
//if the key is Enter then
//we exit the city screen
if(wParam == VK_RETURN)
inCity = false;
break;
}
//handled if city screen,
//so treat as normal game now
switch(wParam)//figure out which key it is
(
case VK_UP://the up arrow
//inform everyone else
//that we should move up
moveUp = true;
break;
case VK_DOWN://the down arrow
moveDown = true;
break;
case VK_LEFT:
moveLeft = true;
case VK_RIGHT:
moveRight = true;
break;
case VK_SPACE:
//we should fire
//note that firing has not been implemented yet
fire = true;
break;
case VK_ESCAPE:
//end the main timer
KillTimer(hWnd, MainTimer);
PostQuitMessage(0);//end the program
break;
}
break;//exit te case opening timer
case WM_KEYUP//if a key is released
switch(wParam)//figure out which key
{
//stop moving if key is Released
case VK_UP:
moveUp = false;
break;
case VK_DOWN:
moveDown = false;
break;
case VK_LEFT:
moveLeft = false;
break;
case VK_RIGHT:
moveRight = false;
break;
case VK_SPACE:
fire = false;
break;
}
break://end case keyup
case WM_TIMER://if a timer goes of
//if player is in city then don't use the timer
if(inCity)
break;//exit the WM_TIMER message
switch( wParam )//find out which timer is going off
{
//the timer for the opening screen
case OpeningTimer:
//starting the main timer
SetTimer(hWnd, MainTimer, 15, NULL);
//kill the opening screen timer
KillTimer(hWnd,OpeningTimer);
//draw the screen for the first time
Draw();
break;//exit case OpeningTimer
//the timer for the main game
case MainTimer:
DoMove();//move the ship based on input
Redraw();//redraw the screen
//if player has entered a city
if(isInCity(&ship->GetPosition()))
{
//inform everyone else
inCity = true;
//display the city screen
DisplayScreen(CityScreen);
//put the player outside of city radius
ship->moveBack();
}
break;//end case MainTimer
}
break;//endcase WM_TIMER
default://let windows handle all other messages
return( DefWindowProc(hWnd, messg, wParam, lParam));
}
return(0);//default return value
}
__________________________________________________________Maybe one day i can not be affraid of venturing out of the beginners section.
