I came across this project on a code review subreddit and thought it could be a good case study on how I would try to improve this code.
Here is a series of posts on things I would do.
- BattleShip Part 1: Initial Review / Functional Requirements
- BattleShip Part 2: Breaking out different components
- BattleShip Part 3: App start and User Interface
- BattleShip Part 4: Game Cli Test
- BattleShip Part 5: Game Mechanics
- BattleShip Part 6: Ship Placement
- BattleShip Part 7: Ship Placement Tests
- BattleShip Part 8: Addressing boundary issue
- BattleShip Part 9: Unit tests for Game class
- BattleShip Part 10: Tests for Game Cli
- BattleShip Part 11: Include columns and rows in display
Leave a comment