[SOLUCIONADO] Validacion sencilla formulario de login con javascript

[jcastro3]

Buenas, tengo un problema y no se donde puede estar el error.

Tengo el siguiente formulario con este codigo:

Código HTML:

<form name="form" method="post">
Usuario: <input type="text" name="usuario" id="usuario"/><br/>
Contraseña: <input type="password" name="password" id="password"/><br/>
<input type="button" id="send" value="Acceder" onclick="valida();"> 

Y por otro lado tengo el siguiente codigo en javascript:

Código Javascript:

function envia(pag){
    document.form.action = '../php/'+pag+'.php';
        document.form.submit();
}
 
function valida(){
    var usuario = document.getElementById('usuario');
    var pass = document.getElementById("password");
            
    if((usuario == "")||(pass == "")){
        window.alert("Los campos usuario y contraseña no pueden estar vacios");
    }else{
        envia('login');
    }
}
 

El problema esta en que no salta el alert cuando el campo usuario o contraseña esta vacio.

El evento onclick si se produce porque si borro la estructura if-else y solo dejo el alert si que salta.

No se que es lo que estoy haciendo mal, necesito ayuda.

[Enano_la22]

Holaa como andas? Lo que pasa ahi es que estas validando sobre el input y no sobre el valor del input.

tienes que poner usuario.value y pass.value


Te quedaria:
function valida(){
var usuario = document.getElementById('usuario');
var pass = document.getElementById("password");

if((usuario.value == "")||(pass.value == "")){
window.alert("Los campos usuario y contraseña no pueden estar vacios");
}else{
envia('login');
}
}


Saludos

[jcastro3]

Jo-der..... muchisimas gracias de verdad, ya funciona perfectamente jaja el caso es que tenia los apuntes de clase delante y lo he qurido hacer tan rapido que no me he dado cuenta del .value jaja muchas gracias.

[Enano_la22]

jaja. si a veces pasa...despues de mirar tanto el codigo ya no podes ver el error.

Saludos!!!