validar formularios

[pagafantas]

hola tengo un problemilla con este javascript y es que no me valida bien los campos, tenia mas metidos por que los hice 1 a unopero los he reducido a 2 por que segun meto 2 no funcionan u.u

Código:

<script>

 function validar(formulario) {
	
	
     //valido el nombre
    if (formulario.nombre.value.length==0){
      document.getElementById('nombrec').innerHTML = '<img src="incorrecto.gif" />';
 formulario.nombre.focus();
<!--return (false);  -->
   
    }
	
	if (formulario.postal.value.length == 5) {
    
	document.getElementById('postalc').innerHTML = '<img src="ok.png" />';

return (false); 

} else {
document.getElementById('postalc').innerHTML = '<img src="incorrecto.gif" />';
   
 return (false); 
 }
  return (true); 
}

</script>

gracias

saludos

[Wasper]

¿Que error te da? ¿Te has asegurado que los nombres de los campos son los que nombras? ¿como haces la llamada para que te valide?

[pagafantas]

se me ha olvidado un punto y coma gracias de todas formas si me surge algun error lo pondre en este tema jeje muchas gracias de todas formas wasper

[pagafantas]

no era eso lo que me producia error pongo todo el codigo aver si veis algun fallo y es que me valida bn hasta que meto otro trozo de codigo y se carga el anterior
y firefox no me da error u.u aver si em podeis ayudar

Código:

<script>

 function validar(formulario) {
	
	
  var checkOK = "ABCDEFGHIJKLMNÑOPQRSTUVWXYZÁÉÍÓÚ" + "abcdefghijklmnñopqrstuvwxyzáéíóú ";
  var checkStr = formulario.nombre.value;
  var allValid = true; 
  for (i = 0; i < checkStr.length; i++) {
    ch = checkStr.charAt(i); 
    for (j = 0; j < checkOK.length; j++)
      if (ch == checkOK.charAt(j))
        break;
    if (j == checkOK.length) { 
      allValid = false; 
      break; 
    }
  }
  if (!allValid) { 
   
	document.getElementById('nombrec').innerHTML = '<img src="incorrecto.gif" />';
   
    
  
  }
  else {
   
	document.getElementById('nombrec').innerHTML = '<img src="ok.png" />'; 
   
  }
  
 
  if (formulario.nombre.value.length < 3) {
    
	document.getElementById('nombrec').innerHTML = '<img src="incorrecto.gif" />';
  

	 
  }
  
     if (formulario.postal.value.length == 5) {
    
	document.getElementById('postalc').innerHTML = '<img src="ok.png" />';


	
	} else{
  document.getElementById('postalc').innerHTML = '<img src="incorrecto.gif" />';
 <!--return (false); -->
 
 }
  
  
     if (formulario.telefono.value.length == 9) {
    
	document.getElementById('telefonoc').innerHTML = '<img src="ok.png" />';


	
	} else{
  document.getElementById('telefonoc').innerHTML = '<img src="incorrecto.gif" />';
<!--   return (false);  -->
 }
 
  var checkOK = "0123456789"; 
  var checkStr = formulario.postal.value; 
  var allValid = true; 
  var decPoints = 0; 
  var allNum = ""; 
  for (i = 0; i < checkStr.length; i++) { 
    ch = checkStr.charAt(i); 
    for (j = 0; j < checkOK.length; j++) 
      if (ch == checkOK.charAt(j))
        break; 
    if (j == checkOK.length) { 
      allValid = false; 
      break; 
    } 
    allNum += ch; 
  } 
  if (!allValid) { 
  document.getElementById('postalc').innerHTML = '<img src="incorrecto.gif" />';
  } 

  if ((formulario.correo.value.indexOf ('@', 0) == -1)||(formulario.correo.value.length < 5)) { 
    document.getElementById('correoc').innerHTML = '<img src="incorrecto.gif" />';
    return (false); 
  } else {
  
  document.getElementById('correoc').innerHTML = '<img src="incorrecto.gif" />';
 return (false); 
 }
  return (true); 
}



</script>

[Wasper]

Si puedes, pasame el codigo de error que te da, es decir, fallo en linea xx no es un objeto o es nulo o algo similar... en IE abajo a la izquierda sale un simbolo de admiracion amarillo, le das doble click y dime que pone...

[pagafantas]

pues no me da ninguno esa es la cuestion pero vamos que el error esta ene l correo que no lo valida bien las otras 3 opciones van bien pero el correo no u.u gracias por tu respuesta

[pagafantas]

aqui esta el error creo que dejare de molestarte wasper perdon por las molestias no se como lo ago pero siempre cometo los errores mas estupidos u.u perdon por acerte perder el tiempo

el error era este

if ((formulario.correo.value.indexOf ('@', 0) == -1)||(formulario.correo.value.length < 5)) {
document.getElementById('correoc').innerHTML = '<img src="incorrecto.gif" />';
return (false);
} else {

document.getElementById('correoc').innerHTML = '<img src="incorrecto.gif" />';
return (false);
}


e puesto incorrecto en las 2 u.u

[Wasper]

Para nada me haces perder el tiempo hombre!!!!! faltaria mas... disculpame tu a mi por no haberte entendido antes... pero si lo tienes solucionado el objetivo esta cumplido!!! Post con problema presentado y duda resuelta.