Activity CacheAdminA_trunk/UI/PR/GA/TandemBox.cls.xml

1:19 pm

Tommy Hebb

More
Re: CR-2073
commented on review

Properder: If ((MatKleur = "Z") & (LadeDiepte > 450)) Dat geeft at runtime exact hetzelfde resultaat, maar is 1 If ipv een geneste If, dus leesbaarder. Idem voor die hieronder...

Properder: If ((MatKleur = "Z") & (LadeDiepte > 450))
Dat geeft at runtime exact hetzelfde resultaat, maar is 1 If ipv een geneste If, dus leesbaarder.

Idem voor die hieronder...

4:00 pm

Hier een beetje hetzelfde... nieuwe code, ook al is het gebaseerd op een copy/paste, graag refactoren volgens huidige conventies. Je hoeft het nu niet meer aan te passen. Onthoud het voor de volgen...

Hier een beetje hetzelfde... nieuwe code, ook al is het gebaseerd op een copy/paste, graag refactoren volgens huidige conventies. Je hoeft het nu niet meer aan te passen. Onthoud het voor de volgende keer.

"$lb" = $ListBuild
"$lg" = $ListGet
"n" = New
"s" = Set
"q" = Quit
Wat spaties hier en daar voor en na =-teken en na komma en zo.

3:55 pm

Tommy Hebb

More
Re: CR-2073
commented on review

Opletten voor casing graag http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif Variabele-namen beginnen met hoofdletter (Msg, Bool). Misschien de variabele...

Opletten voor casing graag

Variabele-namen beginnen met hoofdletter (Msg, Bool).
Misschien de variabele Bool dan ineens hernoemen naar het meer beschrijvende: IsValid.
Set met hoofdletter.

P.s.: OMG wat een klasse!!! lol Je hoeft niet de hele klasse aan te passen in dit geval (veel te veel werk), maar als je code toevoegt, zet die dan wel ineens volgens onze huidige conventies.

3:44 pm

Tommy Hebb

More
Re: CR-2073
commented on review

Als alle lijnen in comment staan, misschien toch een extra comment-lijntje toevoegen met de reden waarom. Ik vermoed in afwachting van dat de code actief moet gezet worden? Maar de call zelf staat ...

Als alle lijnen in comment staan, misschien toch een extra comment-lijntje toevoegen met de reden waarom. Ik vermoed in afwachting van dat de code actief moet gezet worden? Maar de call zelf staat toch in comment in de klasse APPS.EC.impl.ConfiguratorService.KenmerkMogelijkhedenPostProcessor?

Anderzijds... moet je hier niet enkel diegene voorzien die in klasse APPS.EC.impl.ConfiguratorService.KenmerkMogelijkhedenPostProcessor nog niet voorzien waren?

Louter ter info: Ook nog even meegeven dat iedere enum, die dus per definitie af moet leiden van TECH.Enumeration, automatisch daarom ook beschikt over een ValueListIterator. Als je dus alle enumwaarden van een enum moet overlopen, dan is gebruik maken van die iterator wellicht overzichtelijker dan voor iedere waarde een codelijn te voorzien.

1:38 pm