You are on page 1of 2

Deelopdracht 2

Groep: 12-03 Goed resultaat met veel kleine foutjes. Kijk de update na.

Opvolging feedback deelopdracht 1


De opmerkingen van deelopdracht 1 zijn weggewerkt Feedback goed verwerkt

Deelopdracht 2

Algemene Feedback

H2 databank
correct geconfigureerd

Club - Pagination
correct geïmplementeerd Aantal pagina's is statisch. Gebruik Pageable voluit.

Collectie - Pagination
correct geïmplementeerd Aantal pagina's is statisch. Gebruik Pageable voluit.

Rest Controller - algemeen


foutafhandeling in json (invalide invoer, delete/update niet-bestaand item)
correcte requestmapping (bijv deletemapping, putmapping)
correct gebruik van parameters (@pathvariable, @requestbody, @requestparam) PathVariable wordt ook gebruikt waar RequestParam beter is.

Club - Rest Controller


overzicht, delete, sorteer op naam, zoek met max aantal: geeft lijst terug (url slide 22)

Collectie - Rest Controller


overzicht, update, sorteer op land, zoek jaartal: geeft lijst terug (url slide 22) Sort op country staat omgekeerd. Delete geeft geen lijst terug. Update voegt nieuw element toe in plaats
van bestaand element te updaten.

Verzamelaar (geen web-interface)


alle velden verplicht, naam.length>3, 18<leeftijd<110, naam uniek Keuriger als de volledige naam uniek moet zijn
REST api: add, overzicht, update, delete, zoek op regio, zoek op naam-voornaam: geeft lijst Update voegt nieuw element toe in plaats van bestaand element te updaten. Zoek URLs bestaan niet
terug (url slide 22) omdat RequestParam niet gebruikt wordt.
Testing: verzamelaar, repo, service, restcontroller (happy/unhappy scenario's, mocking, Structuur is goed. Probeer testen toe te voegen voor de dingen die nu niet oké zijn zodat je coverage
design pattern) verbeterd.
Munt (geen web-interface)
alle velden verplicht, waarde > 0, jaartal <= today(), naam uniek Geen validatie op jaartal.
REST api: add, overzicht, update, delete, zoek op jaartal, zoek op land-contains-string: geeft Update voegt nieuw element toe in plaats van bestaand element te updaten. Zoek URLs bestaan niet
lijst terug (url slide 22) omdat RequestParam niet gebruikt wordt.
Testing: munt, repo, service, restcontroller (happy/unhappy scenario's, mocking, design Structuur is goed. Probeer testen toe te voegen voor de dingen die nu niet oké zijn zodat je coverage
pattern) verbeterd.

Authentication - Authorization
eigen login page met valide html (mag homepagina zijn), logout knop staat op logische Verkies expliciete login-pagina die steeds beschikbaar is. Nu geef je beetje functionaliteit weg die enkel
plaats, aandacht voor bruikbaarheid voor ingelogde gebruikers beschikbaar is. Geen logout.
gebruikers/wachtwoorden zoals gevraagd
authorization: navigatie en knoppen zijn afgeschermd (gebruiker ziet enkel toegelaten links Knoppen zijn zichtbaar ook al heb je geen rechten. User kan bijvoorbeeld Create Club zien. Zie ook
en knoppen) bovenstaande opmerking. Eleganter om Admin ook USER rol te geven in plaats van permissies dubbel bij
te houden.
authorization: pagina's zijn afgeschermd (wat gebeurt er als gebruiker niet-toegestane url Tip: eindig security configuratie met een generieke denyAll om zeker te zijn dat er geen ongewenste links
manueel ingeeft?) open staan.
rest controller is beschikbaar
h2 databank beschikbaar via url

Error Handling
geen error white label, eigen error page, enkel indien nodig gezien bruikbaarheid
error page afhankelijk van opgegooide fout (indien van toepassing: met gepaste foutmelding)

You might also like