Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1125] Kaderdeurbeelden omzetten van VML naar SVG

-Code Cleanup

    • -0
    • +5
    ./MaakCoordinatenLijstVanPunten/Test.cls.xml
    • -0
    • +2
    ./MaakCoordinatenLijstVanCoordinaten/Test.cls.xml
  1. … 1 more file in changeset.
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

[ICT-1340] SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-toevoegen exceptie voor verkeerde lijsten

    • -1
    • +29
    ./MaakCoordinatenLijstVanPunten/Test.cls.xml
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) 
[ICT-1350] SVG-027 Groene lijntjes laten verdwijnen in het project

-aanpassen variabelen

    • -5
    • +5
    ./MaakCoordinatenLijstVanCoordinaten/Test.cls.xml
    • -1
    • +1
    ./MaakCoordinatenLijstVanPunten/Test.cls.xml
  1. … 1 more file in changeset.
coordinatesservice en factory injecteren (en gebruiken in testen)

coordinatesservice en factory injecteren (en gebruiken in testen)

[ICT-1340]

SVG-024 CoordinatesService aanmaken en rotation opsplitsen bij speciale boringen

-Testen van aangemaakte methodes

    • -0
    • +23
    ./MaakCoordinatenLijstVanPunten/Test.cls.xml
    • -0
    • +28
    ./MaakCoordinatenLijstVanCoordinaten/Test.cls.xml
  1. … 1 more file in changeset.