problemas com lavalidacion de un formaulario

[soldierjesus]

hola como estan
la verdad no se que esta mal en la validacion de este formulario
les pido su colaboracion
saludos

Código:

<script>
function validar()
{
  var x;
  x=window.document.form1;
//-----------------------------------------------------------------
//Validacion de la Cedula
/*La funcion isNaN es la encargada de validar si una variable es de tipo numerico
Si la variable no es de tipo numerico retorna un valor true*/
  if(x.nombre.value=="")
  {
    alert("Debe digitar su Nombre");
    x.nombre.focus();
    return;
  }
 
//-------------------------------------------------------------------------------
if(x.empresa.value=="")
  {
    alert("Debe digitar el nombre de la empresa");
    x.empresa.focus();
    return;
  }

  if(x.acti.value=="")
  {
    alert("Debe digitar su Actividad");
    x.acti.focus();
    return;
  }

//---------------------------------------------------------------------------------
if(x.pais.value=="")
  {
    alert("Debe digitar el Pais");
    x.pais.focus();
    return;
  }

  if(x.ciudad.value=="")
  {
    alert("Debe digitar la Ciudad");
    x.ciudad.focus();
    return;
  }

//---------------------------------------------------------------------------------
  if(x.dir.value=="")
  {
    alert("Debe digitar la Dirección");
    x.dir.focus();
    return;
  }
  
  if(x.tele.value=="")
  {
    alert("Debe digitar Numero de telefono");
    x.dir.focus();
    return;
  }
  else
 {
   if(isNaN(x.tele.value)==true)
   {
     alert("El valor Ingresado en el Telefono no es Numerico");
     x.tele.focus();
     return;
     }
 }
//-------------------------------------------------------------------------------------
if(x.mail.value=="")
  {
    alert("Debe digitar su direccion de correo");
    x.mail.focus();
    return;
  }
 //------------------------------------------------------------------------------------
if(x.web.value=="")
  {
    alert("Debe digitar su pagina web");
    x.web.focus();
    return;
	}
	if(x.destino.value=="")
  {
    alert("Debe selecionar un destino");
    x.destino.focus();
    return;
	}
		if(x.asun.value=="")
  {
    alert("Debe digitar el asunto");
    x.asun.focus();
    return;
	}
		if(x.mensaje.value=="")
  {
    alert("Debe escribir un mensaje");
    x.mensaje.focus();
    return;
	}
		x.submit();
	}
    </script>

este es el form:

