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?

Correctie : in dit uitzonderlijke geval lukt het niet om af te leiden van TECH.RegisteredObject --> in totaal zullen er 3 klassen zijn die dus afleiden van (%RegisteredObject,%XML.Adaptor)

Correctie : in dit uitzonderlijke geval lukt het niet om af te leiden van TECH.RegisteredObject
--> in totaal zullen er 3 klassen zijn die dus afleiden van (%RegisteredObject,%XML.Adaptor)

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)

Je hebt hier en daar ook nog wat dingen gecommit die te maken hebben met SvgGroups enzo. Is dat iets dat ook in dit kaartje (andere overerving) moet zitten?

Je hebt hier en daar ook nog wat dingen gecommit die te maken hebben met SvgGroups enzo. Is dat iets dat ook in dit kaartje (andere overerving) moet zitten?

afleiden van TECH.RegisteredObject, niet van %RegisteredObject. Ook op een paar andere plaatsen.

afleiden van TECH.RegisteredObject, niet van %RegisteredObject. Ook op een paar andere plaatsen.

mag weg

mag weg

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 - ...