Deploy20200124UpdatePasDocMattenlijst.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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)
[ICT-1115] HLX: mattenlijst voor TBX & LBX (toeleveringsnummer)

- Deployklasse aangemaakt zodat de kolomnaam 'Toe:Dos' nu 'Toelev' wordt.

- Voor TBX worden de matten nu enkel gegroepeerd als de verpakkingstype niet V1 is.

    • -0
    • +35
    ./Deploy20200124UpdatePasDocMattenlijst.cls.xml
  1. … 2 more files in changeset.