ECE 2524

Introduction to Unix for Engineers

Review of Battleship

Usage

The [sirajali05 ece2524finalproject](https://github.com/sirajali05/ECE2524-Final-Project] was very impressive. The idea of it was really unique and it was very well made for the time we were given. I had some trouble actually trying to set up the game to actually work. It could have been worded better so its clear for a new user. The game compiled and ran fine. I really liked the game and was really impressed. The only error that I noticed is that if you put an invalid option, then the program goes to an infinite loop. Obviously this was not the actual Battleship game, but a this was definitely an efficient prototype.

Style

The programming was very well done. It was very specific as to what class had to do what specific task. Comments could be added to improve a better understanding of the code. The .h and .cpp files have appropriate names and adds to the level of organization. The only thing I did not like so much was how the main.cpp had a lot of material. I believe that the work could be spread into the other files or organized better.

Philosophy

The program is fairly user friendly. The usage has some issues with invalid input, but the code structure overall is well put together. The code is seperated into different parts like Board and player. This is part of the Unix Philosophy and it was fairly easier to understand.

Overall, this is a good project and has the potential to become a great one. I suggest that fixing some bugs are necesssary. Also I believe that more visuals could be improved like adding a real ship and whole layout of board. I enjoyed playing the game and it was a very creative project that has a lot of potential.