Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Deze method hoort niet thuis in deze klasse, want heeft niets met BeslagBoringen te maken. We zullen samen bekijken hoe we dit best kunnen afsplitsen.

Deze method hoort niet thuis in deze klasse, want heeft niets met BeslagBoringen te maken.
We zullen samen bekijken hoe we dit best kunnen afsplitsen.

beetje muggeziften, maar maakt de method wel iets overzichtlijker : --> zet de 2 lijnen met Doorboring.SvgBox mee bovenaan, dan staan de (3) lijnen met Doorboring-object bij elkaar en de (3) lijnen...

beetje muggeziften, maar maakt de method wel iets overzichtlijker :
--> zet de 2 lijnen met Doorboring.SvgBox mee bovenaan, dan staan de (3) lijnen met Doorboring-object bij elkaar en de (3) lijnen met Circle-object staan dan ook bij elkaar.

Een andere volgorde, op basis van een andere logica, is wellicht ook oké.

Deze klasse mag idd weg.

Deze klasse mag idd weg.

Zelfde opmerking als bij ICT-1374 CrossFactory UT : Welgemikte getallen kiezen : bvb MaakBoring (60, 40, 5, Kleur.blauw) Een essentiele verantwoordelijkheid van de UT is om bij (x,y)-coordinaten e...

Zelfde opmerking als bij ICT-1374 CrossFactory UT :
Welgemikte getallen kiezen : bvb MaakBoring (60, 40, 5, Kleur.blauw)

Een essentiele verantwoordelijkheid van de UT is om bij (x,y)-coordinaten een foutieve (y,x)-switch te detecteren.

Zelfde opmerking als bij ICT-1374 CrossFactory UT : DummyCirkel en (misschien) DummySvgBox --> dit laatste is mogelijk een beetje tricky :-P

Zelfde opmerking als bij ICT-1374 CrossFactory UT :
DummyCirkel en (misschien) DummySvgBox --> dit laatste is mogelijk een beetje tricky :-P

Bij iedere UT moet je denken "Kan deze setup fouten verbergen?" of anders gezegd "Kan ik een programmeerfout aan het licht brengen, puur door andere input te kiezen?" Dus de (30, 30) zou je beter ...

Bij iedere UT moet je denken "Kan deze setup fouten verbergen?" of anders gezegd
"Kan ik een programmeerfout aan het licht brengen, puur door andere input te kiezen?"

Dus de (30, 30) zou je beter aanpassen naar verschillende waarden (zoals bvb. erboven 25,30)

Zoals het er nu staat, kan je enigszins verwarring scheppen tussen de echte MaakLijn() en de VerwachtMethodCall, aangezien deze (eerste) dezelfde parameters hebben. Bij voorkeur 2 verschillende (du...

Zoals het er nu staat, kan je enigszins verwarring scheppen tussen de echte MaakLijn() en de VerwachtMethodCall, aangezien deze (eerste) dezelfde parameters hebben.
Bij voorkeur 2 verschillende (dummy) Lijn-objecten gebruiken bij de DanReturn :

DummyLine1 = MaakLijn(0,0,0,0)
DummyLine2 = MaakLijn(0,0,0,0)

ge gaat toch eens een search voor Do ##class(Tools.Wlip).%New(238) moeten doen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif.

ge gaat toch eens een search voor Do ##class(Tools.Wlip).%New(238) moeten doen .

constructor bovenaan zetten. Technisch maakt dat helemaal geen verschil, maar het maakt het consequent met de rest. Een volgende lezer van de code kan dan ook direct zien welke andere objecten er i...

constructor bovenaan zetten. Technisch maakt dat helemaal geen verschil, maar het maakt het consequent met de rest. Een volgende lezer van de code kan dan ook direct zien welke andere objecten er in deze klasse gebruikt worden

ge kunt het niet laten hé http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (hieronder nog een paar)

ge kunt het niet laten hé (hieronder nog een paar)

[ICT-1365] SVG-029 Mock Testen uitbreiden
[ICT-1365] SVG-029 Mock Testen uitbreiden
[ICT-1326] SVG-023 Uitbreiden testen svg-project
[ICT-1326] SVG-023 Uitbreiden testen svg-project
deze test gaat niet echt uitgevoerd worden. Ofwel wel iets (nuttigs) laten doen, ofwel wegsmijten (idem voor andere boringtesten)

deze test gaat niet echt uitgevoerd worden. Ofwel wel iets (nuttigs) laten doen, ofwel wegsmijten (idem voor andere boringtesten)

zie opmerking in kaartje van CrossFactory

zie opmerking in kaartje van CrossFactory

well well well http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (en er staan er nog een paar)

well well well (en er staan er nog een paar)

AssertClassName bestaat ook

AssertClassName bestaat ook

nu die constructor leeg is, kan je je de vraag stellen of hij hier überhaupt nog moet blijven staan. Een constructie als deze kan een potentieel risico inhouden: aangezien je enkel een Quit $$$OK ...

nu die constructor leeg is, kan je je de vraag stellen of hij hier überhaupt nog moet blijven staan.

Een constructie als deze kan een potentieel risico inhouden: aangezien je enkel een Quit $$$OK doet, maskeer je logica die in de constructor van de Superklasses zit. In dit geval is dat niet echt relevant, omdat het toch klasses zijn zonder enige inhoud (bijna). Wou het gewoon even melden.

(logica van de superklasse oproepen: do ##super() )

[ICT-1367] SVG-030 Aanmaken BoringElementFactory
[ICT-1367] SVG-030 Aanmaken BoringElementFactory