search

khaledosman / terasoft-summer-reunited

Automatically exported from code.google.com/p/terasoft-summer-reunited
0 stars 0 forks source link

UI2. As a system, I should have a sprite animation player. #29

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Story Description:
1.Sprite Animation Class.

Original issue reported on code.google.com by vendett...@gmail.com on 26 Jul 2012 at 1:24

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r12.

updated solution

Original comment by vendett...@gmail.com on 31 Jul 2012 at 9:42

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r17.

updated solution

Original comment by vendett...@gmail.com on 1 Aug 2012 at 2:40

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r18.

updated solution

Original comment by vendett...@gmail.com on 1 Aug 2012 at 2:41

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r24.

uploaded class SpriteAnimation

Original comment by vendett...@gmail.com on 1 Aug 2012 at 7:43

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r28.

updated solution

Original comment by vendett...@gmail.com on 2 Aug 2012 at 1:17

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r32.

updated solution

Original comment by vendett...@gmail.com on 2 Aug 2012 at 3:17

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r61.

Modified for the use of everyone

Original comment by vendett...@gmail.com on 3 Aug 2012 at 1:21

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision r82.

added methods to help in collision detection

Original comment by vendett...@gmail.com on 3 Aug 2012 at 9:24

GoogleCodeExporter commented 9 years ago 
rename variables that start with a capital letter..
replace draw(spritebatch) with draw(gametime)..
Color[] c rename variable to something meaningful and use var

Original comment by khaled.osmaan on 14 Aug 2012 at 9:31

GoogleCodeExporter commented 9 years ago 
for class Player
remove unused usings
make variables private
remove content from load
replace spritebatch with gametime in draw
in method GetBoundingRectangle()
use paranthesis around arithmetic operations
use bool instead of Boolean

Original comment by khaled.osmaan on 14 Aug 2012 at 9:37

GoogleCodeExporter commented 9 years ago 
1,2,3.. Done 

4,5  :D 
No. Retardhead this isnt a screen.. its not associated with a screen manager... 
and nothing says loadcontent shouldnt take any parameters or so.. Its just a 
method.. and it happens to be with the same name as XNA's method. 

6. Done
7. @Shirin

Original comment by vendett...@gmail.com on 15 Aug 2012 at 2:14

GoogleCodeExporter commented 9 years ago 
again... graphics attributes shouldnt be passed as parameters in your methods, 
the screen manager is there for a reason you know... and the draw method 
doesn't use spriteBatch by default so why change it? I don't care whether if 
its a scree or not, that doesn't give you the right to change XNA c# 
conventions, please search over the internet if you found a draw method that 
takes spritebatch as a parameter I'll smash my face on the nearest wall. and 
don't argue if you're not sure and be polite.

Original comment by khaled.osmaan on 15 Aug 2012 at 6:08

GoogleCodeExporter commented 9 years ago 
rofl.

Original comment by vendett...@gmail.com on 15 Aug 2012 at 2:24