Formulario siempre envia

[alexg88]

Es sencillo, sólo tienes que poner lo siguiente (fíjate en el return, es imprescindible)

Código HTML:

<form action="registrar.php" class="formEncuesta" name="formulario" method="post" onsubmit="return validar()">
...
</form>
 

La función validar debe ser una función que devuelva verdadero si se puede enviar el formulario y falso si no.

Ten en cuenta que la función validar que has hecho es incorrecta, no tiene sentido que declares la funciones dentro de la función validar, se tienen que declarar fuera.

Puede ser algo así (lo he hecho rápido así que verificalo):

Código Javascript:

function validar(){
return validar('evaluador1') && validar('evaluador2') && validar('evaluador3') && verificar('pregunta')
}
 
 
function validarEntero(input){
return !isNaN(input)&&parseInt(input)==input;
}
 
 
//Campos valoración
 
function validar(campo){ 
var valoracion = document.getElementById(campo);   
if((!validarEntero(valoracion.value))||(valoracion.value == "")||(valoracion.value <= 0)||(valoracion.value > 100)){
valoracion.value = "";
valoracion.focus();
return false;
}
else return true;
}
 
 
//Campo pregunta
 
function verificar(campo){
var pregunta = document.getElementById(campo); 
if((!validarEntero(pregunta.value))||(pregunta.value == "")||(pregunta.value < 1)||(pregunta.value > 90)){
pregunta.value = "";
pregunta.focus();
return false;
}
else return true;
}
 

Te paso un enlace que explican algunas cosas de validaciones sencillas: http://www.elcodigo.net/tutoriales/j...vanzado13.html

Saludos.