Skip to content

Commit 444a746

Browse files
authored
Create Refactor.md
1 parent 412644f commit 444a746

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

Refactor.md

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
### List of refactorings made
2+
3+
* Remove unnecessary and unused variables.
4+
* Improve the Implementation of Grid User file by breaking into fragments and reusing the common elements from other files.
5+
* Removed redundant code from Battleship_Grid_Pane.java and merged its functionality with other features.
6+
* Player can now check itself if he has won or not instead of Game telling who is the winner.
7+
* Other minor refactorings suggested by IDE.
8+
9+
### Methods used to determine points
10+
* Team Discussions
11+
* IDE suggestions
12+
* Peer programming
13+
14+
### How to chose the targets
15+
16+
* We checked if the refactor targets were backed by tests or not.
17+
* Refactorings were done in order to simplify the coding process and any possible future change which might be needed in Build 3.
18+
* For every phase, we checked if the code could be optimized.

0 commit comments

Comments
 (0)