4950 / DirtyGame

CSCI 4950 Project
csci4950.azurewebsites.net
3 stars 0 forks source link

PlayerScore incorrectly(?) calculated for SuicideBombers #338

Closed Quillwerth closed 10 years ago

Quillwerth commented 10 years ago

From GameSessionController.cs:

SET @TotalHealth = @TotalHealth + CASE @Monster WHEN 'MeleeMonster' THEN 240 WHEN 'SuicideBomber' THEN 1 WHEN 'LandmineDropper' THEN 120 WHEN 'RangedMonster' THEN 120 WHEN 'Grenadier' THEN 100 WHEN 'Flametower' THEN 100 WHEN 'SnipMonster' THEN 120 WHEN 'WallHugger' THEN 25 ELSE 0 END;

Note that addition for 'SuicideBomber'. That's not a true reflection of its health, which is 60. Why did we write it like this? @AOEChamp

AOEChamp commented 10 years ago

Because magic. Yeah, that needs to be updated when we change health. Idk how else the sql would know max health for monsters.

On Thu, Apr 17, 2014 at 3:15 PM, Patrick Severin notifications@github.comwrote:

From GameSessionController.cs:

SET @TotalHealth = @TotalHealth + CASE @Monsterhttps://github.com/Monster WHEN 'MeleeMonster' THEN 240 WHEN 'SuicideBomber' THEN 1 WHEN 'LandmineDropper' THEN 120 WHEN 'RangedMonster' THEN 120 WHEN 'Grenadier' THEN 100 WHEN 'Flametower' THEN 100 WHEN 'SnipMonster' THEN 120 WHEN 'WallHugger' THEN 25 ELSE 0 END;

Note that addition for 'SuicideBomber'. That's not a true reflection of its health, which is 60. Why did we write it like this? @AOEChamphttps://github.com/AOEChamp

Reply to this email directly or view it on GitHubhttps://github.com/4950/DirtyGame/issues/338 .

Quillwerth commented 10 years ago

Fixed it. Fix is sitting in "god", and was pushed to the server.