problema con validación de formulario

[mipavluk]

Hola a todos! Es la primera vez que hago un formulario que valide los campos antes de enviar... he seguido otros posts del foro... pero tengo un error y no logro solucionarlo. El script valida, sale el pop up indicando que falta completar un campo, pero luego el formulario es enviado igual, a pesar del error. Me ayudan? gracias desde ya...

Este es mi script:

Código:

<script>
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; 
    } 

        //valido el mail 
    if (document.fvalida.email.value.length==0){ 
       alert("Tiene que escribir su e-mail") 
       document.fvalida.nombre.focus() 
       return false; 
    } 

    //valido el telefono 
    if (document.fvalida.telefono.selectedIndex==0){ 
       alert("Tiene que escribir su telefono.") 
       document.fvalida.interes.focus() 
       return false; 
    } 

    //valido el celular 
    if (document.fvalida.celular.selectedIndex==0){ 
       alert("Tiene que escribir su celular.") 
       document.fvalida.interes.focus() 
       return false; 
    } 

    //valido el mensaje 
    if (document.fvalida.mensaje.selectedIndex==0){ 
       alert("Tiene que escribir su mensaje.") 
       document.fvalida.interes.focus() 
       return false; 
    } 

    //el formulario se envia 
    alert("Muchas gracias por enviar el formulario"); 
    document.fvalida.submit(); 
} 
</script>

y este es mi formulario:

Código:

<form id="fvalida" name="fvalida" method="post" action="enviar.php" onSubmit="javascript:return validar();">
                  <label>
                  <p align="center"><span class="style2">Nombre:</span>
                    <input name="nombre" type="text" id="nombre" size="64" />
                  </p>
                  <p align="center" class="style2">
                    <label>E-Mail:
                    <input name="email" type="text" id="email" size="66" />
                    </label>
                  </p>
                  <p align="center" class="style2">
                    <label>Teléfono Fijo:
                    <input name="telefono" type="text" id="telefono" size="58" />
                    </label>
                  </p>
                  <p align="center" class="style2">
                    <label>Celular:
                    <input name="celular" type="text" id="celular" size="65" />
                    </label>
                  </p>
                  <p align="center" class="style2">
                    <label>Mensaje:
                    <textarea name="mensaje" id="mensaje" cols="60" rows="8"></textarea>
                    </label>
                  </p>
                  <p align="center" class="style2">
                    <label>
                    <input type="submit" name="enviar" id="enviar" value="Enviar" onclick="valida_envia()"/>
                    </label>
                  </p>
                </form>

[lair]

Hola.

prueba cambiando tu boton de envio
cambia esto:
<input type="submit" name="enviar" id="enviar" value="Enviar" onclick="valida_envia()"/>
por esto:
<input type="button" name="enviar" id="enviar" value="Enviar" onclick="valida_envia()"/>

suerte.

[mipavluk]

muchas gracias lair, funciona perfecto

mientras he seguido leyendo y creo que debería mejorar mi validación en el campo e-mail, corregir la función para que verifique un e-mail válido en lugar de que solamente rellene el campo con algo

pero no se cómo modificar lo que ya tengo... me ayudas?

gracias desde ya

[lair]

Claro.

para hacer ese tipo de validaciones necesitas hacer uso de Expresiones Regulares

cambia tu if del mail por este:

if (/^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,3,4})+$/.test(document.fvalida.email.value))

Suerte.

La informacion la saque de aqui:
http://lineadecodigo.com/javascript/...on-javascript/

[mipavluk]

nop, no funciona...
si el e-mail no es correcto, no sale ningún aviso... el formulario se envía... y en el mail que recibo aparece vacío el dato de e-mail... como si no hubieran escrito nada...

[lair]

Prueba con estas expresiones, tambien viene un breve tutorial de como construir tus expresiones
http://www.webintenta.com/validacion...avascript.html

[mipavluk]

ok... probando probando... llegué a esta

Código:

    if ( /^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,3,4})+$/);{ 
       alert("Tiene que escribir un e-mail válido") 
       document.fvalida.email.focus() 
       return false; 
    }

pero resulta que no me acepta ningún e-mail... como que todos son inválidos....

gracias desde ya por tu tiempo...

[Adler]

Hola

Si que es usada esa expresión. Aquí tienes el problema resuelto

Suerte

[mipavluk]

Hola Adler

la corregí de esta manera:

