The following code snippet is supposed to increase the opacity of a certain image(its id is ‘img’) by 0.01 per every 10 milliseconds until its opacity is set to 1.
```var t;
var frontimage=document.getElementById('img');
frontimage.style.top=200+'px';
frontimage.style.opacity=0; //line x
frontimage.style.left=200+'px';
frontimage.style.width=500+'px';
frontimage.style.height=300+'px';
function right(){
if(frontimage.style.opacity>=0 && frontimage.style.opacity<1){ //condition
frontimage.style.opacity=frontimage.style.opacity+0.01; //first statement
console.log(frontimage.style.opacity);
}else{
window.clearInterval(t);
}
}
t= window.setInterval(right,10);```
I believe that the 1st statement after the if condition(lets just call it as the first statement for now) gets executed only once while the other statement in the same condition gets executed perfectly as long as the condition remains true. The first statement is supposed to increase the opacity of the image by 0.01. The reason why I say that the first statement gets executed only once is because the 2nd statement of the same condition, which is supposed to display the opacity of the image on the console always displays 0.01(the opacity had been set to 0 before). Could someone please explain to me as to why the first statement gets executed only once when it is supposed to get executed until the condition remains true?
FYI the following code snippet has been created by modifying the ‘line x’, ‘if condition’ and the ‘first statement’ of the above code snippet and is supposed to decrease the opacity of the same image by 0.01 per every 10 milliseconds. This code gets executed perfectly.
var frontimage=document.getElementById('img');
frontimage.style.top=200+'px';
frontimage.style.opacity=1;
frontimage.style.left=200+'px';
frontimage.style.width=500+'px';
frontimage.style.height=300+'px';
function right(){
if(frontimage.style.opacity>0 && frontimage.style.opacity<=1){
frontimage.style.opacity=frontimage.style.opacity-0.01;
console.log(frontimage.style.opacity);
}else{
window.clearInterval(t);
}
}
t= window.setInterval(right,10);
>Solution :
- SO in your code every time opacity was getting set to 0 that’s your code was keep going in an infinite loop.
- I have used one temporary
countervariable and compared it and increased it by0.01and updated it and in the end assigned its value toimage.style.opacity. - Moreover, you were comparing
stringvalue withnumber. as it was not with the type check but you should also keep that thing in mind.
let timeIntervalId = null;
const image = document.querySelector("#img");
image.style.top = 200 + 'px';
image.style.left = 200 + 'px';
image.style.width = 500 + 'px';
image.style.height = 300 + 'px';
image.style.opacity = 0;
var counter = 0;
function right() {
if (counter >= 0 && counter <= 1) { //condition
counter = counter + 0.01;
image.style.opacity = counter;
//first statement
console.log(image.style.opacity);
} else {
window.clearInterval(timeIntervalId);
}
}
timeIntervalId = window.setInterval(right, 10);
<img id="img" src="https://cdn.pixabay.com/photo/2022/06/23/09/46/mountain-7279430__480.jpg" />