Code review 2
Deze code review is uitgevoerd door Joris Philipsen op 22-11-2018
Code review
- In
GraphAPIUtils
heeft deconvertAmount
methode geen duidelijke beschrijving. - Op sommige plekken worden deprecation warnings gesupressed, dit is niet nodig voor zo'n nieuw project, dus dit oplossen.
- In
GraphPolylineController
staat een try/catch een geeft een harde error terug, terwijl dit beter afgevangen kan worden. - Grote functies in
GraphModifier
meer opsplitsen. - CrossOrigin toestaan voor alles, of gebruikmaken van een config.
DatabaseConnection
instellingen voor client in database kunnen in een config.
Acties
- In
GraphAPIUtils.convertAmount
meer comments en betere variabele namen. - Niet gebruikte classes met gesupressede warnings verwijderd.
- Betere API error messages in de
GraphPolylineController
- Functies in
GraphModifier
meer opgesplitst. - CrossOrigin staat alles toe.
- Een
system.conf
bestand en configuratie toegevoegd.