Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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

Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets

Ge hebt een Assign en Act gedeelte, maar genen Assert. Deze test doet dus eigenlijk niets

Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan

Deze locatie gaat ge toch ergens in een ConfigItem moeten steken en niet zo hardcoded laten staan

elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is (kleine kaartjes, rap committen, enz enz http://subversion02.vanhoecke.b...

elementtoevoeger en factory hier niet newen, maar gezien dat de file al outdated is en deze opmerking niet meer relevant is

(kleine kaartjes, rap committen, enz enz )

Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen. Idem bij constructors andere boringen

Geen code uitvoeren in een constructor (behalve het newen van andere objecten). Code afzonderen naar andere method en die dan apart oproepen.

Idem bij constructors andere boringen

klasse newen in constructor

klasse newen in constructor

indents ongedaan maken

indents ongedaan maken

[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger
[ICT-1349] SVG-026 Toevoegen van SvgBox.ElementToevoeger
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)

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

-Voorlopig uitschakelen van testen v/ boringen

  1. … 3 more files in changeset.
Sam, deze implementatie gaan we in "pair reviewen" (wss rechtstreeks vanop de code). We kunnen hier immers vele kanten mee uit http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/ico...

Sam, deze implementatie gaan we in "pair reviewen" (wss rechtstreeks vanop de code).
We kunnen hier immers vele kanten mee uit

Al opgelost? Indien niet, dan mag je een "UnimplementedException" throwen (voorbeeldjes via ctrl-shift-find) Als je echt wil om deze te implementeren, dan kan deze boring best voorgesteld worden do...

Al opgelost?
Indien niet, dan mag je een "UnimplementedException" throwen (voorbeeldjes via ctrl-shift-find)
Als je echt wil om deze te implementeren, dan kan deze boring best voorgesteld worden door de combinatie van een "interne zijboring" + een "externe zijboring".

Ja, ik geef ook de voorkeur aan nieuwe conventies, namelijk : While (Iterator.HasNext() ) i.p.v. While ($IsObject(Boring)) Merk op : testen met $$$HasLength() gebruiken we niet (meer) bij objecten...

Ja, ik geef ook de voorkeur aan nieuwe conventies,
namelijk : While (Iterator.HasNext() ) i.p.v. While ($IsObject(Boring))

Merk op : testen met $$$HasLength() gebruiken we niet (meer) bij objecten, mag alleen nog bij DataTypes, als %String. Voor objecten is het correcter om te testen met $IsObject()

Opm. 2: (naming convention) niet BoringenIterator maar BoringIterator (enkelvoud)