Activity CacheAdminA_trunk2010/vhConsistency/Checks/TerminalNavigatieTaken.cls.xml

2:37 pm

Jelle Baele

More
63579
changed 1 file
root: CR-2247

[ICT-1899] [rvTHB] Consistencycheck TerminalNavigatieTaken failed on [CACHE01]

- Review opmerkingen verwerkt

- -3
- +4
./TerminalNavigatieTaken.cls.xml
- History
- Source
- Diff
./TerminalNavigatieTaken.cls.xml

11:35 am

Je zit hier met een If/Else met een $$$Not in de If. Hoewel de logica correct is, leest dit net iets minder makkelijk dan zonder de $$$Not, dus als je de logica in de If en de Else omdraait, dan ku...

Je zit hier met een If/Else met een $$$Not in de If. Hoewel de logica correct is, leest dit net iets minder makkelijk dan zonder de $$$Not, dus als je de logica in de If en de Else omdraait, dan kun je de $$$Not weglaten.
Flow-gewijs is het ook corrrecter in dit geval... Als de computernaam huppeldepup is, (If...) dan gaat het over data waar we niets aan hebben, dus verwijderen we die. (Else...) In het andere geval moet het wel gemeld worden.

11:29 am

Tommy Hebb

More
Re: CR-2247
commented on review

Aangezien het object al eens eerder is geopend en toen in een lokale variabele is gestoken (TerminalNavigatieTaak), kun je die beter hier gebruiken. Het heeft niet veel nut om de data 2 keer na elk...

Aangezien het object al eens eerder is geopend en toen in een lokale variabele is gestoken (TerminalNavigatieTaak), kun je die beter hier gebruiken. Het heeft niet veel nut om de data 2 keer na elkaar op te halen, het zit dan toch al in memory. Dus:
Do ..VoegMeldingToe("[ Openstaande taak ]"_$C(13,10)_TerminalNavigatieTaak.ToString())

10:15 am