function comenzar(){
var caja=document.getElementById("caja");
var numero=document.getElementById("numero");
numero.addEventListener("change",background,false);
setInterval(background(),50);
}
function background(){
valores=[1,3,5];
for (i=0; i<valores.length;i++){
if (numero.value == valores[i]){
caja.style.background="red";
}
else{caja.style.background="#000";}
}
}
window.addEventListener("load",comenzar,false);
1 respuesta
Independientemente de los otros errores que comentan, el principal error por el cual crees que solo se ejecuta el ultimo if se debe a que estas comparando valores diferentes "numero.value"
es string, valors[i]
es Integer y ademas que no sales del for con break;
una vez que la condiciones es cierta para lograr el efecto que deseas.
Haz la siguiente corrección y te funcionara:
function comenzar(){
var caja=document.getElementById("caja");
var numero=document.getElementById("numero");
numero.addEventListener("change",background,false);
setInterval(background(),50);
}
function background(){
valores=[1,3,5];
for (i=0; i<valores.length;i++){
if (parseInt(numero.value) == valores[i]){
caja.style.background="red";
break;
}
else{caja.style.background="#000";}
}
}
window.addEventListener("load",comenzar,false);
<input id="caja">
<input id="numero">
var
o aún mejor,const
olet
al declarar una variable, aún ésta sea parte del control de un loop, automáticamente dicha variable se hace global? Empieza por erradicar esa muy mala práctica. 2. ¿Sabías quesetInterval
acepta como primer parámetro una función y que él se encargará de ejecutarla? Por ende, no ejecutes la funciónbackground
, solo pásala como tal. Si la ejecutas, solo se ejecutará una vez.