Problema para validar dos campos

[yole]

Hola,
Tengo el siguiente script para validar dos campos pero falla en lo siguiente:

al presionar enviar, efectivamente me indica el campo vacío, pero posterior a eso igual envía el formulario.
Se supone que no debiera enviarlo hasta completar los dos campos.

Este es el script. A ver si me dan una mano para descubrir el problema.

function valida_envia(){
//valido el nombre
if (document.fvalida.user.value.length==0){
alert("Debe ingresar su nombre de usuario")
document.fvalida.user.focus()
return false;
}
if (document.fvalida.pass.value.length==0){
alert("Tiene que escribir su password")
document.fvalida.pass.focus()
return false;
}


document.fvalida.submit();

}

De antemano, gracias.

[lair]

Hola.

prueb acambiendo esto:
document.fvalida.user.value.length==0
por:
document.fvalida.user.value==''

Ora cosa te recomiendo que hagas referencia a los campos por su id de la siguiente manera:
document.getElementById("id_del_campo").value

Suerte

[yole]

Ya resolví el problema.

Al botón enviar del formulario había que quitarle la acción de envío.
Osea:

<input type="submit" name="button" id="button" value="Entrar" onclick="valida_envia()"/>

por:

<input type="button" name="button" id="button" value="Entrar" onclick="valida_envia()"/>

eso era todo.

Gracias lair por tu ayuda, también agregue tu sugerencia.

[caricatos]

Hola:

Al enviar el formulario con javascript, estás en el caso de un formulario "no-accesible", ya que no se enviará nunca con javascript desactivado.
La forma accesible (y correcta) es mediante un botón del tipo submit, y la validación desde el evento del mismo nombre del formulario (<form onsubmit...)

Al evento debes pasarle el valor false para cancelar el envío... y ya que he mencionado la forma correcta, recalco que la validación "buena" debe hacerse en el servidor.

Saludos