Comprobacion de valores de campos select

[3nr1c]

Hola a todos, tengo un formulario parecido a este (excluyo código irrelevante):

Código HTML:

<form name="entrcadetes" action="set_entrenos.php" method="post" />
 
<select name="cadetesdia1" onChange="comprob()"/>
<option value="1">Lunes</option>
<option value="2">Martes</option>
<option value="3">Miércoles</option>
</select> <br/>
 
<select name="cadetesdia2" onChange="comprob()"/>
<option value="2">Martes</option>
<option value="3">Miércoles</option>
<option value="4">Jueves</option>
</select> <br/>
 
<select name="cadetesdia3" onChange="comprob()"/>
<option value="3">Miércoles</option>
<option value="4">Jueves</option>
<option value="5">Viernes</option>
</select> <br/>
 
<select name="cadetesdia4" onChange="comprob()"/>
<option value="4">Jueves</option>
<option value="5">Viernes</option>
<option value="6">Sábado</option>
</select> <br/>
 
<select name="cadetesdia5" onChange="comprob()"/>
<option value="5">Viernes</option>
<option value="6">Sábado</option>
<option value="7">Domingo</option>
</select> <br/>
 
<input type="submit" value="Actualiza!" name="enviar"/>
</form>
 

La cuestión es, que si os fijáis los select tienen varias posibilidades de coincidir en valores, cosa que quiero evitar. Es por esto que tengo este código js:

Código Javascript:

function comprob(){
    for(i=1;i<5;i++){
        ic = document.entrcadetes.cadetesdia+i.value
        for(e=1;e<5;e++){
            ec = document.entrcadetes.cadetesdia+e.value
            if(ic==ec){
                document.entrcadetes.enviar.style.disabled = "true";
            }
        }
    }
}
 

Aunque si cambio el valor de cualquier select, no hace nada de esto. Voy bien encaminado o me estoy dejando algo?

Gracias por adelantado

[jarios]

Hola soy nuevo en el foro me voy a estrenar dandote un pequeño empujon si puede ser.

cosas que veo en el javascript:

Código:

      function comprob(){
          for(i=1;i<5;i++){
              ic = document.entrcadetes["cadetesdia"+i].value;
              for(e=1;e<5;e++){
                  ec = document.entrcadetes["cadetesdia"+e].value;
                  if(ic==ec){
			document.entrcadetes.enviar.disabled = true;
                  }
              }
          }
      }

He modificado la forma que accedes a el valor value ya que no funciona del otro modo (probado solo en firefox imagino que funciona en los demas tambien).

la otra cosa es que disable debe ser un atributo de input y no un style de css. asi que como aparece arriba es la forma de modificar el disable. para terminar true no va entre comillas que no es un string sino un boolean.

aun con todo veo que tendras que agregar algo al "if" para que no compruebe el mismo elemento y te de como resultado false. ademas de que si ya hay dos campos que den false desde el principio ya hay que salir del todo por que no conviene comprobar mas por que ya tiene que quedar "disable"

espero haberme explicado.

saludos

[jarios]

Bueno aqui te dejo para que te hagas mas o menos a la idea lo que yo considero mejor. he hecho una busqueda secuencial sin repetir los que ya se habian hecho que es tonteria. si no te aclaras con el codigo preguntame

Código Javascript:

function comprob(){
i=1;
semaforo=true;
while(i<5 && semaforo){
ic = document.entrcadetes["cadetesdia"+i].value;
j=i;
while(j<5){
j++;
ec = document.entrcadetes["cadetesdia"+j].value;
if(ic == ec){
document.entrcadetes.enviar.disabled = true;
semaforo=false;
break;
}
j++;
}
i++;
}
if(semaforo){
/*No hay iguales"*/
document.entrcadetes.enviar.disabled = false;
}}
 

no se si seria correcto que en el html colocaras la etiqueta desactivada desde el principio

Código HTML:

<input type="submit" value="Actualiza!" name="enviar" disable="true"/>
 

saludos

[3nr1c]

Muchas gracias a los dos, ahora entiendo donde estaba el problema , me ha servido muchísimo.

Jarios, el atributo disable desde el principio podría ir bien, pero como la asignación sin coincidencias se hace a través de una base de datos, no es necesario (ya se nota que es un juego, verdad? )

Saludos