<form name="form1" id="form1" action="envia_mail.php" method="post" >
<table width="467" border="0" align="center" cellpadding="0" cellspacing="5" class="Estilo10" >
<tr>
<td align="right" class="Estilo9" >&nbsp;</td>
<td class="style7 Estilo5 Estilo5 style8" ><div align="left">
<p>Formulario de contacto</p>
<p>&nbsp;</p>
</div></td>
</tr>
<tr>
<td width="24%" align="right" class="Estilo9" ><div align="left" class="style7">
<div align="left"> Nombre Completo</div>
</div></td>
<td width="76%" class="Estilo9" ><span class="style1 Estilo2 Estilo5 ">
<input id="Nombre" type="text" name="Nombre" size="30" value="" />
<span class="style5">* </span></span></td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" ><div align="left" class="style7">
<div align="left">Nombre Empresa</div>
</div></td>
<td class="Estilo9" ><span class="style1 Estilo2 Estilo5 ">
<input id="Empresa" type="text" name="Empresa" size="30" value="" />
</span></td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" ><div align="left" class="style7">
<div align="left">Actividad</div>
</div></td>
<td class="Estilo9" ><span class="style1 Estilo2 Estilo5 ">
<input id="Actividad" type="text" name="Acti" size="30" value="" />
</span></td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" ><div align="left" class="style7">
<div align="left">Pa&iacute;s</div>
</div></td>
<td class="Estilo9" ><span class="style1 Estilo2 Estilo5 ">
<input id="pais" type="text" name="pais" size="30" value="" />
<span class="style5">* </span></span> </td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" ><div align="left" class="style7">
<div align="left">Ciudad</div>
</div></td>
<td class="Estilo9" ><span class="style1 Estilo2 Estilo5 ">
<input id="ciudad" type="text" name="ciudad" size="30" value="" />
</span>
<label></label>
<span class="style5">*</span> </td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" ><div align="left" class="style7">
<div align="left">Direcci&oacute;n</div>
</div></td>
<td class="Estilo9" ><label>
<input name="dir" type="text" id="dir" size="30" />
</label> </td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" ><div align="left" class="style7">
<div align="left">Tel&eacute;fono N&ordm;</div>
</div></td>
<td class="Estilo9" ><span class="style1 Estilo2 Estilo5 ">
<input id="tele" type="text" name="tele" size="30" maxlength="30" value=""/>
<span class="style5">* </span><br />
</span> </td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" ><div align="left" class="style7">
<div align="left">E-mail</div>
</div></td>
<td class="Estilo9" ><span class="style1 Estilo2 Estilo5 ">
<input id="Mail" type="text" name="Mail" size="30" value=""/>
<span class="style5">* </span><br />
</span></td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" style="height: 24px" ><div align="left" class="style7">
<div align="left"> P&aacute;gina Web </div>
</div></td>
<td class="Estilo9" style="height: 24px" ><span class="style1 Estilo2 Estilo5 "><span class="Estilo9" style="height: 24px">
<input id="web" type="text" name="web" size="30" maxlength="100" value="" />
</span></span></td>
</tr>
<tr>
<td align="right" valign="top" class="Estilo9" style="height: 24px" ><div align="left" class="style7">Destino Mensaje</div></td>
<td class="Estilo9" style="height: 24px" ><span class="Estilo9" style="height: 24px">
<select name="destino" id="destino">
<option>Mantenimiento y reparación</option>
<option>Filtración</option>
<option>Dotación</option>
<option selected="selected">-Seleccione el Destino-</option>
</select>
</span></td>
</tr>
<tr>
<td align="right" valign="top" class="style7" style="height: 24px" ><div align="left">Asunto</div></td>
<td class="Estilo9" style="height: 24px" ><input type="text" name="asun" id="asun" /></td>
</tr>
<tr align="center" >
<td class="Estilo9"><div align="left"><b><span class="style6">Mensaje</span>:</b></div></td>
<td class="Estilo9"><div align="left"><span class="style1 Estilo2 Estilo5 ">
<textarea id="mensaje" name="mensaje" cols="24" rows="6"></textarea>
</span></div></td>
</tr>
<tr align="center" >
<td class="Estilo9"><p align="left" class="style1 Estilo2 Estilo5 "><b><br />
</b><br />
</p></td>
<td class="Estilo9"><div align="left">
<label>
<input type="submit" name="envia" id="envia" value="Enviar" onclick="validar()" />
</label>
<label>
<input type="reset" name="cancel" id="cancel" value="Cancelar" />
</label>

</div></td>
</tr>
</table>
</form>

[monxas]

yo lo que hago es poner return false; en todos esos returns, y en vez de x.submit() un return true;
ademas lo hago un poco diferente. mira, te pongo un cacho de uno mio, si te gusta genial!


El form:

Código javascript:

<form id="Fform" method="post" action="tratar_form.asp" onSubmit="return ValidarFormulario(this);">
...
 

y luego arriba del todo la funcion:

Código javascript:

<script type="text/javascript">
function ValidarFormulario(f){
    if(f.Nombre.value==""){
    alert("Por favor, introduzca el nombre de la sección");
    f.eNombre.focus();
    return false;
  }
  if (isNaN(f.eOrden.value)){
    alert("El campo orden debe ser un valor numérico");
    f.eOrden.focus();
    return false;
    }
  return true;
}
 

No hace falta declarar el formulario porque lo paso con el (this)
si devuelve false no lo envia y si es true si.

Si necesitas mas aclaracion estare pendiente

suerte!

[soldierjesus]

hey amigo valida y todo pero igual al darle click se me va y no valida

saludos

[monxas]

te valida hasta el final?
yo lo que hago es ir metiendo alerts y ver donde esta el codigo roto, que es por eso lo que falla.
si has cambiado el codigo JS de la validacion y lo vuelves a pegar lo reviso yo tambien

[soldierjesus]

en el primero lo dejo en blanco,me aparece el mensaje pero igual al darle click se envia

[monxas]

pon
return false;
en vez de
return;

[soldierjesus]

ok gracias