why do you use the function before it is declared?
And if that is a complete copy / paste then you need a closing bracket "}" for the function..

But as Wildhoney says, that is the way i would prefer it to be done, but be aware that is you use some sort of external input in the script, like a $_GET or $_POST, it could evolve into some sort of security breach, but as it is now it is okay :)

And by the way, you don't have to declare variables twice? I refer to these: