>     #declare Weeks_Notice = int(Years_Employed)

I'm not sure if int is a valid function. Replace with ceil or floor (or 
round?) as appropriate; would make it more explicit.

>   #debug concat("Weeks notice required: ",str(Weeks_Notice))

Better design if the function returns the value it found, instead of 
printing it.

Also, you could have used #switch and #range; but I agree with 
Invisible: the way it's written now is clear even for non-coders...

Post a reply to this message