need help with brick game - GameCreators Forum

Author

Message

jjeff

13

Years of Service

User Offline

Joined: 11th Mar 2011

Location:

Posted: 11th Mar 2011 20:09

Link

I am trying to make a brick game like this one http://www.youtube.com/watch?v=K1Ih5IyoydQ&feature=related

I have created all the bricks with this code

+ Code Snippet

// creates all the bricks
	int spriteID = 14;
	for(int i = 0; i < 80; i += 20)
	{
		for(int j = 0; j < 640; j += 80)
		{
			dbSprite(spriteID, j, i, imgID);
			spriteID++;
		}
	}

The problem, however, is with my collision function. It checks one brick at a time to see if a brick is being hit by the ball. it only works if you hit the brick it happens to be checking. this is my function, i haven't finished it. how can i make it check all bricks at all times or should i be doing this completely differently?

int bricks::collision()
{
	//int spriteID = 14;
	for(int spriteID = 14; dbSpriteExist(spriteID); spriteID++)
	{
		if(spriteID > 48)
		{
			spriteID = 14;
		}
		for(int i = 0; i < 80; i += 20)
		{
			// if the ball hits a brick
			if(dbSpriteCollision(spriteID, 4) == 1)
			{
				// if the ball hits the bottom of the brick
				if(dbSpriteHeight(spriteID) + dbSpriteY(spriteID) <= dbSpriteY(4) && dbSpriteX(4) + 10 >= dbSpriteX(spriteID) && dbSpriteX(4) + 10 <= dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
				{
					if(dbSpriteAngle(4) == 315)
					{
						dbRotateSprite(4, 225);
					}
					if(dbSpriteAngle(4) == 45)
					{
						dbRotateSprite(4, 135);
					}

dbDeleteSprite (spriteID);
				}
				// if the ball hits the top of the brick
				if(dbSpriteHeight(spriteID) >= dbSpriteHeight(4) + dbSpriteY(4) && dbSpriteX(4) + 10 >= dbSpriteX(spriteID) && dbSpriteX(4) + 10 <= dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
				{
					if(dbSpriteAngle(4) == 225)
					{
						dbRotateSprite(4, 315);
					}
					if(dbSpriteAngle(4) == 135)
					{
						dbRotateSprite(4, 45);
					}

dbDeleteSprite (spriteID);
				}

}
	
		}
	
	}
	return 0;
}

+ Code Snippet

int bricks::collision()
{
	//int spriteID = 14;
	for(int spriteID = 14; dbSpriteExist(spriteID); spriteID++)
	{
		if(spriteID > 48)
		{
			spriteID = 14;
		}
		for(int i = 0; i < 80; i += 20)
		{
			// if the ball hits a brick
			if(dbSpriteCollision(spriteID, 4) == 1)
			{
				// if the ball hits the bottom of the brick
				if(dbSpriteHeight(spriteID) + dbSpriteY(spriteID) <= dbSpriteY(4) && dbSpriteX(4) + 10 >= dbSpriteX(spriteID) && dbSpriteX(4) + 10 <= dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
				{
					if(dbSpriteAngle(4) == 315)
					{
						dbRotateSprite(4, 225);
					}
					if(dbSpriteAngle(4) == 45)
					{
						dbRotateSprite(4, 135);
					}

					dbDeleteSprite (spriteID);
				}
				// if the ball hits the top of the brick
				if(dbSpriteHeight(spriteID) >= dbSpriteHeight(4) + dbSpriteY(4) && dbSpriteX(4) + 10 >= dbSpriteX(spriteID) && dbSpriteX(4) + 10 <= dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
				{
					if(dbSpriteAngle(4) == 225)
					{
						dbRotateSprite(4, 315);
					}
					if(dbSpriteAngle(4) == 135)
					{
						dbRotateSprite(4, 45);
					}

					dbDeleteSprite (spriteID);
				}

			}
	
		}
	
	}
	return 0;
}

Back to top

Profile PM

jjeff

13

Years of Service

User Offline

Joined: 11th Mar 2011

Location:

Posted: 11th Mar 2011 23:48

Link

Sorry, my collision function was really sloppy. i cleaned it up a bit

int bricks::collision()
{	
	for(int spriteID = 14; dbSpriteExist(spriteID); spriteID++)
	{
		if(spriteID > 48)
		{
			spriteID = 14;
		}
		// if the ball hits a brick
		if(dbSpriteCollision(spriteID, 4) == 1)
		{
			// if the ball hits the bottom of the brick
			if(dbSpriteHeight(spriteID) + dbSpriteY(spriteID) <= dbSpriteY(4) && dbSpriteX(4) + 10 >= dbSpriteX(spriteID) && dbSpriteX(4) + 10 <= dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
			{
				if(dbSpriteAngle(4) == 315)
				{
					dbRotateSprite(4, 225);
				}
				if(dbSpriteAngle(4) == 45)
				{
					dbRotateSprite(4, 135);
				}

dbDeleteSprite (spriteID);
			}

// if the ball hits the top of the brick
			if(dbSpriteHeight(spriteID) >= dbSpriteHeight(4) + dbSpriteY(4) && dbSpriteX(4) + 10 >= dbSpriteX(spriteID) && dbSpriteX(4) + 10 <= dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
			{
				if(dbSpriteAngle(4) == 225)
				{
					dbRotateSprite(4, 315);
				}
				if(dbSpriteAngle(4) == 135)
				{
					dbRotateSprite(4, 45);
				}

dbDeleteSprite (spriteID);
			}
			
			// if the ball hits the right side of the brick
			if(dbSpriteY(4) + 10 > dbSpriteY(spriteID) && dbSpriteY(4) + 10 < dbSpriteY(spriteID) + dbSpriteHeight(spriteID) && dbSpriteX(4) > dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
			{
				if(dbSpriteAngle(4) == 225)
				{
					dbRotateSprite(4, 135);
				}
				if(dbSpriteAngle(4) == 315)
				{
					dbRotateSprite(4, 45);
				}

dbDeleteSprite (spriteID);
			}

// if the ball hits the left side of the brick
			if(dbSpriteY(4) + 10 > dbSpriteY(spriteID) && dbSpriteY(4) + 10 < dbSpriteY(spriteID) + dbSpriteHeight(spriteID) && dbSpriteX(4) + 20 < dbSpriteX(spriteID))
			{
				if(dbSpriteAngle(4) == 45)
				{
					dbRotateSprite(4, 315);
				}
				if(dbSpriteAngle(4) == 135)
				{
					dbRotateSprite(4, 225);
				}

dbDeleteSprite (spriteID);
			}

else
			{
				break;
			}
		}
	}
		
	return 0;
}

+ Code Snippet

int bricks::collision()
{	
	for(int spriteID = 14; dbSpriteExist(spriteID); spriteID++)
	{
		if(spriteID > 48)
		{
			spriteID = 14;
		}
		// if the ball hits a brick
		if(dbSpriteCollision(spriteID, 4) == 1)
		{
			// if the ball hits the bottom of the brick
			if(dbSpriteHeight(spriteID) + dbSpriteY(spriteID) <= dbSpriteY(4) && dbSpriteX(4) + 10 >= dbSpriteX(spriteID) && dbSpriteX(4) + 10 <= dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
			{
				if(dbSpriteAngle(4) == 315)
				{
					dbRotateSprite(4, 225);
				}
				if(dbSpriteAngle(4) == 45)
				{
					dbRotateSprite(4, 135);
				}

				dbDeleteSprite (spriteID);
			}

			// if the ball hits the top of the brick
			if(dbSpriteHeight(spriteID) >= dbSpriteHeight(4) + dbSpriteY(4) && dbSpriteX(4) + 10 >= dbSpriteX(spriteID) && dbSpriteX(4) + 10 <= dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
			{
				if(dbSpriteAngle(4) == 225)
				{
					dbRotateSprite(4, 315);
				}
				if(dbSpriteAngle(4) == 135)
				{
					dbRotateSprite(4, 45);
				}

				dbDeleteSprite (spriteID);
			}
			
			// if the ball hits the right side of the brick
			if(dbSpriteY(4) + 10 > dbSpriteY(spriteID) && dbSpriteY(4) + 10 < dbSpriteY(spriteID) + dbSpriteHeight(spriteID) && dbSpriteX(4) > dbSpriteX(spriteID) + dbSpriteWidth(spriteID))
			{
				if(dbSpriteAngle(4) == 225)
				{
					dbRotateSprite(4, 135);
				}
				if(dbSpriteAngle(4) == 315)
				{
					dbRotateSprite(4, 45);
				}

				dbDeleteSprite (spriteID);
			}

			// if the ball hits the left side of the brick
			if(dbSpriteY(4) + 10 > dbSpriteY(spriteID) && dbSpriteY(4) + 10 < dbSpriteY(spriteID) + dbSpriteHeight(spriteID) && dbSpriteX(4) + 20 < dbSpriteX(spriteID))
			{
				if(dbSpriteAngle(4) == 45)
				{
					dbRotateSprite(4, 315);
				}
				if(dbSpriteAngle(4) == 135)
				{
					dbRotateSprite(4, 225);
				}

				dbDeleteSprite (spriteID);
			}			

			else
			{
				break;
			}
		}
	}
		
	return 0;
}

Back to top

Profile PM

DeadTomGC

14

Years of Service

User Offline

Joined: 11th Aug 2010

Location: LU

Posted: 12th Mar 2011 04:40 Edited at: 12th Mar 2011 04:40

Link

You seem to be making this harder than it needs to be.

I would suggest making it change the x then check collision.
If there is a collision then you know it was on a side. By looking at the sign of the velocity in the x component you can know which side it was, but you don't even need to know this.
You could just reverse the velocity by multiplying it by -1 and not pay attention to which side it hit.

Then do the same thing for the y component.

Also, that break could be causing a problem. I don't see a need for it.

Btw, Have a look at this demo code (11th post) which uses this process of analyzing the x and y movement separately.

http://forum.thegamecreators.com/?m=forum_view&t=182549&b=22

I'll get a new signature someday.

Back to top

Profile PM Email Website

jjeff

13

Years of Service

User Offline

Joined: 11th Mar 2011

Location:

Posted: 15th Mar 2011 07:07

Link

Okay, I rewrote my ball movement function to change the ball's vertical displacement, check for collision, change horizontal displacement, and then check for collision again. this works great for the bricks, but when the ball hits the user's bar, it just bounces up and down on the bar, rather than moving back towards the bricks. also, i have it set to spawn the ball when the user hits space bar and although that works for initially spawning the ball, the user is unable to respawn the ball.

int ball::movement()
{

// respawns the ball
	if(dbSpaceKey() == 1 && dbSpriteExist(2) == 0)
	{
		dbSprite(2, 320, 420, 3);
	}
	
	// moves the ball
	if (dbSpriteExist(2) == 1)
	{
		// adjust vertical displacement of the ball
		y += dy;
		for(int i = 3; i < 40; i++)
		{
			// if the ball hits a brick from above or below it
			if(dbSpriteCollision(2, i) == 1)
			{
				dy = -dy;
				// if the ball does not hit the barrier
				if(i != 3)
				{
					dbDeleteSprite(i);
				}
			}
		}
		// adjust horizontal displacement of the ball
		x += dx;
		for(int i = 4; i < 40; i++)
		{
			// if the ball hits the brick from either side
			if(dbSpriteCollision(2, i) == 1)
			{
				dx = -dx;
				// if the ball does not hit the barrier
				if(i != 3)
				{
					dbDeleteSprite(i);
				}
			}
		}

//If the ball hits either side of the window
		if (x <= 0 || x >= 640 - dbSpriteWidth(2))
		{
			dx = -dx;		// reverse the horizontal direction
		}

//if the ball hits the top of the window
		if (y <= 0)
		{
			dy = -dy;		// reverse the vertical direction
		}

// if the ball falls off the screen
		if (y > 480)
		{
			dbDeleteSprite(2);
			return 0;
		}

// redraw the ball at the new location
		dbSprite(2, x, y, 3);
	}

return 0;
}

+ Code Snippet

int ball::movement()
{

	// respawns the ball
	if(dbSpaceKey() == 1 && dbSpriteExist(2) == 0)
	{
		dbSprite(2, 320, 420, 3);
	}
	
	// moves the ball
	if (dbSpriteExist(2) == 1)
	{
		// adjust vertical displacement of the ball
		y += dy;
		for(int i = 3; i < 40; i++)
		{
			// if the ball hits a brick from above or below it
			if(dbSpriteCollision(2, i) == 1)
			{
				dy = -dy;
				// if the ball does not hit the barrier
				if(i != 3)
				{
					dbDeleteSprite(i);
				}
			}
		}
		// adjust horizontal displacement of the ball
		x += dx;
		for(int i = 4; i < 40; i++)
		{
			// if the ball hits the brick from either side
			if(dbSpriteCollision(2, i) == 1)
			{
				dx = -dx;
				// if the ball does not hit the barrier
				if(i != 3)
				{
					dbDeleteSprite(i);
				}
			}
		}

		//If the ball hits either side of the window
		if (x <= 0 || x >= 640 - dbSpriteWidth(2))
		{
			dx = -dx;		// reverse the horizontal direction
		}

		//if the ball hits the top of the window
		if (y <= 0)
		{
			dy = -dy;		// reverse the vertical direction
		}

		// if the ball falls off the screen
		if (y > 480)
		{
			dbDeleteSprite(2);
			return 0;
		}

		// redraw the ball at the new location
		dbSprite(2, x, y, 3);
	}

	return 0;
}

Back to top

Profile PM

Sorry your browser is not supported!

Dark GDK / need help with brick game