CacheAdminA_trunk2010/APPS/Halux/PPS/ALUFrees/Svg

BasisElement

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 1
Last Commit: 22 Jan 20
Commits this week: 0
Total Lines of Code (LoC): 209
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Monday 23 Nov 2020
8:56 am

Sam Van Hoey

8:55 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:52 am

Sam Van Hoey

8:51 am

Sam Van Hoey

8:47 am

Sam Van Hoey

Thursday 02 Apr 2020
5:56 pm

Wim Vermeulen

4:20 pm

Wim Vermeulen

4:19 pm

Wim Vermeulen

Na een Catch() moet er altijd een "reporting" of een nieuwe throw gedaan worden. In dit geval zegt mijn gevoel dat hier geen try-catch moet staan, want het is aan de gebruiker/oproepende code om ze...

Na een Catch() moet er altijd een "reporting" of een nieuwe throw gedaan worden.
In dit geval zegt mijn gevoel dat hier geen try-catch moet staan, want het is aan de gebruiker/oproepende code om zelf te bepalen welke error-handling er nodig is.

Anderszijds lijkt dit meer op een stukje test-code, dan kan je dat best ook zo aangeven (in methodnaam of ev in de comment) of moet je deze testcode verplaatsen naar een zTryout-klasse.
Zo niet, dan krijg je de reviewer(s) op uw dak :-D

10:53 am

Wim Vermeulen

10:51 am

Wim Vermeulen

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)

10:49 am

Wim Vermeulen

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)

Wednesday 25 Mar 2020
12:03 pm

Tom Vermeulen

12:03 pm

Tom Vermeulen

Dit is opeetcode. Als er een exception geraised wordt, gaat niemand het weten. Is dit het gewenste gedrag? Indien niet: er iets mee doen (en testje(s) voor schrijven)

Dit is opeetcode. Als er een exception geraised wordt, gaat niemand het weten. Is dit het gewenste gedrag? Indien niet: er iets mee doen (en testje(s) voor schrijven)

12:01 pm

Tom Vermeulen

indenten tussen de accolades

indenten tussen de accolades

12:00 pm

Tom Vermeulen

11:41 am

Tom Vermeulen

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 .

11:39 am

Tom Vermeulen

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

11:38 am

Tom Vermeulen

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)

Monday 23 Mar 2020
4:35 pm

Sam Van Hoey

[ICT-1348] SVG-025 ' aanpassen naar $$$NOT() + naamgeving van variabelen uitbreiden
[ICT-1348] SVG-025 ' aanpassen naar $$$NOT() + naamgeving van variabelen uitbreiden
4:31 pm

Sam Van Hoey

[ICT-1365] SVG-029 Mock Testen uitbreiden
[ICT-1365] SVG-029 Mock Testen uitbreiden
8:34 am

Tom Vermeulen

8:32 am

Tom Vermeulen

AssertClassName bestaat ook

AssertClassName bestaat ook

8:28 am

Tom Vermeulen

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() )

Thursday 19 Mar 2020
2:34 pm

Tom Vermeulen

11:42 am

Sam Van Hoey

[ICT-1374] SVG-031 Aanmaken CrossFactory
[ICT-1374] SVG-031 Aanmaken CrossFactory
Monday 16 Mar 2020
4:16 pm

Wim Vermeulen

3:58 pm

Wim Vermeulen

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)

Thursday 12 Mar 2020
6:26 pm

Wim Vermeulen

6:21 pm

Wim Vermeulen

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