Quote:Original post by TheUnbeliever
sprite2[].x + 1;

What is sprite2[] meant to do? Your problems have nothing to do with the fact that you're passing it to a function - you have to index into an array. I think what you're looking to do is iterate over the array and test for collision for every element in it.

However, because arrays decay to pointers when passed to functions, you lose information about the length of the array. You need to either a) pass the length of the array to the function then loop over it, or b) pass a wrapper around an array that makes it behave sensibly, and won't lose information like this, such as Boost.Array.

I'd rewrite to give an example, but since I don't know exactly what the return value of the function, or of IntersectRect, represents, I can't.

Quote:Original post by MJP

Quote:Original post by xM1k3x
So your saying I cant just call the function like this:

for(int count = 0; count < ENEMYNUM; count++)
{
if(CollDet(Ship, EnmeyFire[count]))
{
ship->Destroy(); //or soomething
}
}

So I couldnt do that and have the function just take the index that is provided to it?

You can do that, you just want to make the CollDet function accept a PlayerShip as a parameter (not an array of PlayerShip's).

int CollDet(PlayerShip sprite1, PlayerShip sprite2){  RECT rect1;  rect1.left = sprite1.x + 1;  rect1.top = sprite1.y + 1;  rect1.right = sprite1.x + sprite1.width-1;  rect1.bottom = sprite1.y + sprite1.height-1;  RECT rect2;  rect2.left = sprite2.x + 1;  rect2.top = sprite2.y + 1;  rect2.right = sprite2.x + sprite2.width-1;  rect2.bottom = sprite2.y + sprite2.height-1;  RECT dest;  return IntersectRect(&dest,&rect1,&rect2);}