dto

= Code van Wim. Wellicht zijn er teveel verschillende mogelijkheden om een al te specifieke te gebruiken hé.

Goed gezien :-s Bij deze aangepast!

Als er ooit vhossen bij mvx komen, gaat ge het probleem hebben dat er geen vhossetiketten worden afgedrukt. Dat is een uitbreiding die dan nog moet toegevoegd worden op de KlantEtiket.PrinterFactory

is het niet beter om een specifiekere exception te gebruiken? CommunicationErrorException ofzo

groter dan wat? (550, dat zie ik wel door naar de code te kijken, maar als je een comment toevoegt doe je dat om niet naar de code te moeten kijken ) (meerdere keren)

'> ?? gebruik beter <= (meerdere keren)

detail, maar in principe gebruiken we die domeincontext niet meer (zie WV en MVI voor details) en kan die gewoon genewed worden. (staat op meerdere plaatsen, ga dat niet overal schrijven )

$$$TAB

geen $translate in deze klasse maar ook de method van de pxLabel gebruiken
het originele probleem is ook nog steeds aanwezig: je gaat eerst tabs toevoegen aan TeSchrijven om ze dan in de laatste lijn weer te verwijderen

ik zou er nog "Vervang..." ipv "Verwijder..." van maken

Als het over een oneliner gaat, mag je ook "Quit $Translate(...." gebruiken

voor de volledigheid zou je ook een test moeten voorzien met een lege string eentje met enkel een te vervangen karakter, eentje met het karakter in het begin en eentje in het midden. Vervolgens ook nog eens een test met meerdere te vervangen karakters (dezelfde/verschillende). Op die manier zijn alle mogelijkheden afgedekt. Aja ook een testje waarbij er geen karakter te vervangen is.

Eigenlijk is het niet VerwijderNietPrintbareKarakters, maar VervangNietPrintbareKaraktersDoorSpatie

we gebruiken nooit All Caps
de geneste $Translates kan je wijzigen in 1 translate

de wijzigingen voor de andere files waar etiketten worden gegenereerd zitten er nog niet tussen

Voor de leesbaarheid misschien ook ineens de *9 door $$$Tab vervangen
Het kan wel zijn dat die macro niet beschikbaar is in deze klasse. (misschien even uitzoeken hoe je die wel kan gebruiken?)
In dat geval is het misschien toch nog iets leesbaarder om de *9 te vervangen door $Char(9), wat de link met een TAB-karakter iets duidelijk maakt.
Up to you

oudste vorm: *9
minder oude vorm: $Char(9) of $C(9)
huidige vorm: $$$TAB

hetzelfde geldt natuurlijk ook voor CR en LF (13 en 10)

deze moet eigenlijk niet mee in de review, maar dat maakt nu niet zoveel uit

Hier gaat ge een licht probleem krijgen: de TeSchrijven variabele bevat tabs ($Char(9)) als veldseparator. Door ze er weer uit te halen gaat alles in 1 veld komen.
Zelfde opmerking voor de CRLF die aan TeSchrijven wordt toegevoegd, ge gaat die ook kwijt geraken en alles op 1 lijn zetten
Vervang en passant die $Char(9) door $$$Tab

Quit ipv QUIT