Can anyone tell me what's wrong with this simple code?

Thanks, I'm now working on the AD input. That's my next problem, to read and manipulate the AD data. Once I have that, then my next project will be to put this into tables and access them... I'm not too concerned with the particular form of the delay, as long as I can get about a 1s delay, I'm happy. Then I can move on the next hurdle. In this case, the AD.