Code mess..

[minas1]

Well my main() function has become a mess... What should I do next time to prevent this?

int main(int argc, char *args[])
{
	srand((unsigned)time(0));

	if(!init())
		return 1;

	atexit(cleanup); // to cleanup the surfaces
	
	// load images
	background = loadImage("background.jpg");
	message = loadImage("play_again.jpg");
	youWinMessage = loadImage("you_win.jpg");
	youLoseMessage = loadImage("you_lose.jpg");

start:
	// the ball
	SDL_Rect bounds; bounds.x = 0; bounds.y = 0; bounds.w = SCREEN_WIDTH; bounds.h = SCREEN_HEIGHT;
	Sprite ball(SCREEN_WIDTH / 2, SCREEN_HEIGHT / 2, loadImage("ball.gif"), bounds );
	ball.setStep(BALL_SPEED);
	randomizeDirections(ball);

	//Rectangles for the player and the computer
	Rect player(SCREEN_WIDTH / 2 - RECT_WIDTH / 2, SCREEN_HEIGHT - int(RECT_HEIGHT * 1.5), RECT_WIDTH, RECT_HEIGHT),
		computer(SCREEN_WIDTH / 2 - RECT_WIDTH / 2, int(RECT_HEIGHT * 1.5) - RECT_HEIGHT, RECT_WIDTH, RECT_HEIGHT);

	player.setColor(rand() % 256, rand() % 256, rand() % 256);
	computer.setColor(rand() % 256, rand() % 256, rand() % 256);
	
	bool playerWins = false, computerWins = false;

	if(SDL_Flip(screen) == -1)
		return 1;

	timer.start(); // start the timers
	bool quit = false; // the quit flag

	//**************************MAIN LOOP****************************************************
	while(!quit)
	{
		if(timer.getTicks() % 30 == 0)
		{
			//apply the background
			applySurface(0, 0, background, screen);

			// handle events
			while( SDL_PollEvent( &event ) )
			{
				//If the user has Xed out the window
				if(event.type == SDL_QUIT)
					quit = true;
				if(keystates[SDLK_ESCAPE])
					quit = true;

				if(keystates[SDLK_LEFT])
				{
					if(player.x >= 0)
						player.x -= STEP;
				}
				if(keystates[SDLK_RIGHT])
				{
					if(player.x + player.Width() <= SCREEN_WIDTH)
						player.x += STEP;
				}
				if(keystates[SDLK_UP])
				{
					if(player.y == Sint16(SCREEN_HEIGHT - int(RECT_HEIGHT) * 1.5))
						player.y -= STEP;
				}
				else
				{
					if(player.y < Sint16(SCREEN_HEIGHT - int(RECT_HEIGHT) * 1.5))
						player.y += STEP;
				}

				if(keystates[SDLK_RETURN])
				{
					if(playerWins || computerWins)
					{
						playerWins = false;
						computerWins = false;
						goto start;
					}
				}
			}
		
			if(timer.isStarted())
			{
				//calcuate the computer's move and motion of the ball
				AI(computer, ball);
				ball.move();

				if(ball.Y() + ball.H() + ball.getStep() - 2 >= SCREEN_HEIGHT)
				{
					computerWins = true;
					timer.stop();
				}
				if(ball.Y() - ball.getStep() + 5 <= 0)
				{
					playerWins = true;
					timer.stop();
				}
				
				//check for collision
				bool thereWasCollision = false;
				if(collision(ball.getOffset(), player.getSDL_Rect()))
				{
					ball.setDirectionY(Sprite::TO_ZERO);
					changeDirectionOnCollision(ball, player.getSDL_Rect());
					thereWasCollision = true;
				}
				if(collision(ball.getOffset(), computer.getSDL_Rect()))
				{		
					ball.setDirectionY(Sprite::TO_INFINITY);
					changeDirectionOnCollision(ball, computer.getSDL_Rect());
					thereWasCollision = true;
				}
				
				if(thereWasCollision)
					if(ball.getStep() < BALL_SPEED_MAX)
						ball.setStep(ball.getStep() + 1);
			}

			//draw
			player.draw();
			computer.draw();
			applySurface(ball.X(), ball.Y(), ball.getImage(), screen);

			if(playerWins || computerWins)
			{
				applySurface( (SCREEN_WIDTH - message->w) / 2, (SCREEN_HEIGHT + message->h + 10) / 2, message, screen);
				
				if(playerWins)
					applySurface( (SCREEN_WIDTH - youWinMessage->w) / 2, (SCREEN_HEIGHT - youWinMessage->h - 10) / 2, youWinMessage, screen);
				if(computerWins)
					applySurface( (SCREEN_WIDTH - youLoseMessage->w) / 2, (SCREEN_HEIGHT - youLoseMessage->h - 10) / 2, youLoseMessage, screen);
			}
			

			if(SDL_Flip(screen) == -1) return 1;
		}
	}

	SDL_Quit();
	return 0;
}

[Salem]

> What should I do next time to prevent this?
Start with a design.

Even simple things like

do {
  if ( inputPresent ) {
    doInput();
  }
  moveEnemies();
  dead = checkGameOver();
  redrawScreen();
} while ( !dead );

Also, the moment you can no longer see the { and } of the whole of main (or any other function) on screen, then STOP and think about your code structure for a few minutes, rather than just carrying on typing.

[minas1]

One thing about collisions. At the moment I'm using functions that take 2 arguments. Would it be better if each object had a collision function so I could say:

Sprite s;
s.collision(some_rect_type rect); // returns bool

Is there a better alternative?

[Salem]

I dunno - you need to look at the overall design and implementation, before deciding whether specific local improvements are a good idea.

When you're into writing larger programs, there isn't a single "best way" of doing anything.

[Comatose]

Except for not using goto's :P

edit: and using functions/methods is always a plus.