Maatwerk

Opgelet met de uitdrukking %ClassName(1) = "..."
want dit zal falen op afgeleide klassen --> beter object.%Extends(...) alsook gebruik maken van #class(DOM.PM.Maatwerk.Calc.Common.impl.TBXKenmerken).%ClassName($$$True)

De method naam kan wel tot verwarring leiden, want wat met een LBX-kenmerk?
Ik zou het stukje "Tbx" weglaten en de implementatie lichtjes herschijven, met een lokale variabele en een lijn "If ... %Extends (TBXKenmerken) then ... "

Domme opmerking, maar om de "issen" te onderscheiden zet ik meestal haakjes rond de conditie. LES 1 in caché : haakjes is altijd veiliger dan geen haakjes

LegacyPartijApi :
Door deze implementatie wordt de method GeefLegacyPartijAPI() van de context minstens 5x opgeroepen. Dit valt nu eenmaal op door de aanpassing in deze method.
Dus boyscout-gewijs : graag de LegacyPartijApi aanmaken in de constructor, op de gekende wijze. Voor mij hoeft er nog geen $$$Inject() te staan, gewoon newen is oké.

VHoss : A of B
Misschien VerpakkingType eerst in een variabele steken. Dit maakt de "OF"-condities dan leesbaarder.

VerpakkingType Probox : X of Y

Heb je na de wijziging in de EXTERN... de multicalc voor LBX nog eens getest?

In de LBXLangtekstCreator kan dan een tijdje na de livegang de overbodige code nog wat opgekuist worden maar dat is natuurlijk voor later

De methodnaam 'BevatRol is niet zo terecht. Het zou beter zijn BevatVerwijderdeRol of iets in die aard. Met BevatRol zou ik meteen denken aan de rol op de boom zelf, maar niet dat het eigenlijk gaat over de verwijderde items

Ik ben nogal gevoelig voor overdreven veel keer herhaling van dezelfde GetAt's en dergelijke.

Hierboven staan meer dan 10x KenmernNamen.GetAt(i ) en KenmernWaarden.GetAt(i )
Binnen de for-loop in een lokale variabele steken, zou dit al een stuk rustiger (en beetje performanter) maken.

Idem voor GekoppeldeIK.GetAt(##class(DOM.PM.Maatwerk.Calc.Common.enu.GekoppeldIKType).SpaceStep()) --> SstKenmerken (+ voordeel van code-completion)

Nog een kleine optimalisatie mogelijk :
De uitdrukking If $IsObject(ArrayVanTralala.GetAt(key)) kan je ook vervangen door iets leesbaardere uitdrukking If ArrayVanTralala.IsDefined(key)
N.B.: dit is niet 100% equivalent, want dit covert niet wanneer een array-element met "" (leeg) is ingevuld. Maar in dit geval, bij gekoppeldeMatten is dat normaal gezien niet het geval.
Ik geef in dit geval de voorkeur aan .IsDefined(key)

Hier kan je ook perfect de variabele SstKenmerken = IngegevenKenmerken.GekoppeldeIngegevenKenmerken.GetAt(##class(DOM.PM.Maatwerk.Calc.Common.enu.GekoppeldIKType).SpaceStep())
binnen deze If definieren.