DefaultVoetTekst.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots

- Reviewopmerkingen behandelen

  1. … 2 more files in changeset.
Zoals elders gezegd... geen waarden via parameters doorgeven in een constructor, dat moet je dan via een aparte public method doen! De ZetTekstLinks en ZetTekstMidden mogen wel blijven staan, omdat...

Zoals elders gezegd... geen waarden via parameters doorgeven in een constructor, dat moet je dan via een aparte public method doen!
De ZetTekstLinks en ZetTekstMidden mogen wel blijven staan, omdat het louter defaultering is.

Ik zie dat dit een copy/paste is van hierboven, maar dan toch even de casing aanpassen aub http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif For Set $Orde...

Ik zie dat dit een copy/paste is van hierboven, maar dan toch even de casing aanpassen aub
For
Set
$Order
Quit

En ook graag een leeg lijntje tussen de For-loop en de uiteindelijk Quit
Misschien gelijk ook de casing elders in deze klasse aanpassen, is niet zo veel.

Oeps! Dit is een beetje tegen de schenen schoppen van onze code-conventions http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif De constructor gebruiken we ...

Oeps! Dit is een beetje tegen de schenen schoppen van onze code-conventions
De constructor gebruiken we ENKEL voor het intantiëren van een object. De énige parameters die daar dus aan meegegeven mogen worden zijn degene die voor die intantiëring essentieel zijn, zoals dependencies.
Wat we zeker NIET doen in constructors, of toch zeker zoveel als mogelijk proberen te vermijden, is het doorgeven van waarden (als parameters) om properties te zetten. Dat maakt namelijk een knoeiboel van eventuele testen.
Wat je dus eigenlijk moet doen is die 2 parameters er weer uithalen en in de klasse waarvan je hier een object maakt een public method maken om het zetten van die properties te regelen.
Dat maakt dan wel dat je hier 2 coderegels gaat hebben: Eentje om een object te instantiëren en direct daarna een tweede om op die instantie die public method te callen met deze 2 parameters, bv:
Do VoetTekst.ZetPaginaNrEnTotaalAantalBladzijden(HuidigPaginaNr, TotaalAantalBladzijden)

[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots
[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots
[ICT-1494] HLX: TAX - aangeven hoeveelste tray van batch aan robots

- In de koptekst wordt nu aangegeven welke tray huidige tray is en hoeveel er in totaal zijn

- in de voettekst klopt de paginanummer nu wel en wordt ook aangegeven hoeveel paginas er zijn in totaal

  1. … 24 more files in changeset.