Activity CacheAdminA_trunk2010/WSimpl/AX/CalculatedProduct/VariantDescriptionBepaler

3:26 pm

Tom Vermeulen

More
Re: CR-1982
commented on review

zie andere review van dit kaartje

8:17 am

ik snap wat je wil zeggen, maar zou je dit niet vervangen door "identnummer van product"? Zonder naar de code te kijken is het precies of ge een random nummer gaat teruggeven http://subversion02.va...

ik snap wat je wil zeggen, maar zou je dit niet vervangen door "identnummer van product"? Zonder naar de code te kijken is het precies of ge een random nummer gaat teruggeven

8:16 am

Tom Vermeulen

More
Re: CR-1986
commented on review

probeer geen early quits te gebruiken, dat maakt het voor latere debugging/uitbreiding veel lastiger. In principe zijn er maar twee plaatsen waar je quits mag zetten: helemaal bovenaan een method o...

probeer geen early quits te gebruiken, dat maakt het voor latere debugging/uitbreiding veel lastiger. In principe zijn er maar twee plaatsen waar je quits mag zetten: helemaal bovenaan een method om bij trivialiteiten al te quiten en natuurlijk onderaan als er iets gereturned moet worden.

Dus beter een variable "resultaat" of zoiets maken en die dan assignen en returnen op het einde

10:02 am

Tom Vermeulen

More
Re: CR-1986
commented on review

ipv If $$$Not(xxxx = "") kan je if $$$HasLength(xxx) gebruiken leest wat gemakkelijker In Caché kan je ook met een postconditional werken: ipv If blablabla{ Do zwamzwam } kan je volgende lijn ge...

ipv If $$$Not(xxxx = "") kan je if $$$HasLength(xxx) gebruiken
leest wat gemakkelijker

In Caché kan je ook met een postconditional werken:
ipv
If blablabla

Unknown macro: { Do zwamzwam }

kan je volgende lijn gebruiken

"Do:blablabla zwamzwam"

8:56 am

Tom Vermeulen

More
Re: CR-1982
commented on review

Aangezien er met VerwachteUitkomst niks gebeurt tot op het einde van de test, is het niet per sé nodig om die hier al te dimmen. Je zou de assert evengoed kunnen reduceren tot Do $$$AssertEquals("O...

Aangezien er met VerwachteUitkomst niks gebeurt tot op het einde van de test, is het niet per sé nodig om die hier al te dimmen. Je zou de assert evengoed kunnen reduceren tot
Do $$$AssertEquals("Omschrijving in het Nederlands"$$$CRLF"Kleur:VertalingKleur",DescriptionBepaler.GeefDescription(ProductMock,RandomTaal))
Hoe dan ook is dat iets waarover je kan blijven discussiëren: langere lijnen en minder variabelen, of kortere lijnen en meer one-time-use variabelen...
Indien je toch gaat voor meer variabelen: de gewoonte is om de #dim's niet allemaal bovenaan te smijten, maar vlak voordat je ze nodig hebt

11:50 am

Peter Vercoutter

More
CR-1986
started review

[ICT - 883] UAT 1422: verkeerde tekst in engels document

3:31 pm

Tom Vermeulen

More
Re: CR-1982
commented on review

Maak hier maar ineens een Setterke van, want zoals je ziet kan je in dit geval (lange namen) niet zien of er dingen aan de constructor worden meegegeven. Idem voor andere #dim/Set combinaties in de...

Maak hier maar ineens een Setterke van, want zoals je ziet kan je in dit geval (lange namen) niet zien of er dingen aan de constructor worden meegegeven.
Idem voor andere #dim/Set combinaties in deze en de andere test

3:21 pm

Tom Vermeulen

More
Re: CR-1982
commented on review

om goed te zijn zou je ook een test moeten voorzien die laat zien dat er geen Nederlands gebruikt wordt indien de omschrijving in andere taal niet ontbreekt

9:45 am