...
De code voldoet aan de volgende clean code principes: "Be consistent, If you do something a certain way, do all similar things in the same way", "use searchable names", "Functions do one thing", "Don't comment out code. Just remove".
Eindoordeel Tussentijds
Als ik het product een cijfer zou geven op dit moment dan, geef ik een 6.5. De UI voldoet grotendeels aan de criteria die hiervoor gelden. De code kwaliteit is beoordeeld op basis van de principes die van toepassing zijn op de code, en voldoet de code aan een aantal principes. Na het verbeteren van de opmerkingen, is het mogelijk om van het cijfer een voldoende een goed te maken.
| Criteria | Cijfer | Toelichting |
|---|---|---|
Eindoordeel
| Criteria | Cijfer | Toelichting |
|---|---|---|
Typography | 8 | Text is duidelijk zichtbaar en er zijn niet meer dan 2 fonts. |
| Style conformity | 8 | Bulma framework toegepast waardoor er de huisstijl wordt aangehouden. |
| The functionality use: | 8 | Duidelijk zichtbaar wat de knop is en bevat functionaliteit, elementen zijn voldoende groot. |
Check the spelling | 8 | Er zijn geen spellingsfouten |
Gestructureerder uitleggen criteria, en het vergelijken met het oordeel
...
Voor het SDD zijn er in het plan van aanpak criteria opgesteld. Deze criteria staan in de onderstaande tabel. Per criteria geef ik een cijfer, de losse cijfers vormen gezamenlijk het eindcijfer.
| Criteria | Cijfer | Toelichting |
|---|---|---|
| Architecturaal overzicht | 6 | Overzicht geeft de architectuur weer, voor een hoger raad ik aan om de logo's te gebruiken van de te software e.d. omschrijvingen van de drie systemen mogen uitgebreider. |
| Deployment diagram | 8 | Het deployment diagram voldoet aan de volgende omschrijving (https://creately.com/blog/diagrams/deployment-diagram-tutorial/) Er is terug te zien welke devices, servers en verbindingsprotocollen er nodig zijn. Het diagram komt overeen met de omgevingen waar het zich in afspeelt. |
| Sub-systemen | 6 | Er is een omschrijving bij ieder subsysteem aanwezig, voor de database is iedere kolom toegelicht. |
| Database ontwerp | 5,5 | Alle data die in het domein thuis horen staan in de database. In het ISE-semester is er aangeleerd om verwoordingen op te stellen en deze met de opdrachtgever te bespreken. Dit is niet gedaan en zijn er op eigen inzicht tabellen aangemaakt. Hierdoor bevatten tabellen meer data dan nodig. Zo staat er in de werknemerstabel welke rechten de werknemer heeft. De rechten horen los van elkaar in aparte tabellen te staan. Wat resulteert dat de database in de tweede normaalvorm staat (https://nl.wikipedia.org/wiki/Databasenormalisatie) |
| Ontwerpbeslissingen | 7,5 | Beslissingen zijn opgebouwd op basis van een probleem, daarop is er voor iedere beslissing een oplossing gegeven met onderbouwing. Enkele beslissingen bevatten alternatieve oplossingen. |
| Commentaar verwerken | 8 | Al het commentaar verwerkt na de tussentijdse feedback. |
| Eindcijfer deelproduct | 6,8 |
Unit tests
Voor de unit tests zijn er in het plan van aanpak criteria opgesteld. Deze criteria staan in de onderstaande tabel. Per criteria geef ik een cijfer, de losse cijfers vormen gezamenlijk het eindcijfer. De criteria en de toelichting is op basis van de door mij geschreven test voor de front end.
| Criteria | Cijfer | Toelichting |
|---|---|---|
| Alle gemaakte test slagen voor 100% | 8 | Alle tests slagen. Geen verdere toelichting. |
| De test coverage is hoger dan 80% van de code waar het nodig is om te testen | 6 | De onderdelen die worden getest hebben een dekking van 80% en hoger. Er zijn componenten die ik heb gemaakt die geen tests hebben omdat het simpele functionaliteit bevat. Omdat er dus onderdelen zijn die niet worden getest geef ik voor deze criteria een 6. |
| Alle tests zijn zinnig en hebben toegevoegde waarde | 6 | Ik heb de tests voor de http requests uitgeschreven. Er zijn tests die alleen kijken http requests worden geaccepteerd. |
| Test Driven Development toegepast | 4 | Tests driven development niet toegepast. Ben pas begonnen met testen nadat ik Vue ben gaan beheersen. |
| Arrange-Act-Assert pattern (https://betterprogramming.pub/clean-code-with-unit-tests-5f28020828a5) | 6 | Er wordt een soortgelijk patroon toegepast waar een mock wordt voorbereid en daarna wordt vergeleken of de uitkomsten overeenkomen. |
| Evaluate a single concept per test (https://betterprogramming.pub/clean-code-with-unit-tests-5f28020828a5) | 7 | Per test wordt een onderdeel per functie getest |
| Er is maximaal 1 assertion per test (Summary of “Clean Code” by Robert C. Martin, z.d.) | 8 | Iedere test bevat maximaal een assertion. |
| Eindcijfer deelproduct | 6,2 |
Testrapport
TODO lijst deelproducten
...