Validacion Formulario

[orchating]

Hoola vereis tengo este codigo:

Código:

<script>
    function comprobar()
	{
	var mal=false;
    numero=document.forms[0].elements.length;
	var pos1, pos2, pos3 = true; 
    var correo = document.formulario.email.value; 
    for(a=0;a<numero;a++){
    if (document.forms[0].elements[a].value==""){document.forms[0].elements[a].style.backgroundColor="#ffffcc";mal=true;}
    else{document.forms[0].elements[a].style.backgroundColor="white";}
    } 
    pos1 = correo.indexOf('@', 0); 
    pos2 = correo.indexOf('.', 0); 
    pos3 = pos3 && (pos1 > 0); 
    pos3 = pos3 && (pos2 != -1); 
    pos3 = pos3 && (pos1 < pos2 - 1); 
    pos3 = pos3 && (pos2 < correo.length - 1);   
	if(!pos3){alert("E-mail Incorrecto");}
    else{document.forms[0].submit()}
    } 	
	if(mal){alert("Por favor, rellene todos los campos");}
    else{document.forms[0].submit()}
    }
    </script>

No veo donde esta el fallo lo que pretendo hacer es que cuando no inserte cualquiera campo diga rellenalos y se ponga de otro color y si meten el correo y este esta mal que diga email incorrecto, pero no se no lo hago funcionar.

Si alguien su`piera donde ve el fallo... muchas gracias de antemano

[orchating]

Tengo psobiblemente una mejor solucion, Estoy intentado convinarlos pero soy incapaz, me valida y aunke el correo este mal lo envia...


1º Script que valida los campos.

Código:

<script>
    function comprobar(){var mal=false;
    numero=document.forms[0].elements.length;
    for(a=0;a<numero;a++){
    if (document.forms[0].elements[a].value==""){document.forms[0].elements[a].style.backgroundColor="#ffffcc";mal=true;}
    else{document.forms[0].elements[a].style.backgroundColor="white";}
    }
    if(mal){alert("Por favor, rellene todos los campos");}
    else{document.forms[0].submit()}

    }
    </script>

2º Script que valida que el correo este bien.

Código:

<script language="JavaScript"> 
     function Validar(form)
	{ 
      var pos1, pos2, pos3 = true; 
      var correo = document.formulario.email.value; 
      if (correo == "")
	{
	  document.formulario.email.style.backgroundColor="#ffffcc";
      document.formulario.email.focus(); 
      return false; 
    } 
      pos1 = correo.indexOf('@', 0); 
      pos2 = correo.indexOf('.', 0); 
      pos3 = pos3 && (pos1 > 0); 
      pos3 = pos3 && (pos2 != -1); 
      pos3 = pos3 && (pos1 < pos2 - 1); 
      pos3 = pos3 && (pos2 < correo.length - 1); 
      if (!pos3)
	{ 
      alert("E-mail Incorrecto"); 
      document.formulario.email.focus(); 
      return false; 
    } 
   } 
  </script>

Una pequeña ayuda se agradece...

[caricatos]

Hola:

Llevo unos días viendo validaciones horrorosas... forzadas con el método submit()... el mismo caso tuyo.

Te recomiendo que leas este artículo/tutorial: El abc de los formularios, donde se explica bastante bien las validaciones (escrito después de algunos años de experiencia), y también encontrarás la validación de emails con expresiones regulares...

También se explica cómo combinar validaciones (pero correctamente)

Saludos

[orchating]

Gracias de todas maneras caricatos encontre un blog en el que modifique unas cosillas y me funciono a la perfeccion os invito a ver la pagina todavia esta en construcion.

http://www.cercedillaventura.com