Hola. Mejor hazlo con expresiones regulares, puesto que de la forma como lo estas haciendo permite espacios en blanco, con expresiones regulares puedes incluso evitar que digiten algunos caracteres. Por lo pronto puedes hacerlo así:
Código:
function valida(){valor = document.forms.formulario.usuario.value;
if( valor == null || valor.length == 0 || /^\s+$/.test(valor) ) {alert("Ingresa tu nombre de usuario");
document.forms.formulario.usuario.focus();
return false;
}
valor = document.forms.formulario.pass.value;if( valor == null || valor.length == 0 || /^\s+$/.test(valor) ) {
alert("Ingresa tu contraseña");
document.forms.formulario.pass.focus();
return false;
}
document.forms.formulario.submit();
}
Puedes leer más
aquí y
aquí, y descargar un ejemplo
acá. Saludos.