...

Kwaliteitsoordeel Deelproducten

 Onderzoeksverslag

...

 Software Design Description (SDD)

Het SDD is naar mijn mening uiteindelijk van goede kwaliteit, echter is dit met veel moeite zo gebeurd. Er waren een aantal verkeerde opvatting over met name de sub systems waardoor dit in de laatste week nog omgegooid en aangepast moest worden. Dit had eerder gekund als we als groep vaker feedback hadden gevraagd en het ook bij hadden gehouden met de actuele stand van zaken. De rest van het document is door het project heen wel beter bijgehouden en is dan ook goed uitgewerkt. Kijkend naar de eisen die vernoemd staan zijn deze allemaal gehaald. Ook voldoet het aan het meegegeven SDD Template dat uitgereikt is door de HAN. 

Unittests

De geschreven unittests voor het project project zijn belangrijk om te kijken of de applicatie als geheel nog werkt. Als er unittests zijn die niet slagen dan betekent dat dat er iets is aangepast in de code waardoor er een bepaalde functionaliteit niet meer werkt. Dit is dan ook waarom er is afgesproken dat er altijd unittests mee worden gecommit als er nieuwe code op de develop branch komt te staan. Zodra alle tests nog steeds slagen wanneer je nieuwe functionaliteit toevoegd dan weet je dat alles goed is gegaan. Doordat wij als projectgroep zo goed mogelijk deze afspraken hebben gevolgd heb ik er vertrouwen in dat er ook kwalitatief goede unittests zijn geschreven voor iedere aparte funtionaliteit. Zo is er niet alleen voor het succes scenario een unittest geschreven bij iedere functionaliteit maar ook voor wanneer er verkeerde informatie wordt ingevoerd. Dit is goed te zien in het bestand DeclaratieServiceTest.java. In deze code wordt er bijvoorbeeld niet alleen getest of een declaratie goed gaat met de goede informatie, maar ook of het fout gaat als er een niet bestaande/verkeerde locatie wordt ingevoerd.

Code

Voor het beoordelen van een stuk code heb ik ervoor gekozen om de code van het bestand LoginService.java te beoordelen. De code in dit bestand voldoet voor een groot deel aan de opgestelde kwaliteitseisen die zijn opgesteld. Er is namelijk traceability naar de specifieke requirement/Jira taak doordat er vanuit een Jira taak een branch is aangemaakt op de bitbucket. Hierdoor kan er altijd gezien worden bij welke Jira taak de branch hoort en ook andersom. Verder is de code kwaliteit zelf ook van volgoende niveau doordat er als projectgroep is afgesproken en ingesteld dat er minimaal 2 groepsleden de code moeten reviewen voordat dit ook pas echt wordt toegevoegd aan het geheel. Door op deze manier 2 andere personen kritisch te laten kijken naar de code zie je meer dan wanneer je alleen zelf er naar kijkt en hier een oordeel over maakt. Echter staat qua taal nederlands en engels af en toe door elkaar maar dit komt omdat er bepaalde termen zijn waarbij er liever engels werd aangehouden zoals bij getters en setters. Verder zijn alle termen die betrekking hebben tot het bedrijf wel in het nederlands zoals werkplekken en werknemers. Dit is goed te zien in het bestand WerkplekService.java.

...