ucsb-cs56-projects / cs56-games-battleship-alt

-
0 stars 2 forks source link

Clean up packaging and Source Code #6

Open mastergberry opened 10 years ago

mastergberry commented 10 years ago

This project is got a lot of awesome features, but the code is kind of messy. Take some time learning how this project works and try to clean up some of the classes. A good way to organize this would be to use MVC (Model View Controller). More info on MVC can be found here: http://en.wikipedia.org/wiki/Model%E2%80%93view%E2%80%93controller

There was another version of this project also https://github.com/UCSB-CS56-Projects/cs56-games-battleship-alt which shows some MVC usage of this project (it looks like that version is older and has less features).

When cleaning up the code please fix the packaging to edu.ucsb.cs56.projects.games.battleship

It also seems some of the features in this game are broken. Such as allowing the same type of ship to be placed multiple times. Rather than having a drop down menu it should cycle through the different ships. There might be other issues as you work on this project. Clean up as many of them as you can and ask for a re-evaluation if you find too many bugs to earn more points.

~Estimated 500 points.