Código:

    if (/^[0-9a-zA-Z]+(?:[._][0-9a-zA-Z]+)*@[0-9a-zA-Z]+(?:[._-][0-9a-zA-Z]+)*\.[0-9a-zA-Z]{2,4}$/);{ 
       alert("Tiene que escribir un e-mail válido") 
       document.fvalida.email.focus() 
       return false; 
    }

pero sigue sin funcionar, sigue saliendo el cartel de que debo escribir un correo válido

[Adler]

Hola

Fíjate bien en código, no copies solamente la expresión

if (/^[0-9a-zA-Z]+(?:[._][0-9a-zA-Z]+)*@[0-9a-zA-Z]+(?:[._-][0-9a-zA-Z]+)*\.[0-9a-zA-Z]{2,4}$/.test(valor)){

Suerte

[mipavluk]

con esta nueva modificación, al darle click al botón de enviar, no envía, no sale ningún pop up y me aparece el error en pagina en el navegador...

te iba a pedir que revisaras todo mi script... porque me da la impresión de que tengo algo mal...

Código:

<script>
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; 
    } 

        //valido el mail 
    if (/^[0-9a-zA-Z]+(?:[._][0-9a-zA-Z]+)*@[0-9a-zA-Z]+(?:[._-][0-9a-zA-Z]+)*\.[0-9a-zA-Z]{2,4}$/.test(valor)){ 
       alert("Tiene que escribir un e-mail válido") 
       document.fvalida.email.focus() 
       return false; 
    } 

    //valido el telefono 
    if (document.fvalida.telefono.value.length==0){ 
       alert("Tiene que escribir su telefono.") 
       document.fvalida.telefono.focus() 
       return false; 
    } 

    //valido el celular 
    if (document.fvalida.celular.value.length==0){ 
       alert("Tiene que escribir su celular.") 
       document.fvalida.celular.focus() 
       return false; 
    } 

    //valido el mensaje 
    if (document.fvalida.mensaje.value.length==0){ 
       alert("Tiene que escribir su mensaje.") 
       document.fvalida.mensaje.focus() 
       return false; 
    } 

    //el formulario se envia 
    alert("Muchas gracias por enviar el formulario"); 
    document.fvalida.submit(); 
} 
</script>

[mipavluk]

en el otro post... en la función aparece (valor)

function validarEmail(valor) {

en cambio en la mía no... porque tengo una función para todo...

[Adler]

Hola

valor en tu caso es document.fvalida.email.value

suerte

[mipavluk]

ahora no valida el e-mail... osea... coloco cualquier palabra, sin arroba, sin .com y manda el formulario sin ningún pop up
=(

[mipavluk]

if (/^[0-9a-zA-Z]+(?:[._][0-9a-zA-Z]+)*@[0-9a-zA-Z]+(?:[._-][0-9a-zA-Z]+)*\.[0-9a-zA-Z]{2,4}$/.test(document.fvalida.email.value)){
alert("Tiene que escribir un e-mail válido")
document.fvalida.email.focus()
return false;
}

[lair]

Prueba con este codigo:

Código HTML:

<html>
<head>
<title>Ejemplo de función para validar dirección de email en javascript</title>
<script language="JavaScript">
function valEmail(){    // Cortesía de http://www.ejemplode.com
        valor=document.getElementById('mail').value;
    re=/^[_a-z0-9-]+(.[_a-z0-9-]+)*@[a-z0-9-]+(.[a-z0-9-]+)*(.[a-z]{2,3})$/
    if(!re.exec(valor))    {
        alert("correo no valido");
    }else{
        alert("correo valido");
    }
}
 
</script>
<body>
<form action="ejemplode.com">
Email:<input type="text" name="email" id='mail'>
<input type="button" onClick="valEmail()">
</form>
</body>
</html>
 

[mipavluk]

pero este código... solo valida el e-mail...

cómo agrego ahí la validación de campo no vacío de todos los demás campos de mi formulario?

tengo:

nombre
e-mail
teléfono
celular
mensaje

[Adler]

Hola

A ver, así

Código Javascript:

//valido el mail 
var valor = document.fvalida.email.value;
var patron = /^[0-9a-zA-Z]+(?:[._][0-9a-zA-Z]+)*@[0-9a-zA-Z]+(?:[._-][0-9a-zA-Z]+)*\.[0-9a-zA-Z]{2,4}$/;
   if (patron.test(valor)){
       alert("Tiene que escribir un e-mail válido") 
       document.fvalida.email.focus() 
       return false; 
    }
 

Suerte

[mipavluk]

ahora si!!!!!! graciassssssssssssssssss