Deze oefening is deel van de DEA Course aan de Hogeschool Arnhem/Nijmegen. Onderwerp is het bekend raken met de Lambda-expressies.
In deze oefening refactor je bestaande code, waarbij je uiteindelijk ook functies omschrijft naar Lambda Expressies om de code nog leesbaarder te maken. Run na elke refactoring de aanwezige unit tests om te verifieren dat de code nog correct werkt (en zo niet backtrace dan en fix het; gebruik van git versiebeheer kan dit vergemakkelijken).
De oplossingen zijn als verschillende branches beschikbaar.
De huidige codebase bevat een Magic Number. Vervang deze door een constante.
Run de Unittests om er zeker van te zijn dat alles nog werkt.
Merk op dat de methodes createMaleAdultList(), createFemaleAdultList()
en createOtherwiseAdultList() uit ListMaker duplicate code bevatten. Deze
code bevat feitelijk twee checks:
- Of een persoon ouder dan 18 is
- Of een persoon MALE/FEMALE/OTHERWISE is
Creëer voor beide checks een methode die de check uitvoert en roep deze aan
vanuit createMaleAdultList(), createFemaleAdultList() en
createOtherwiseAdultList().
Run de Unittests om er zeker van te zijn dat alles nog werkt.
Mogelijk heb je voorgaande methodes als private methodes toegevoegd aan
de ListMaker. Dat is niet waar ze thuishoren. Verplaats beide methodes naar de
klasse waar ze thuishoren.
Run de Unittests om er zeker van te zijn dat alles nog werkt.
Ondanks de verbeteringen bevatten createMaleAdultList(),
createFemaleAdultList() en createOtherwiseAdultList() nog steeds duplicate
code. Creëer een private methode,
genaamd createAdultList(List<Person> allPerson, Gender gender), die als tweede
parameter de Gender krijgt waarvoor deze een lijst moet maken. Roep deze
methode vervolgens vanuit createMaleAdultList(), createFemaleAdultList () en createOtherwiseAdultList() aan.
Run de Unittests om er zeker van te zijn dat alles nog werkt.
De nieuwe methode createAdultList() bevat een for-loop. Vervang deze door
gebruik te maken van de forEach van de Stream API.
Run de Unittests om er zeker van te zijn dat alles nog werkt.
Merk op dat er nu twee checks in de body van de forEach staan:
- Of een persoon ouder dan 18 is
- Of een persoon
MALE/FEMALE/OTHERWISEis
Dat kan je netter oplossen. Juist voor deze situatie heeft de Stream API
de filter methode. Deze methode heeft een Predicate als parameter, wat een
Lambda-expressie is die een boolean retourneert.
In dit onderdeel gaan we een Lambda-expressie maken die checkt of een Person
ouder dan 18 is. Deze passen we vervolgens toe als een filter, waarmee we
de forEach vervangen. Functioneel is het dan nog niet compleet (de tweede
check ontbreekt nog) en je breekt dan ook de unittests, maar dat los je in de
volgende oefening op.
Creëer een Predicate die er als volgt uitziet:
Predicate<Person> isAdult = (person) -> {/* Hier volgt de implementatie van de check */}
Deze Predicate is een gewone variabele en kun je toevoegen als private
instantievariabele aan de ListMaker klasse. Verwijder de forEach en gebruikt
de Stream API met een filter (en de isAdult Lambda-expressie) en uiteindelijk
de Collectors.toList() om een gefilterde lijst te maken die nu enkel Person
-instanties bevat die ouder zijn dan 18.
Run de Unittests om er zeker van te zijn dat alles nog werkt.
Gebruik de kennis van onderdeel 6 voor het maken van een Predicate die bepaalt
of een Person MALE, FEMALE of OTHERWISE is. Deze is wat lastiger,
aangezien je in Oefening 4 de Gender als parameter hebt toegevoegd en je
daarmee eigenlijk twee parameters (de Gender en de Person) aan
je Predicate moet meegeven. Dit staat de Interface van Predicate echter niet
toe. Er zijn dan ook verschillende manieren dit op te lossen; het is aan jou!
Run de Unittests om er zeker van te zijn dat alles nog werkt.