la validación no funciona

[venkman]

No tiene nada que ver que las definas como globales o locales o les pongas var. El problema que te indicaba es que cuando tienes que leerlas, cuando tienes que hacer la carga del valor que tienen los campos es cuando llames a validar.

Prueba a poner en tu función validar un alert(email) nada más empezar. Hazlo poniendo las líneas dentro y fuera, y verás la diferencia.


Por lo demás, la expresión regular que estás poniendo, está mal. Tienes que ponerles \ a las w:
/\w+[@][\w-]+([.]([\w-]+)){1,3}$/

O eso, o utiliza las que te ha puesto antes codigojavascript, que son más completas.

(Además, no, cuando les pones var dentro de una función, las variables son locales a la función. Pero como digo, no se trata de cómo sean las variables sino de cuándo estás leyendo los valores de email = document.getElementById("email").value;)