Fixed Issue #3 and Issue #14. Also fixed Issue #10 and #12

ucsb-cs56-projects / cs56-games-roguelike

5 stars 13 forks source link

Fixed Issue #3 and Issue #14. Also fixed Issue #10 and #12 #15

Closed rick-lee closed 10 years ago

rick-lee commented 10 years ago

@andrewberls

~requested: 700

rick-lee commented 10 years ago

@andrewberls Breakdown of points: Issue #3 : 200 Issue #10 : 250 Issue #12 : 100 Issue #14 : 150

For Issue #10 : Had to redesign entire game system to incorporate level system.

andrewberls commented 10 years ago

This is looking really good! I really like the attention to detail with JUnit test coverage and the comments / Javadoc throughout. I think my main suggestion for improvement would be to examine "dense" lines or sections of code and really try to maximize the amount of code that can be extracted into separate variables and methods.

andrewberls commented 10 years ago

PROJ1 MOD REVIEW DONE

kjorg50 commented 10 years ago

Overall, this is fantastic, nice job! :clap: