no hace la funcion

[fermin3d]

Hola amigos estoy practicando con javascritp y no se porque no me funciona este script espero me puedan ayudar gracias.

Código Javascript:

valor = document.getElementById("nombre").value;
function validacion(){
if ( valor == null || valor.length == 0 || /^\s+$/.test(valor) ){
    
    
    alert('error');
    return false;
    }
}
 

y el html

Código HTML:

<form action="enviar.php" method="" name="" onsubmit="return validacion()">
<input type="text" name="nombre"  />
<input type="submit" value="enviar" />
</form>
 

Muchas gracias

[maycolalvarez]

1 - estás tratando de obtener el valor del input en el scope global, además de que en ese instante ni el DOM se ha cargado, por lo tanto no existe, y si detectaras eso solo tendrías el valor inicial, lo ideal es que lo obtengas dentro de la función que actúa como handler del evento

2 - use var para declarar variables explícitamente, la declaración implícita la soportan pocos navegadores

3 - usa getElementById cuando el elemento NO tiene el atributo ID

[fermin3d]

es verda muchas gracias y tenia otra duda tengo este script

Código Javascript:

valor3 = document.getElementById("email").value;
if ( !(/\w+([-+.']\w+)*@\w+([-.]\w+)*\.\w+([-.]\w+)/.test(valor3)) ){
                        
                        alert('Por favor, Escriba un Email correcto');
                        return false;
                        
                    }
 

y siempre me da error no se si las expresiones regulares estab bien pero creo que si muchas gracias

[KsrZ]

Hola @fermin3d, el problema es que \w tambien omite guion bajo _ y en ese caso, no le das un comienzo y un final a tu expresión, no estaria bien del todo utilizar guiones bajos en un comienzo, o final, mira aquí:
http://www.forosdelweb.com/f13/expreg-email-1012756/

[fermin3d]

aahh es cierto Muchas gracias KsrZ