Skip to content

Latest commit

 

History

History
18 lines (14 loc) · 826 Bytes

Refactor.md

File metadata and controls

18 lines (14 loc) · 826 Bytes

List of refactorings made

  • Remove unnecessary and unused variables.
  • Improve the Implementation of Grid User file by breaking into fragments and reusing the common elements from other files.
  • Removed redundant code from Battleship_Grid_Pane.java and merged its functionality with other features.
  • Player can now check itself if he has won or not instead of Game telling who is the winner.
  • Other minor refactorings suggested by IDE.

Methods used to determine points

  • Team Discussions
  • IDE suggestions
  • Peer programming

How to chose the targets

  • We checked if the refactor targets were backed by tests or not.
  • Refactorings were done in order to simplify the coding process and any possible future change which might be needed in Build 3.
  • For every phase, we checked if the code could be optimized.