khaledosman / terasoft-summer-reunited

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

UI7 : As a user, I should see my immunity bar with animated changes. #15

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
As a user, I should see my immunity bar with animated changes.

Original issue reported on code.google.com by hollaxy...@gmail.com on 25 Jul 2012 at 8:52

GoogleCodeExporter commented 9 years ago

Original comment by hollaxy...@gmail.com on 30 Jul 2012 at 8:12

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

Added the implementation of Immunity bar story

Original comment by hollaxy...@gmail.com on 31 Jul 2012 at 11:57

GoogleCodeExporter commented 9 years ago
you should handle the case such that the max value is not too small or not too 
big.. throw an exception or add an if statement
"currentvalue=100", add it to class constants and use it from there
remove content from loadcontent method and use screenmanager attributes
replace spritebatch with gametime in draw and use the spritebatch from 
screenmanager
the update method should be the place where you use shirin's methods to check 
for collision and update the bar accordingly, copy your work there

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

GoogleCodeExporter commented 9 years ago
Actually by what you saying will making the bar not generic one to be used 
later as library as i planned.Therefore what you say about shirins method will 
not be done :)

I will look at the other comment and see what i can do for it.

Original comment by hollaxy...@gmail.com on 14 Aug 2012 at 10:13

GoogleCodeExporter commented 9 years ago
I understand but we already agreed all the load update draw methods should be 
the same as those in class GameScreen and use the screenManager attributes like 
I showed an example in class PauseScreen, and draw method should use GameTime 
by default already, the xna default autogenerated methods for load has an empty 
constructor and its update and draw methods use GameTime

Original comment by khaled.osmaan on 14 Aug 2012 at 10:21

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

Deleted the begin and end in draw of bar and score.

Original comment by hollaxy...@gmail.com on 16 Aug 2012 at 11:49

GoogleCodeExporter commented 9 years ago
use getters and setter fields instead of methods... you can also define this 
logic there.

Original comment by vendett...@gmail.com on 17 Aug 2012 at 10:08