Error al validar campo obligatorio en Formulario, se envia siempre y con errores

[kahlito]

Hola estoy intentando hacer un formulario php y validar algunos campos vacios (hacerlos obligatorios) con javascript de la siguiente manera:

Código HTML:

<script language="JavaScript">
   function valida_envia(){
    //valido el nombre
    if (document.fvalida.nombre.value.length==0){
       alert("Tiene que escribir su nombre")
       document.fvalida.nombre.focus()
       return false;
    } 
	if (document.fvalida.apellido.value.length==0){
       alert("Tiene que escribir su apellido")
       document.fvalida.apellidos.focus()
       return false;
    } 
	else {//el formulario se envia
    alert("Muchas gracias por enviar el formulario");
    document.fvalida.submit();}
} 
</script> 

Código HTML:

<form name="fvalida" action="formulariovalida.php" method="post" >
<input type="text" name="nombre" size="30" maxlength="50" />
<input type="text" name="apellidos" size="30" maxlength="50" />
<input type="submit" value="  Enviar  " onclick="valida_envia()"/> 

Sin embargo al darle a submit me sale la alerta de rellenar el nombre y luego automáticamente se envia el formulario sin haber puesto el nombre ni nada de nada , ¿alguna otra solución?

[Saruman]

haslo asi mejor:

Código PHP:

  <script language="javascript">
 function ValidarContacto(f) {
  if (f.nombre.value == "") {
   alert("Debe colocar su Nombre.");
   f.nombre.focus();
   return false;
  }
  
  if (f.apellidos.value == "") {
   alert("Debe colocar su Apellido.");
   f.apellidos.focus();
   return false;
  }
  
  document.body.style.cursor = "wait";
  
  return true;
 }
</script> 
  

Código:

 
<form name="fvalida" action="formulariovalida.php" method="post" onsubmit="return valida_envia(this)">
<input type="text" name="nombre" size="30" maxlength="50" />
<input type="text" name="apellidos" size="30" maxlength="50" />
<input type="submit" value="  Enviar  ">
</form>
 

te falto colocar el tag "</form>"

saludos

[kahlito]

Hola Saruman, ahora si me funciona.

Dejo aquí un pequeño cambio (de nombres) sin importancia por si le sirve a alguien:

Código HTML:

<form name="f" action="formulariovalida.php" method="post" onsubmit="return ValidarContacto(this)"> 

Muchas gracias por tu ayuda

Saludosss

[Saruman]

no, no es necesario.
el this que puse es para que no halla necesidad de ponerle el nombre al form... comprendes?

chequeate de nuevo la funcion

saludos

[kahlito]

Hola de nuevo.

Llevas razón en lo del name veo que no hace falta.

Sin embargo lo dejo con el nombre de la funcion cambiada para no confundir

onsubmit="return valida_envia(this)"> por onsubmit="return ValidarContacto(this)">

Gracias de nuevo, saludosss

[realxp]

No se como borrar este mensaje, ya puse uno nuevo en la principal del JS