Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Een vraag bij deze deploy-klasse: Zou dit ook werken met gewoon "Do ##class(APPS.Halux.PPS.DocumentDefinitie.GekoppeldeIK.MattenLijst).%New().MaakDocument()"? Dit roept SaveDocument op, wat volgens...

Een vraag bij deze deploy-klasse:
Zou dit ook werken met gewoon "Do ##class(APPS.Halux.PPS.DocumentDefinitie.GekoppeldeIK.MattenLijst).%New().MaakDocument()"?
Dit roept SaveDocument op, wat volgens mij hetzelfde doet. Maar misschien mis ik hier iets waardoor het manueel gedaan moet worden.

Anyway, het is zeker niet fout

[ICT-3622] Mattenlijst - Kolom 'Tot#' als eerste kolom zetten
[ICT-3622] Mattenlijst - Kolom 'Tot#' als eerste kolom zetten
[ICT-3594] Mattenlijst - totalen weergeven
[ICT-3594] Mattenlijst - totalen weergeven
Misschien kunnen deze filters ook in een meer algemene klasse in een (bestaande) common package?

Misschien kunnen deze filters ook in een meer algemene klasse in een (bestaande) common package?

[ICT-2426] Nieuwe aansturingsactie MVX: Mattenlijsten
[ICT-2426] Nieuwe aansturingsactie MVX: Mattenlijsten
lange statements proberen vermijden.je kan vergelijkingen vervangen door methods. ..IsProductiegroepKast() bv

lange statements proberen vermijden.je kan vergelijkingen vervangen door methods. ..IsProductiegroepKast() bv

or gebruiken. binnen de "if" is het zelfde code.

or gebruiken. binnen de "if" is het zelfde code.

indent gelijk houden. we formatteren meestal else op dezelfde lijn van de accolades. } else {

indent gelijk houden.

we formatteren meestal else op dezelfde lijn van de accolades.
} else {

..BatchService bestaat al, geen nieuwe instantie nodig.

..BatchService bestaat al, geen nieuwe instantie nodig.

ook hier methods maken van vergelijken, zo wordt de lijn korter en leesbaarder. (boolean steeds "Is" -> bv ..IsProductieGroepKast() etc

ook hier methods maken van vergelijken, zo wordt de lijn korter en leesbaarder. (boolean steeds "Is" -> bv ..IsProductieGroepKast() etc

zelfde "insert" code. ik zou geen elseif gebruiken, maar OR statement.

zelfde "insert" code. ik zou geen elseif gebruiken, maar OR statement.

kleintje : indent aanhouden.

kleintje : indent aanhouden.

Proberen onnodige wijzigingen te committen.

Proberen onnodige wijzigingen te committen.

ik zou hier if $$$Not(..IsStuk(Batch) && $$$Not(..IsFlatpack(Batch)) zetten. maw vergelijkingen naar een nieuwe method brengen waarvan de naam duidelijk aangeeft wat hij doet. Dan is de if statemen...

ik zou hier
if $$$Not(..IsStuk(Batch) && $$$Not(..IsFlatpack(Batch)) zetten.
maw vergelijkingen naar een nieuwe method brengen waarvan de naam duidelijk aangeeft wat hij doet. Dan is de if statement makkelijk leesbaar.

Quit enkel op eind van method gebruiken. Opvangen door een variabele in het begin van de method. en variabele te zetten in de "if" en mee te geven bij de quit.

Quit enkel op eind van method gebruiken. Opvangen door een variabele in het begin van de method. en variabele te zetten in de "if" en mee te geven bij de quit.

BatchService zou ik als property maken en toevoegen aan New van deze klasse, dan kan je hier enkel ..BatchService.GeefBatch() als code zetten.

BatchService zou ik als property maken en toevoegen aan New van deze klasse,
dan kan je hier enkel ..BatchService.GeefBatch() als code zetten.

[ICT-1115] HLX: mattenlijst voor TBX & LBX (toeleveringsnummer)
[ICT-1115] HLX: mattenlijst voor TBX & LBX (toeleveringsnummer)
els if van maken controle op count() is gemeenschappelijk en kan in 1 instructie. if (count()=0) ( insert mat) else if (overeenkosmtigemat gevonden) (insert mat)

els if van maken controle op count() is gemeenschappelijk en kan in 1 instructie.

if (count()=0) ( insert mat)
else if (overeenkosmtigemat gevonden) (insert mat)

Algemene richtlijn voor ons, niet blokkerend, maar naar leesbaarheid spatie gebruik bij = en + tekens.

Algemene richtlijn voor ons, niet blokkerend, maar naar leesbaarheid spatie gebruik bij = en + tekens.