TheCoderPal / terasoft-summer-reunited

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

UI15. As a user, I should see losing screen if my immunity reached zero where i can enter my name. #34

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Story Description:
1. Losing Screen with text box and a button for each letter.
2. Buttons for retry, main menu, etc.

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

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

Added methods AddScore, CheckScore
Added file HighScores.txt

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

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

Added writing to file. 

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

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

added buttons handlers

Original comment by vendett...@gmail.com on 5 Aug 2012 at 11:18

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

changed condition in OKButton Listener

Original comment by vendett...@gmail.com on 8 Aug 2012 at 1:11

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

added debug line

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

GoogleCodeExporter commented 9 years ago
HandCursor Hand rename to hand, same for Font and TextOrigin
if (scores.Count < 10)
                return true;

            if (Score > Int32.Parse(scores[scores.Count - 1].Split(',')[1]))
                return true;
use ||

ContentManager Content = ScreenManager.Game.Content;
set ContentManager as a variable with small letter outside loadcontent method
call the super class methods using base
StreamWriter sw rename variable and use var
remove the debug lines
instead of all the lines for initializing each button adding an event handler 
and addding it to  the buttons list and initializing it
create a method that takes a button and a string its name A,B,C or whatever and 
do all the necessary things instead.. add a switch case on the string to 
diffrentiate the events and initialize calls and create the event for each, or 
you can make an abstract class for letters which already has these events, but 
it looks ugly this way
remove content from Load methods
 SpriteBatch spriteBatch = ScreenManager.SpriteBatch;
dont add intitializations in draw method its called according to gametime it 
might cause lag, put it in the loadcontent method
make variables private

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

GoogleCodeExporter commented 9 years ago
1. Done
2. Why ? Contentmanager is only used in loadcontent.. why does it have to be an 
instance variable ? 
3. No comment.
4. Why do I use var ? This is a streamwriter.
5. Done
6. No, simply because there is a difference between initializing.. loading.. 
etc. I cannot create a method that does the same. There are differences in 
positions in initializing. Looks ugly ? thats why there are regions surrounding 
them
7. No. LoadContent method normally takes a contentmanager as input.
8. huh?
9. huh? 

Original comment by vendett...@gmail.com on 15 Aug 2012 at 1:56

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

implementation review 

Original comment by vendett...@gmail.com on 15 Aug 2012 at 1:57

GoogleCodeExporter commented 9 years ago
I told you make a switch statement on the string and do all the necessary stuff 
for each button according to its name/type, but it looks pretty ugly this way

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

GoogleCodeExporter commented 9 years ago
If i do what you say im gonna write as much code and its gonna look as ugly... 
check playscreen if you dont believe me.. except that it'll be 26 cases.. plus 
i told you thats why there are regions surrounding them

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

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

fixed exception

Original comment by vendett...@gmail.com on 15 Aug 2012 at 4:50