BouwsteenIterator.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

DOM.PM.enu.LinkType (hieronder nog een paar keer) (en ook in ander testje)

my point exactly (zie comment). Afzonderen naar apart klaske dat getest kan worden

my point exactly (zie comment).
Afzonderen naar apart klaske dat getest kan worden

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

TestCases gemaakt voor beide public methods (en onderweg maken van product afgezonderd)

ClassMethod!

ClassMethod!

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Als ge de lijn toch verandert, kunt ge ineens de ' door $$$Not vervangen

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Boyscout principe: er bestaan geen testjes van deze klasse. Misschien is het niet veel moeite om een paar dingen toe te voegen?

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (ju...

Op den duur zou een mens zich beginnen afvragen of al deze StartVerwerking-code niet beter in een aparte klasse gestoken wordt. Ooit is dat begonnen als een klein methodje dat niet te veel doet (juist wat parameters zetten enzo) en dan was het geen probleem dat het niet in een niet-testbare classmethod steekt. Met al de extra logica die er ondertussen is bijgekomen is dat eigenlijk niet goed meer.

[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:
[DEF727] SPT: Voorraadbewegingen ontbreken op eindproduct, bij receptie productie afmelden:

- Method Key was niet gedefinieerd en gaf problemen indien BouwsteenIterator meegegeven werd aan IteratorOfIterators => BouwsteenIterator implementeert DynamicIterator => Daarvan wordt nu de Key doorgeven

In gedachten voor volgende keer

In gedachten voor volgende keer

Dit is sneller http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif

Dit is sneller

Mondeling besproken http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Mondeling besproken

Is contains en bijgevolg de LijstMetUniekeIdentnummers nodig? Je zou kunnen voor de if-else kijken of de LijstMetUniekeBouwstenen de bouwsteen al bevat (via FindPropertyWithValue)

Is contains en bijgevolg de LijstMetUniekeIdentnummers nodig? Je zou kunnen voor de if-else kijken of de LijstMetUniekeBouwstenen de bouwsteen al bevat (via FindPropertyWithValue)

Naamgeving: BouwsteenLijst of Bouwstenen (idem voor iterator)

Naamgeving: BouwsteenLijst of Bouwstenen (idem voor iterator)

Je was goe bezig met 'dimmen', dat mag ook voor lists http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Je was goe bezig met 'dimmen', dat mag ook voor lists

[DEF732] PM: VHIP481: BOMBOL: Space Tower: aantal van "middenniveau" niet in rekening gebracht:
[DEF732] PM: VHIP481: BOMBOL: Space Tower: aantal van "middenniveau" niet in rekening gebracht:
[DEF732] PM: VHIP481: BOMBOL: Space Tower: aantal van "middenniveau" niet in rekening gebracht:

- De lijst in de iterator passert nu langs de LijstOmvormer die alle dubbels uit de lijst haalt (volgens property Identnummer op ieder object) en de aantallen van de dubbels optelt en deze som meegeeft op de property Aantal en de Ids hernummert

- De TestCase van de BouwsteenIterator gebruikt nu een TotUniekeBouwstenenLijstOmvormerFake die dezelfde lijst teruggeeft als dat hij ontvangt => onafhankelijkheid van implementatie

  1. … 7 more files in changeset.
[DEF732] PM: VHIP481: BOMBOL: Space Tower: aantal van "middenniveau" niet in rekening gebracht:

- Juiste implementatie => aantal op kindbouwsteen wordt nu gebruikt als factor om aantal op bouwsteen te bepalen

[DEF732] PM: VHIP481: BOMBOL: Space Tower: aantal van "middenniveau" niet in rekening gebracht:

- Aantal van Kindbouwstenen wordt nu in rekening gebracht voor samenstellen lijst van Bouwstenen

  1. … 1 more file in changeset.
  • More
  • CR-811
  • finished reviewing
[UST3267] VHIP481: BOMBOL: enkel product zonder kinderen doorgeven
[UST3267] VHIP481: BOMBOL: enkel product zonder kinderen doorgeven
[UST3267] VHIP481: BOMBOL: enkel product zonder kinderen doorgeven
  1. … 8 more files in changeset.
  • More
  • CR-675
  • finished reviewing