Sorry for the late reply! There isn’t anything standing out to me that would slow the code down, other than I’m not sure why you are making a second copy of the texture object?

The only thing I’d say you should bare in mind is that pixel by pixel collision detection is quite an expensive operation in itself, but until such time as you are beginning to experience performance issues try not to get too wrapped up in optimisation.

There are many conflicting opinions on this matter, but I’d personally say don’t get too hung up in optimisation before hand as otherwise you can be spending significant amounts of time and in some cases making code unreadable for the sake of getting rid of performance problems that wouldn’t actually occur.

Glad the code was of use to you

added 2 new fields to the class called frames (default=1) and currentframe (default=0), changed the rect fuction to return (0,0,width/frames,height)

and added these lines to the IsColliding

Texture2D temptex = texture;
Color[] temtextureData = new Color[temptex.Width * temptex.Height];
temptex.GetData(0, new Rectangle(currentFrame * (texture.Width / frames), 0, texture.Width / frames, texture.Height), temtextureData, 0, (texture.Width / frames * texture.Height));


then replaced

// if (IntersectPixels(this.Transform, this.Texture.Width, this.Texture.Height, this.TextureData, collidable.Transform, collidable.Texture.Width, collidable.Texture.Height, collidable.TextureData))

sorry if the code formatting is all messed up im not sure how to make it look right. i hardly ever post anything and could not get the code formatting thing you used above to work right (there was no link to paste where it said to paste “following HTML script tag:”

Thank you for your time. sorry about any spelling/grammer errors