Jump to content

  • Log In with Google      Sign In   
  • Create Account

Interested in a FREE copy of HTML5 game maker Construct 2?

We'll be giving away three Personal Edition licences in next Tuesday's GDNet Direct email newsletter!

Sign up from the right-hand sidebar on our homepage and read Tuesday's newsletter for details!


#Actualkrrice

Posted 07 January 2013 - 04:37 PM

On collision you need to exit the loop. It only works on the last one because it will always run through the entire tree and always overwrite your 'collided' variable with the last "2" that was found.

 

I would give you some advice on how to restructure this but I don't know how you are trying to use it.

 

The simplest fix would be instead of collided = true; to simply do "return true;" remove the else and at the end of the loop do return false;

 

But I would say you need to rewrite this completely since you can obviously index into your array. (you're already doing it) and only test against the tile that you need to know if it had a collision on, instead of running a loop testing all possible points.

 

This is how i am using it for the moment. I have a MapLoader class that sends the Collision map, which is a vector of vectors to the Player class. In the Players Update that is where the for statement is.

I am just using the collided variable for testing right now. In main.cpp I have a text function displaying the collided varible and its value.So when ever I move over the tiles that are twos it will show 1 and if not 0. 

 

Here is Player.cpp

 

#include "Player.h"

Player::Player()
{}

void Player::Destroy()
{
	GameObject::Destroy();
}

void Player::Init(MapLoader *colMap,ALLEGRO_BITMAP *image = NULL)
{
	GameObject::Init(672, 554, 6, 6, 0, 0, 19, 15 );

	SetID(PLAYER);
	SetAlive(true);

	collided = false;

	map = colMap->GetColMap();

	lives = 3;
	score = 0;

	maxFrame = 4;
	curFrame = 0;
	frameDelay = 3;
	frameWidth = 36;
	frameHeight = 72;
	animationColumns = 4;
	animationDirection = 1;

	animationRow = 2;

	if(image != NULL)
		Player::image = image;
}

void Player::Update()
{
	GameObject::Update();

	if(x - boundX < 0)
		x  =  boundX;
	else if(x + boundX > WIDTH)
		x = WIDTH - boundX;

	if(y < 0)
		y = 0;
	else if(y > HEIGHT)
		y = HEIGHT;

	//Collision

	for(int i = 0; i < map.size(); i++)
	{
		for(int j = 0; j < map[i].size(); j++)
		{
			if(map[i][j] == 2)
			{
				if(x + 24 > j * 24 && x < (j * 24) + 24 &&
					y + 24 > i * 24 && y < (i * 24) + 24 )
				{
					collided = true;
				}
				else
					collided = false;
			}
		}
	}

	if(++frameCount >= frameDelay)
	{
		if(curFrame > 4)
			curFrame = 0;

		fx = (curFrame % animationColumns) * frameWidth;
		fy = animationRow * frameHeight;

		frameCount = 0;
	}
}
void Player::Render()
{
	GameObject::Render();

	//al_draw_rectangle(x + boundX,y + boundY,x - boundX,y - boundY,al_map_rgb(0,255,0),5);
	al_draw_bitmap_region(image, fx, fy, frameWidth, frameHeight,
		x - frameWidth / 2, y - frameHeight / 2, 0);
}

void Player::MoveUp()
{
	animationRow = 3;
	curFrame++;
	dirY = -1;
}
void Player::MoveDown()
{
	animationRow = 0;
	curFrame++;
	dirY = 1;
}
void Player::MoveLeft()
{
	animationRow = 1;
	curFrame++;
	dirX = -1;
}
void Player::MoveRight()
{
	animationRow = 2;
	curFrame++;
	dirX = 1;
}

void Player::ResetAnimation(int position)
{
	if(position == 1)
	{
		dirY = 0;
	}
	else
	{
		dirX = 0;
	}
}

void Player::Collided(int objectID)
{
	if(objectID == ENEMY)
		al_draw_rectangle(x + boundX,y + boundY,x - boundX,y - boundY,al_map_rgb(0,255,0),5);
		//lives++;
}


#1krrice

Posted 07 January 2013 - 04:32 PM

On collision you need to exit the loop. It only works on the last one because it will always run through the entire tree and always overwrite your 'collided' variable with the last "2" that was found.

 

I would give you some advice on how to restructure this but I don't know how you are trying to use it.

 

The simplest fix would be instead of collided = true; to simply do "return true;" remove the else and at the end of the loop do return false;

 

But I would say you need to rewrite this completely since you can obviously index into your array. (you're already doing it) and only test against the tile that you need to know if it had a collision on, instead of running a loop testing all possible points.

 

This is how i am using it for the moment. I have a MapLoader class that sends the Collision map, which is a vector of vectors to the Player class. In the Players Update that is where the for statement is.

I am just using the collided variable for testing right now. In main.cpp I have a text function displaying the collided varible and its value.So when ever I move over the tiles that are twos it will show 1 and if not 0. 

 

Here is Player.cpp

 

#include "Player.h"

Player::Player()
{}

void Player::Destroy()
{
	GameObject::Destroy();
}

void Player::Init(MapLoader *colMap,ALLEGRO_BITMAP *image = NULL)
{
	GameObject::Init(672, 554, 6, 6, 0, 0, 19, 15 );

	SetID(PLAYER);
	SetAlive(true);

	collided = false;

	map = colMap->GetColMap();

	lives = 3;
	score = 0;

	maxFrame = 4;
	curFrame = 0;
	frameDelay = 3;
	frameWidth = 36;
	frameHeight = 72;
	animationColumns = 4;
	animationDirection = 1;

	animationRow = 2;

	if(image != NULL)
		Player::image = image;
}

void Player::Update()
{
	GameObject::Update();
	if(x - boundX < 0)
		x  =  boundX;
	else if(x + boundX > WIDTH)
		x = WIDTH - boundX;

	if(y < 0)
		y = 0;
	else if(y > HEIGHT)
		y = HEIGHT;

	//Collision
	//map = colMap.GetColMap();

	for(int i = 0; i < map.size(); i++)
	{
		for(int j = 0; j < map[i].size(); j++)
		{
			if(map[i][j] == 2)
			{
				if(x + 24 > j * 24 && x < (j * 24) + 24 &&
					y + 24 > i * 24 && y < (i * 24) + 24 )
				{
					collided = true;
				}
				else
					collided = false;
			}
		}
	}

	if(++frameCount >= frameDelay)
	{
		if(curFrame > 4)
			curFrame = 0;

		fx = (curFrame % animationColumns) * frameWidth;
		fy = animationRow * frameHeight;

		frameCount = 0;
	}
}
void Player::Render()
{
	GameObject::Render();

	//al_draw_rectangle(x + boundX,y + boundY,x - boundX,y - boundY,al_map_rgb(0,255,0),5);
	al_draw_bitmap_region(image, fx, fy, frameWidth, frameHeight,
		x - frameWidth / 2, y - frameHeight / 2, 0);
}

void Player::MoveUp()
{
	animationRow = 3;
	curFrame++;
	dirY = -1;
}
void Player::MoveDown()
{
	animationRow = 0;
	curFrame++;
	dirY = 1;
}
void Player::MoveLeft()
{
	animationRow = 1;
	curFrame++;
	dirX = -1;
}
void Player::MoveRight()
{
	animationRow = 2;
	curFrame++;
	dirX = 1;
}

void Player::ResetAnimation(int position)
{
	if(position == 1)
	{
		dirY = 0;
	}
	else
	{
		dirX = 0;
	}
}

void Player::Collided(int objectID)
{
	if(objectID == ENEMY)
		al_draw_rectangle(x + boundX,y + boundY,x - boundX,y - boundY,al_map_rgb(0,255,0),5);
		//lives++;
}


PARTNERS