problema con js

[morenita7722]

Hola
Tengo un formulario que quiero validar con una js, pero no funciona. Lo que hago:

<form action="alumnos.do" onsubmit="return validar(this)">
.........................
<input type="submit" name="operacion" value="CANCELAR" />
<input type="submit" name="operacion" value="ACEPTAR" onclick="selecciona(true)" />
<input type="reset" name="limpiar" value="LIMPIAR FORMULARIO" />
</form>

y la js:

//función para validar el formulario
function validar(formulario){
var correcto=true;
var mensaje="";
if (formulario.operacion.value=="ACEPTAR"){
if (isNaN(formulario.horasAsistenciaReal.value)){
formulario.horasAsistenciaReal.focus();
mensaje="El valor de las horas de asistencia real debe ser numérico";
correcto=false;
}

if (formulario.DNI.value==""){
mensaje="El DNI no puede estar vacío";
formulario.DNI.focus();
correcto=false;
}

if (!correcto)alert(mensaje);

return correcto;
//}
}

Saludos!!

[EPROM]

Yo haría asi la validación... no sé si te sirva!!

Código:

<form action="alumnos.do" name="f1">
.........................
<input type="button" value="CANCELAR" />
<input type="button" value="ACEPTAR" onclick="validar()" />
<input type="reset" name="limpiar" value="LIMPIAR FORMULARIO" />
</form>

Código:

//función para validar el formulario
function validar() {
var formulario = document.f1;

if (isNaN(formulario.horasAsistenciaReal.value)) {
 formulario.horasAsistenciaReal.focus();
 alert("El valor de las horas de asistencia real debe ser numérico");
 return;
}

if (formulario.DNI.value=="") {
 formulario.DNI.focus();
 alert("El DNI no puede estar vacío");
 return;
}
 f1.submit();
}

SALUDOS

[morenita7722]

Probaré tu código, gracias, pero aún así me gustaría saber que estoy haciendo mal en el que yo he puesto.
Saludos!!

[David]

No entiendo qué estás tratando de hacer con la comparación:

Código javascript:

if (formulario.operacion.value=="ACEPTAR")
 

Ya que hay dos botones submit llamados operacion, por lo que es un array de campos, así que hacer tal comparación no funcionará.

P.S.: En cuanto al código de EPROM lo que no concuerdo es usar el método submit() para enviar cuando lo correcto es usar un elemento submit. Véase: http://www.forosdelweb.com/f13/malas...itarse-620808/.

[morenita7722]

En función del botón que pulses el valor cambiará, sólo voy a validar el formulario en el caso que valga aceptar. El código javascript funciona si está puesto en la jsp, en etiquetas <script></script>, dentro de la cabecera. Sin embargo estoy intentando pasar ese javascript a una js para usarlo en más jsp y no va.

[David]

Como te dije, operacion es un array de objetos, por lo que .value == "ACEPTAR" NUNCA será verdadero, por lo cual siempre enviará el formulario sin validarlo. Además, hay que tener en cuenta que el botón "CANCELAR" (cuyo propósito desconozco) también enviará el formulario.

[morenita7722]

El valor de operacion (que puede ser aceptar o cancelar) lo tengo que recoger en un controlador de spring, y dependiendo del valor que tenga me envía a una jsp o a otra y hace las operaciones correspondientes (en este caso editar al alumno o no).
Lo que no entiendo es por que es un array de objetos, no acabo de verlo. Es un botón que puede tener un valor u otro, pero no los dos, por eso lo hice con .value.
No discuto que tu tengas razón, yo llevo muy poco tiempo utilizando javascript. Lo que si me extraña es que ese código puesto en la jsp si me valida el formulario, porque yo lo voy viendo con alert().

[David]

¿Por qué es un array de objetos?. Porque son dos input con el mismo nombre.

Una idea que se me ocurre es hacer la validación en el onclick del input "ACEPTAR".

[morenita7722]

Si, tienes razón en lo del onclick, ya lo había pensado, pero tengo que mirar si se puede llamar a dos funciones en el onclick y como. Voy a modificar eso y a ver si consigo que me funcionen las funciones desde la js... ya iré comentando como va la cosa.
Gracias David
Saludos!!

[David]

Para llamara a dos funciones solo necesitas separlas con el separador de instrucciones ";"

[morenita7722]

Hola!
Ya he cambiado la llamada a las dos funciones en el botón de aceptar.
Y encontré el error! que caprichoso es el javascript! jaja, un paréntesis que sobraba y no decía nada, ;)
De todas formas sigo haciendo pruebas con este código, si tengo problemas os vuelvo a dar un toque.
Saludos!!