Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Indentatie (extreem rechts ? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/wink.gif )

Indentatie (extreem rechts ? )

Deze lijn kan onder de if-structuur, moet niet telkens herhaald worden binnen de if.

Deze lijn kan onder de if-structuur, moet niet telkens herhaald worden binnen de if.

Ook doen of is het dan niet leesbaar genoeg?

Ook doen of is het dan niet leesbaar genoeg?

normaalgezien zou je heel deze if-elseif-else kunnen vervangen door Quit (Totaal && ##class(TECH.Math).IsEven(Totaal))

normaalgezien zou je heel deze if-elseif-else kunnen vervangen door Quit (Totaal && ##class(TECH.Math).IsEven(Totaal))

private

private

aaa, the good old wlip

aaa, the good old wlip

na opmerking van wim: testen toevoegen voor extra gedrag. Wat als er 0,1,2, veel coordinaten worden meegegeven?

na opmerking van wim: testen toevoegen voor extra gedrag.
Wat als er 0,1,2, veel coordinaten worden meegegeven?

Deze method is een leuke oefening (vanuit educatief standpunt), maar in de praktijk vind ik dit een nogal tricky interface. Als we deze willen blijven gebruiken/ondersteunen, dan is heb ik een rev...

Deze method is een leuke oefening (vanuit educatief standpunt), maar in de praktijk vind ik dit een nogal tricky interface.

Als we deze willen blijven gebruiken/ondersteunen, dan is heb ik een review-opmerking, namelijk : deze moet een InvalidInput-exception smijten indien het aantal "Punten" oneven is.
Let op : het tellen van Punten... is geen one-liner !

Ofwel overwegen om aan te passen naar $LB(X,Y) parameters, i.e.

MaakCoordinatenLijstVanXYPunten(lbXYPunten... As %List) 
coordinatesservice en factory injecteren (en gebruiken in testen)

coordinatesservice en factory injecteren (en gebruiken in testen)

Ook al is performantie hier momenteel geen echt issue, toch zou ik opteren voor if - else if - else if - ...

Ook al is performantie hier momenteel geen echt issue, toch zou ik opteren voor if - else if - else if - ...

Later te bekijken : of we een andere manier, meer generieke methode, van "svg-objecten toevoegen" mogelijk is.

Later te bekijken : of we een andere manier, meer generieke methode, van "svg-objecten toevoegen" mogelijk is.

Accolades en indentation : graag volgens de VH-conventies aub.

Accolades en indentation : graag volgens de VH-conventies aub.

FYI : %Decimal is perfect ! %Numeric is ook aanvaardbaar (is een meer algemeen datatype, denk. Maar t maakt hoe dan ook ni echt uit aangezien Caché niet strong-typed is)

FYI :
%Decimal is perfect !
%Numeric is ook aanvaardbaar (is een meer algemeen datatype, denk. Maar t maakt hoe dan ook ni echt uit aangezien Caché niet strong-typed is)

mag deze dan niet volledig weg?

mag deze dan niet volledig weg?

Ook properties asserten, niet enkel de SVG output (idem voor andere testjes hieronder)

Ook properties asserten, niet enkel de SVG output (idem voor andere testjes hieronder)