You never define the variable $s!

And please post code in between [CODE] tags as described in ABC's of KiXtart board etiquette and message to new forum users.
_________________________
There are two types of vessels, submarines and targets.