TheCoderPal / terasoft-summer-reunited

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

As a User, I can see a boss fight screen containing immunity bar and virus bar. #52

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Story Description:
1.Implement the screen
2.Add screen design
3.Add immunity bar and virus bar

Original issue reported on code.google.com by khaled.osmaan on 27 Sep 2012 at 2:19

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

status inProgress
added basic implementation of the screen

Original comment by khaled.osmaan on 27 Sep 2012 at 2:21

GoogleCodeExporter commented 9 years ago

Original comment by khaled.osmaan on 27 Sep 2012 at 7:15

GoogleCodeExporter commented 9 years ago
Imp Review:

1)When you have multiple variables with same type,write them in one line ex:
private int screenWidth;
private int screenHeight;
private int virusHealth;

to : private int screenWidth,screenHeight,..

2)do not initialize instance variables before constructor.

Original comment by tamer.na...@gmail.com on 5 Oct 2012 at 12:12

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

Added interface ICamera2D and class Camera2D to do some camera effects in the 
boss fight screen

Original comment by khaled.osmaan on 7 Oct 2012 at 1:28

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

implementation notes fixed

Original comment by khaled.osmaan on 7 Oct 2012 at 1:32

GoogleCodeExporter commented 9 years ago
Review:  
1. You have no background :/
2. You dont even draw the virus's health
3. Message and the gradient box are ??? 
4. You clearly havent tested your stuff 
5. Please seperate lines with spaces and so to make your code readable.

Original comment by vendett...@gmail.com on 7 Oct 2012 at 11:05

GoogleCodeExporter commented 9 years ago
1. I really suck at picking backgrounds even with the settings screen, I cant 
find any relevant ones
2. I'm not sure how to deal with tamer's bar, I copied the bar part from the 
playscreen I guess
3. message and gradient boxes are the popup messages that will tell the player 
the boss level and congrats you have won the boss or whatever, they won't be 
added this sprint I believe I just added them while brainstorming...

Original comment by khaled.osmaan on 8 Oct 2012 at 12:12

GoogleCodeExporter commented 9 years ago
Just chose a plain background 7ata... ay 7aga to seperate it from the play 
screen 3shan theyre overlapping
The bar.. ure using it right... theres no need for this line though... 
virusBar.Update(gameTime);
and you forgot to draw -_- 
Change the message and the box aw remove it for now 3shan shaklaha mesh lateef
Remove any logic from the draw method.. the calculations and all.. put them f 
loadcontent or elsewhere.. but not in draw
I think you should add text 3shan twada7 lel user which is his health bar and 
which is the boss's ... and lastly.. just please combine el 7agat elly zai ba3d 
wara ba3d... 3shan the code looks like a mess
ya3ny maslan f load content

            font = content.Load<SpriteFont>("SpriteFont1");
            virusBar.LoadContent(content);
            rightSword = content.Load<Texture2D>("Textures//sword");
            leftSword = rightSword;
            bump = content.Load<SoundEffect>("Audio//bump");
            Texture2D shieldSprite = content.Load<Texture2D>("Textures//shield");

The lines that are content.Load<... etc. wara ba3d... w ba3deen line fady .. w 
ba3deen el ba2y and so ... just to make it more readable.. and thank you :) 

Original comment by vendett...@gmail.com on 8 Oct 2012 at 2:09

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

integrated the camera in the screen and fixed some implementation issues

Original comment by khaled.osmaan on 9 Oct 2012 at 6:11

GoogleCodeExporter commented 9 years ago
You have problem in your boss screen ,it is not loaded and getting this 
exception: 

http://i.imgur.com/Ycer0.png

Original comment by tamer.na...@gmail.com on 11 Oct 2012 at 5:13

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

commented line cam.Scale+=0.1 to fix the exception, however there's another 
exception in shirin's code

Original comment by khaled.osmaan on 11 Oct 2012 at 9:56

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

Added method restoreGame in PlayScreen, in BossFightScreen I fixed the bar and 
camera bugs, also added a button to be drawn and updated when the boss died 
which switches back to the playScreen 

Original comment by khaled.osmaan on 13 Oct 2012 at 6:13

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

commented two lines that switches to boss fight screen used when testing

Original comment by khaled.osmaan on 13 Oct 2012 at 6:19

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

made the hand and button be drawn on winning the boss

Original comment by khaled.osmaan on 13 Oct 2012 at 6:26

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

made health in class boss a field to use it in the virus bar in the boss fight 
screen, also used a new flag to check if the boss died instead of boss.bossdied 
because it had some problems

Original comment by khaled.osmaan on 14 Oct 2012 at 1:32

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

Integrated the camera, fixed its bugs, deleteed interface IFocusable and added 
a vector2 position focus in the camera instead. deleted an unused variable from 
class UserAvatar and removed unused usings, made the hand be drawn after the 
button sothat it doesnt disappear behind it, made the background music loop 
after both songs end.

Original comment by khaled.osmaan on 14 Oct 2012 at 8:58