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() en createFemaleAdultList() uit ListMaker duplicate code bevatten. Deze code bevat feitelijk twee checks:
- Of een persoon ouder dan 18 is
- Of een persoon MALE/FEMALE is
Creëer voor beide checks een methode die de check uitvoert en roep deze aan vanuit createMaleAdultList()
en createFemaleAdultList().
Run de Unittests om er zeker van te zijn dat alles nog werkt.
Mogelijk heb je voorgaande twee 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() en createFemaleAdultList() 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() en createFemaleAdultList() 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/FEMALEis
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 maken 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 of FEMALE 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.