Completar validación de Formulario

[majete]

Hola chicos... tengo un código javascript que me hace una validación de que ciertos campos de un encabezado se rellenan: Nombre, Grado, etc...

Código Javascript:

function validarFormulario() {
if (evaluacion.nombre.selectedIndex==0) {  
    alert("Debes introducir obligatoriamente el Nombre de la persona evaluada");   
    evaluacion.nombre.focus(); return;   
    }
 
myOption2 = -1;
   for (u=evaluacion.grado.length-1; u > -1; u--) {
   if (evaluacion.grado[u].checked) {
   myOption2 = u; u = -1;
   }
   }
   if (myOption2 == -1) {
   alert("Debes introducir obligatoriamente tu grado de conocimiento de la persona evaluada");
   evaluacion.id.focus(); return;
   }
 

Entonces, mi problema viene ahora cuando quiero validar unos campos de lista desplegable en el mismo formulario, que ya no forman parte del encabezado.
No puedo utilizar el mismo método que antes porque son 30 campos y claro, se alargaría el código una barbaridad y ralentizaría la carga.

Quiero hacer un bucle FOR para que pase por todos los campos y vea si alguno no está seleccionado, me salte una venta de alerta indicando que se ha dejado en blanco alguno de los campos.

Hice esto pero como no tengo idea de programar en javascript me salta error:

Código Javascript:

var i;
   for (i=1;i<=30;i++) {
   if (evaluacion.i.selectedIndex==0) {  
    alert("No puedes dejar en blanco ningún item. Por favor revisa el formulario y rellena todos los campos");   
    evaluacion.id.focus(); return;
    }
    }
 

El id de los campos va del 1 al 30.

El código completo sería este:

Código Javascript:

<SCRIPT language="javascript">   
  
// Creamos la función que valida los campos de formulario   
function validarFormulario() {  
 
 
// Validamos el campo Evaluado comprobando que se ha seleccionado una opción diferente a la index0   
    if (evaluacion.nombre.selectedIndex==0) {  
    alert("Debes introducir obligatoriamente el Nombre de la persona evaluada");   
    evaluacion.nombre.focus(); return;   
    }    
    
// Validamos el campo Grado comprobando que se haya seleccionado alguna opción    
   myOption2 = -1;
   for (u=evaluacion.grado.length-1; u > -1; u--) {
   if (evaluacion.grado[u].checked) {
   myOption2 = u; u = -1;
   }
   }
   if (myOption2 == -1) {
   alert("Debes introducir obligatoriamente tu grado de conocimiento de la persona evaluada");
   evaluacion.id.focus(); return;
   }
 
// Validamos todos los campos numéricos
   var i;
   for (i=1;i<=30;i++) {
   if (evaluacion.i.selectedIndex==0) {  
    alert("No puedes dejar en blanco ningún item. Por favor revisa el formulario y rellena todos los campos");   
    evaluacion.id.focus(); return;
    }
    }
 
   
// Si se han introducido todos los campos correctamente se envia el cuestionario  
// Aquí podemos enviar el formulario mediante document.formulario.submit()   
 
    else {  
    alert("ATENCIÓN: Se va a enviar el cuestionario.\nNo cierre la ventana del navegador hasta que no vea la pantalla de confirmación del envío.");
    evaluacion.submit();   
    }  
  
}// Fin de la función validarFormulario()
 

Lo curioso de esto es que al cargar la página aparece el interrogante y me dice que hay un error en la línea 38, donde tengo el else ?¿?¿

¿Alguien que pueda orientarme?
Gracias
JC

[quimfv]

No es tan curioso, else debe ser parte de un condicional iniciado por if, antes del else tienes un for.

Quita las lineas 38 y 41 y te funcionará.

Cundo se cumple los if anteriores terminas con return por lo que no llegará al alert final ni a mandar el formulario...

[majete]

Vale... ya no me aparece el error al cargar la página, pero sí al intentar enviar el formulario... me dice:

Detalles de error de página web

Mensaje: 'evaluacion.i.selectedIndex' es nulo o no es un objeto
Línea: 28

¿Tendrá algo que ver que los campos se llamen "1", "2", etc..? No sé de todas formas si interpreta bien el código, porque mi intención es que valide "evaluación.1.selectedIndex" al igual como si pusiera el nombre del campo tal que así "evaluacion.nombrecampo.selectedIndex".

Gracias

[quimfv]

Cuando te he contestado antes iva a decirte que no usarás identificadores numericos, pero al ver que te marcaba el error en el esle he corregido....

Usa

sel1
sel2
...
sel30

y usa document.getElementById("sel"+i).selectedIndex en el condicional, en lugar de evaluacion.i.selectedIndex

[majete]

Bingo!
Me encanta este foro... soluciones rápidas y concretas...

Muchas gracias quimfv por tu ayuda
Un saludo!
JC