LadeUitsparingHelper.cls.xml

copy-paste typo (2x) :
#dim ... As MVX-kenmerken ??

copy-paste 2x zelfde

copy-paste 2x zelfde

Mijn eerste review-opmerking ging zijn : "In een AssertEquals-expressie zou geen $Select() mogen staan."
Dat topic wil ik eventueel nog met jou bespreken.

Door dit op te merken, viel m'n oog wel op een typo in bovenstaande lijn: RugBreedteX en in de $Select() staat BreedteM :-O

Bij voorkeur Z1 en Z2 verschillend kiezen.
idem voor X en Y

Uitsparing Breedte.M moet je verplaatsen naar Dubbele uitsparing.

Deze IF legt een onnodige dependency, en heeft geen meerwaarde binnen de converter.
Integendeel, bij uitbreiding voor LBX sifon of TBX sifon moet je hier iets toevoegen, terwijl de code binnen de IF toch "universeel" is.

Ook de MatKenmerken hebben property "NietMeeleveren", dus ik zou deze method wel laten oproepen, ook al is de impl dan enkel met de eerste en de laatste lijn : %New() en Quit.
Oftewel rechtstreeks in de method Converteer() , dan mag de commentaar hier volledig weg.

VH code conventions : VulAanMatKenmerken()

Als het object MatKenmerken één en dezelfde oref blijft (dus geen %Clone ofzo) dan is het niet nodig om de SetAt() te doen, want de pointer in de GekoppeldeKenmerken verwjist nog steeds naar hetzelfde object in memory.

Een kleine bedenking:
ik ben niet helemaal zeker of de method wel thuishoort in deze klasse, omdat die enkel geschikt is voor MatKenmerken.

Voorlopig is dit oké, en functioneel zeker geen probleem.
Ik wil niet verder uitweiden nu, over deze bedenking, dus enkel als review-opmerking, en om te vermijden dat we vanalles in deze klasse gaan steken dat niet voor alle IngegevenKenmerken van toepassing is.

Er is geen IF nodig, want :
if "true" or "false" --> altijd dus
:-P

toegevoegde parameter "IsBerekendViaDotNet" ook doorgeven via ##super()

Ik snap de redenering maar ik vind dit niet correct. Mss eens in de groep gooien en beslissen wat we hiermee doen?

Vond het eerst zelf ook een beetje vreemd, waarom da aangepast is :-P
Verklaring vrij eenvoudig :
Naam moet consistent zijn met de andere vhTest.Utils ... Kenmerken() klassen, omdat de UT's voor de convertoren dit oproepen:

Method "Test: TAX Standaardlade"()
{
Do ..AssertConverter("StandaardLade")
}

Method "Test: TAX BinnenLade"()
{
Do ..AssertConverter("BinnenLade")
}

Method "Test: TAX LadeMetVeelExtras"()
{
Do ..AssertConverter("LadeMetVeelExtras")
}

Anders falen de UT's ... spijtig genoeg.
(normaal gezien moet dan de UT-impl aangepast worden en niet de code, maar deze wijziging heeft geen invloed op de werking van de nietmeeleverens, denk.)
Deze tactiek was ook al toegepast bij LBX door één van m'n voorgangers.

De "ByRef" is hier om aan te geven dat het EconKenmerken-object zal aangepast worden.
Technisch is het niet fout om da als .local door te geven. Objecten worden trouwens altijd als "pointer naar geheugenplaats" doorgegeven.

Principe is bovendien overgenomen van de 2 lijnen eronder, en dat leek mij niet verkeerd om het zo aan te duiden.

Laat zeker weten als je hier toch aan twijfelt.

Zal bij volgende opkuis verwijderd worden.