Open GoogleCodeExporter opened 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
This issue was updated by revision r29.
Added writing to file.
Original comment by vendett...@gmail.com
on 2 Aug 2012 at 1:26
This issue was updated by revision r123.
added buttons handlers
Original comment by vendett...@gmail.com
on 5 Aug 2012 at 11:18
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
This issue was updated by revision r205.
added debug line
Original comment by vendett...@gmail.com
on 8 Aug 2012 at 1:40
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
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
This issue was updated by revision r236.
implementation review
Original comment by vendett...@gmail.com
on 15 Aug 2012 at 1:57
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
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
This issue was updated by revision r246.
fixed exception
Original comment by vendett...@gmail.com
on 15 Aug 2012 at 4:50
Original issue reported on code.google.com by
vendett...@gmail.com
on 26 Jul 2012 at 1:30