HF

Oke, code eens doorlopen en een GetLatest gedaan bij mezelf lokaal zodat bij verdere aanpassing van het TBX Sifonlades ticket ICT-878 de testen ook runnen met deze aangepaste code

De meegegeven parameter / data wordt ook in deze class meegegeven in de constructor en is dus al gekend en blijft ongewijzigd. Dumper kan dus zonder problemen naar constructor. Voglens mij kan de volledige Dumper...ToStream zelfs in een property gezet worden in de constructor van de klasse. Of kan dit opgelost worden via lazyload ?

We kunnen best niet veronderstellen dat er een crash is. Deze test bevestigd niet dat het gegarandeerd zal crashen... bv als er een verkeerde builder wordt gebruikt en er geen crash optreedt omdat de code niet doet wat we verwachten ?

Volgens ik de code van de logger zie, kan je zonder problemen ##class(vhLib.Logger).%New() hergebruiken en 2x een Error method oproepen zonder problemen. Binnen de klasse is het dezelfde applicatie / groep.

Huidige werking van de logging is zo dat er eigenlijk per te loggen item een nieuwe instantie moet opgevraagd worden.

Dumper werkt zo dat hij via constructor zijn data moet binnen krijgen. dus verplaatsen naar constructor is niet aan de orde

DRY Exceptie staat er al in dus het zou eerder GeefDetail moeten zijn , om toch een duidelijk verschil te hebben tussen de string en stream implementatie is er voor GeefDetailStream gekozen

Er bestaat een test voor dus zou niet weten wat je hier wil zeggen

Een voorstel om linefeeds gebruiken zodat de samenhorende blokken code duidelijker worden of zelfs methods zoals AssertSelectieKenmerk.

Hier wordt verondersteld dat het steeds een nieuwe context is. Als dezelfde context gebruikt wordt dan hebben we hier het risico dat er geen exceptie optreed en dat er nog gebruik gemaakt wordt van Berekeningsproblemen van een vorige melding waardoor de test onterecht kan slagen. We hebben hier dus een controle nodig dat de exceptie wel degelijk gegooid is.

idem testnaam

Als er nog wat tijd over is, testnaam verduidelijken.

Iterator naam verduidelijken, vb door BerekeningProblemenIterator.

.%New() naar constructor en private property van maken zodat het duidelijk is dat Dumper gebruikt wordt in deze klasse. Ingegevenkenmerken worden ook in constructor meegegeven en worden dus niet meer gewijzigd.

vhLib.Logger).%New naar constructor brengen en private property van maken..

Eventueel deze method opsplitsen in VoegToeIngegevenKenmerken, VoegToeBerekenDatum, VoegToeHalfFab,... kleine methods met duidelijke benamingen.

Meer duidelijk benamingen gebruiken zoals ExceptieDetail.

HalfFabIterator als naam gebruiken

Geen unittest voor deze implementatie.