Productie.Planning.ActieveBatches.Kaderdeur.Aansturing

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
ICT-1479: PA-UB-101: Afdrukken etiketten: fout in KCA logica
ICT-1479: PA-UB-101: Afdrukken etiketten: fout in KCA logica
Reverse Merged revision(s) 61124 from trunk2010:
Reverse Merged revision(s) 61124 from trunk2010:
[ICT-1479] PA-UB-101: Afdrukken etiketten: fout in KCA logica

- Generatie DnaCode verplaatst naar afdrukloop

- Deze keer met testbestanden (toch die die ik al gemerkt heb)

  1. … 3 more files in changeset.
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-1108] PA-UB-065: Bartenderfile voor kaderdeuren ook de dnacode laten bevatten

- Testfile met leeg veldje voor DnaCode

[ICT-1115] HLX: mattenlijst voor TBX & LBX (toeleveringsnummer)

- Enkele integratietesten faalden, blijkbaar lopen er ook nog andere soorten lijsten door de method waar de mattenlijsten (voor LBX en TAX) doorlopen. Hierdoor werd een filter ingesteld waardoor enkel de mattenlijsten aangepakt worden

  1. … 9 more files in changeset.
[ICT-1108] PA-UB-065: Bartenderfile voor kaderdeuren ook de dnacode laten bevatten

- nog wat testjbestanden