OnderdeelPickingComparator.cls.xml

Zet jij er dan de besproken check bij die controleert op de lengte aub? Misschien een goede om wat kennis over te verspreiden (demo / mailtje)?

Lijkt mij zeker ok om in die zin te verbeteren hoor. Was overblijfsel van omschakelen naar %-variabele.

Alvast voor het archief: % heb ik ingevoerd omdat er op verschillende plaatsen de berekenaar genewed wordt, en ik daar in eerste instantie niet op wou focussen. Geen idee hoeveel werk het is om die verschillende plaatsen te reduceren.
Verschillende gebruikers verschillende versie van de boom: aan welk scenario denk je juist?

% variabelen ?? Is er dan geen gevaar dat er verschillende gebruikers een verschillende versie van de boom kunnen vasthebben ?

PrijsBerekeningen vs ProductieAansturing

Waarom % variabelen ???? Private properties toch beter

Door de canonische waarde te gebruiken bestaat er de kans dat er een subscriptfout kan ontstaan wegens de Key te lang

Wel raar dat de default hier true is , waarom niet gaan ophalen zoals ingesteld in de parameter

Gelieve commentaar zodanig te schrijven dat deze binnen ettelijke weken nog duidelijk is

Afzonderen naar een aparte klasse lijkt mij hier dan wel een goed idee

Als je dan toch commentaar schrijft gelieve er dan er voor te zorgen dat als het toch moeilijk te begrijpen code is .

Dat je dan aanduidt waar er gefilterd wordt en waar die andere info wordt aangevuld. Kwestie dat het compleet duidelijk is.

Class rename lijkt mij iets meer aan de orde , kwestie om het nog duidelijker te maken