Cita:
Iniciado por wigglyUtrera 
He probado cambiando el nombre de id por id_general y sigue igual
Hombre, si ya tienes en el script una variable que se llama
id y otra
id_general, si cambias
id por
id_general nos quedamos con dos variables con el mismo nombre (
id_general)!
En este caso sería mejor que nos postearas un poquito de código HTML, lo suficiente como para que podamos testearlo en nuestras máquinas y te podamos asesorar un poquito mejor.. También estaría bien que nos explicases tus propósitos, quizá haya una manera más cómoda de hacerlo.
Viendo el código tal y como está, creo que te podrías ahorrar el segundo
for (ten en cuenta que primero recorres los selects, de 0 a 5 y estableces errores[0..5]. Luego recorres los errores (0 a 5), que tendrá los mismos índices que el primer
for.
Yo diría que te puedes ahorrar hasta las variables
errores,
id,
id_general y
tipos, sabiendo que es una cadena y el valor de la variable de iteración
i adjuntado al final. Y si el valor de
numErrores no lo vas a utilizar salvo para mirar
(numErrores == 0), ¿no sería más lógico que fuera un
boolean? Al fin y al cabo, lo que haces al final es equivalente a
return (numErrores == 0) 
Al final será un error tonto, como en todos los casos. No te preocupes. No se trata de ahorrar bytes; de hecho es mejor que sea más legible, y con más código, pero si tenemos que tener en la cabeza (o recordar más tarde) a qué hace referencia cada variable... el código es un poco más dificil de mantener. Es mejor escribirlo todo dentro del mismo
for e ir construyendo los
strings directamente en las condicionales en vez de almacenarlas en un
array.
Un saludo.