Ayuda con este código - Validar

gadget · #1 (**permalink**) 03/06/2003, 14:51

Lo que quiero es: solo si se digita algo en el campo email_solicitante debe validar que tenga arroba y un punto por lo menos.

No me está funcionando pues valida la @ y el . aun cuando no se digitó algo.

<html>
<head>
<title>Información</title>
<link href="main.css" rel="stylesheet">
<script LANGUAGE="JavaScript">

function Validar(form)
{

if (form.email_solicitante.value != "" &&
form.email_solicitante.value.indexOf('@', 0) == -1 ||
form.email_solicitante.value.indexOf('.', 0) == -1)
{ alert("Digite una drección de email correcta"); form.email_solicitante.focus(); return; }

form.submit();
}
</script>
</head>
<body bgcolor="#ffffff">

<form Method="POST" Action="Reg_Email.asp">
<center>
<table bgColor="#ffffff" border="0" cellPadding="2" cellSpacing="1" width="816" style="height: 120px">
<tr>
<td bgcolor="#c7dcff" width="328"><p align="left" class="ve08bldkno">Correo electrónico:</p></td>
<td bgcolor="#ebf1f5" width="331" colSpan=4>
<INPUT id=text12 style="width: 470; height: 23" width="300" name=email_solicitante size="60"></td>
</tr>
</table>
</center>
<br>
<center><table>
<tr>
<td colspan="4"><br>
<input TYPE="button" VALUE="Registrar solicitud" name="Envio" onClick="Validar(this.form)">
<input TYPE="reset" VALUE="Limpiar datos" name="B2">
</td>
</tr>
</table></center>
</form>
</body>
</html>

Kaopectate · #2 (**permalink**) 03/06/2003, 15:57

Hola gadget.

Yo lo intentaría asi:

Código PHP:

  function Validar(form){

 with (form.email_solicitante){

  if (value != "")

   if (value.indexOf('@', 0) == -1 || value.indexOf('.', 0) == -1){

    alert("Digite una drección de email correcta");

    focus();

    return;

   }

 }

 form.submit();

}

Lo que sucede es basicamente que las condiciones que usas necesitan paréntesis o que las separes. No es lo mismo decir:

if (no_vacio && valido_@ || valido_punto)

que decir

if (no_vacio && (valido_@ || valido_punto))

En el primer caso, con que la última condición sea válida, hará que toda la cadena sea válida.

Saludos.

Manoloweb · #3 (**permalink**) 03/06/2003, 16:44

Y por que no usar expresiones regulares???

gadget · #4 (**permalink**) 03/06/2003, 18:16

Funcionó bien, se trata de una validación sencilla, pero realmente lo que estoy haciendo es la validación de todo un formulario, del cual el email era sola una parte y ahora lo voy a integrar.

Sobre lo que dices Manoloweb ya revisé las FAQS y me parece correcta tu apreciación sobre validar con expresiones regulares. En fin ya tengo dos soluciones.

Les agradezco su atención.

Saludos