Problemas al validar campos con script

[Coplo]

Hola, bueno he hecho un formulario de inicio de cesion y he visto que hay personas que no le dan clic al boton de inicio de sesion y cuando terminan de rellenar los campos de texto apretan el enter, mi problema es el siguiente al momento que los campos de texto esten vasion y aprete el enter sale un mensaje de alerta con script disiendo que pofavor rellene los campos vasios y al momento de hacer clic en aceptar del script se dirige a la siguiente pagina en vez de quedarse en la misma pagina a rellenas los campos vacios, me podrian a ayudar porfavor.
Aqui les dejo mi codigo

Formulario

Código HTML:

<script language="javascript" src="js/validar.js"></script>
<form action="login.php" method="post" name="form1" id="form-login" style="clear: both;">
	<p id="form-login-username">
		<label for="modlgn_username">Nombre de usuario</label>
		<input name="user" id="user" type="text" class="inputbox" size="15" value=""/>
	</p>

	<p id="form-login-password">
		<label for="modlgn_passwd">Contraseña</label>

		<input name="password" id="password" type="password" class="inputbox" size="15" value=""/>
	</p>
		
	<div class="button_holder">

	<div class="button1">
		<div class="next">
			<a onclick="validar();">
				Acceder</a>

		</div>
	</div>
	</div>
	<div class="clr"></div>

<input type="submit" style="border: 0; padding: 0; margin: 0; width: 0px; height: 0px;" value="Acceder" onclick="validar();"/>
</form> 

script

Código:

function validar(){
    //validar el user
    if (document.form1.user.value.length==0){
       alert("Ingrese el User porfavor")
       document.form1.user.focus()
       return 0;
    }
	
	if (document.form1.password.value.length==0){
		alert("Ingrese el Password porfavor")
		document.form1.password.focus()
		return 0;
	
	}
	
	document.form1.submit(); 
}

[helacer]

creo que con esto le debe funcionar

Código HTML:

function validar(){
    //validar el user
    if (document.form1.user.value.length==0){
       alert("Ingrese el User porfavor")
       document.form1.user.focus()
    }
	else if (document.form1.password.value.length==0){
		alert("Ingrese el Password porfavor")
		document.form1.password.focus()
	}
	else
		document.form1.submit(); 
}

[Coplo]

Gracias pero sigue con el mismo problema, los campos estan vacios apreto enter aparece el script le ago clic en aceptar si se dirige a la siguiente pagina.

[HackmanC]

Hola,

Posiblemente, te funcione esto...
Elimina el vínculo a acceder y el evento onclick del botón submit,
y usa el mismo nombre para el name y el id del formulario.

Iniciado por Coplo

Código:

<form action="login.php" method="post" name="form1" id="form1" style="clear: both;">
...
	<div class="button1">
		<div class="next">
			<a onclick="validar();">
				Acceder</a>
		</div>
	</div>
<input type="submit" style="border: 0; padding: 0; margin: 0; width: 0px; height: 0px;" value="Acceder" onclick="validar();"/>
</form>

Y el Javascript cambialo completamente por este código,
donde ... al cargar la página se asigna el evento para validar (check)
el usuario y la contraseña:

Código:

<script type="text/javascript">
<!--
window.onload = function() {
 document.getElementById("form1").onsubmit = function() {
  return check(this.username, "Usuario") && check(this.password, "Contraseña");
 }
}
function check(v, n) {
 if(v.value == ""){
  alert("No debe dejar vacío el campo : " + n); v.focus();
  return false;
 }
 return true;
}
// -->
</script>

Saludos,