Validar correo electronico

[floyd_23]

Hola,

Creo lo hago todo bien (aunque es obvio que no, si no, no escribiría este post), pero no consigo que la siguiente función me valide el campo email. Meta lo que meta en el campo de texto me salta el alert.

Código HTML:

<form name="formulario" method="get">
    <fieldset>
        <legend>Datos personales</legend>
        Nombre: <br />
        <input type="text" name="name" />
        <br />
        Apellidos: <br />
        <input type="text" name="apellidos" />
        <br />
        Direcci&oacute;n: <br />
        <input type="text" name="direccion" />
        <br />
        Tel&eacute;fono: <br />
        <input type="text" name="telefono" />
        <br />
        E-mail: <br />
        <input type="text" name="email" />
    </fieldset>
    <fieldset>
        <legend>Datos de la compra</legend>
        Patatas: <input type="checkbox" name="patatas" />&nbsp;(3&euro;/Kg)
        <br />
        ¿Cuantos kilos de patatas desea? <br />
        <input type="text" name="kpatatas" />
        <br />
        Melones: <input type="checkbox" name="melones" />&nbsp;(2&euro;/Kg)
        <br />
        ¿Cuantos kilos de melones desea? <br />
        <input type="text" name="kmelones" />
        <br />
        Sandias: <input type="checkbox" name="sandias" />&nbsp;(1&euro;/Kg)
        <br />
        ¿Cuantos kilos de sandias desea? <br />
        <input type="text" name="ksandias" />
    </fieldset>
    <input type="button" name="calcular" value="Calcular Total" />&nbsp;<input type="button" name="comprar" value="Comprar" onclick="comprobar_personal()" />
    </form> 

Y ahora el codigo javascript:

Código:

function comprobar_personal (){
    var email = formulario.email.value;
    var telefono = formulario.telefono.value;
    var ok;
    ok = true;
    if (formulario.name.value==""){
        alert ("Debe escribir su nombre");
        formulario.name.focus();
        ok=false;
    }
    else if (formulario.apellidos.value==""){
        alert ("Debe escribir sus apellidos");
        formulario.apellidos.focus();
        ok=false;
    }
    else if (formulario.direccion.value==""){
        alert ("Debe escribir su direccion postal");
        formulario.direccion.focus();
        ok=false;
    }
    else if ( !(/^\d{9}$/.test(telefono)) ){
        alert ("Debe escribir un numero de telefono correcto");
        formulario.telefono.focus();
        ok=false;
    }
    else if ( !(/^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,3,4})+$/.test(email)) ){
        alert ("Debe escribir un correo electronico correcto");
        formulario.email.focus();
        ok=false;
    }
}

Saludos y gracias de antemano.

[_cronos2]

Creo que es por \w{2,3,4} . Esa expresión regular creo que la usé yo hace tiempo, pero me dijeron que no valía porque entre las dos llaves ({}) puede haber como máximo dos números. Ej:

\d{1} // un número
\d{1,} // como mínimo un número
\d{1,3} // de uno a tres números
\d{1,2,3} // no existe

Saludos (:

[IsaBelM]

cuantificadores

[_cronos2]

Iniciado por IsaBelM

cuantificadores

Me gusta más el de la W3Schools, aunque vienen a ser lo mismo.
Saludos (:

[IsaBelM]

yo soy una devota de w3schools. ambos los tengo en marcadores y cuando linkeo y se da este echo, pues prefiero hacerlo en español. aparte que el de msdn te explica como evitar el molesto greedy

[floyd_23]

Muchas gracias a los dos, ha sido quitar el 3 de la expresion y ya si que funciona.

Mil gracias, saludos.