Minimal amount of lacking documentation. Some important fields are missing documentation.
Define constants for “Genomen”, “Parents”, “Children”, “line.separator” etc.
Add @Override annotation where needed. Some are missing in DrawableNode, DrawableSegment….
There are useless assignments to local variables e.g. lineRead.
There are some complex classes, split tripple nested if/for/while loops into multiple methods.
Use line breaks to separate statements in GraphController, for example change the following to three lines instead of 1: Text idText = new Text("Genomes: "); idText.setLayoutX(x); idText.setLayoutY(65);.
There is commented code e.g. in SubGraph.java.
Do proper int to string conversion using toString().
Change lambdas to methods where you can.
+- 54% test coverage.
Check nullables for null.
Some pull requests were merged with only 1 approval and no discussion. Doing this on a large refactor can cause problems.
There are some builds failing, but better than last week.
Focus on writing more tests for the final sprint. It is 20% of the grade and can easily be improved by 2 points.
sprint planning:
code evaluation