Thanks, iv had a quick look at taskMacros and it looks pretty handy, will Investigate and have a play.
thing is I thought I had the whole (currentMillis- lastMillis >Period ) thing sorted and am confused about what iv done wrong, am I not even close?