javascript loop with delay code fix

Question

how to fix this code as per

How do I add a delay in a JavaScript loop?.

it still gets executed without delay .

const spkz = responsiveVoice.speak;
let azj = ['hi', 'hello', 'how are ya'];
var i = 1; //  set your counter to 1

function myLoop() { //  create a loop function

  azj.forEach((item, index) => {
    setTimeout(() => { //  call a 3s setTimeout when the loop is called
      alert(item); //  your code here
      i++; //  increment the counter
      if (i < index) { //  if the counter < 10, call the loop function
        myLoop(); //  ..  again which will trigger another 
      } //  ..  setTimeout()
    }, 10000)
  })

}

myLoop();

already using setTimeout() once the loop gets executed there is no delaying it. => prob. — Isquare, Oct 29 '16 at 13:01
each iteration starts a setTimeout immediately, so, you can't pause javascript like that — Jaromanda X, Oct 29 '16 at 13:02
this one says its possible . http://stackoverflow.com/questions/3583724/how-do-i-add-a-delay-in-a-javascript-loop — Isquare, Oct 29 '16 at 13:05
`this one says its possible` - there is no forEach in that answer — Jaromanda X, Oct 29 '16 at 13:13

score 1 · Accepted Answer · edited May 23 '17 at 12:19

1

You can't "pause" javascript like that. a setTimout is asynchronous, meaning it will not block synchronous code from running, so when you run any kind of "for" loop, it will call all the setTimeouts at once.

You can make a manual loop like this and delay it with recursion:

let azj = ['hi', 'hello', 'how are ya'];
var i = 0;

function myLoop() {
  setTimeout(function() {
    console.log(azj[i])
    i++
    
    if (i < azj.length) {
      myLoop()
    }
  }, 3000)
}

myLoop();

For more information, check out this answer.

edited May 23 '17 at 12:19

Community

1
1

answered Oct 29 '16 at 13:05

Sebastian Olsen

10,318
9
46
91

thanks this one seems to work – Isquare Oct 29 '16 at 13:14
@Isquare Because it's the exact same code as in the answer you've mentioned in your question... – Andreas Oct 29 '16 at 13:23
@Andreas Sorry! I should've linked to the answer instead. I didn't realize the answer in the linked question was identical to this. – Sebastian Olsen Oct 29 '16 at 13:25
i will flag it duplicate / delete the qn .. i shoulda took a closer look at it. – Isquare Oct 29 '16 at 13:30
@Isquare Do not delete the question. Flag it as a duplicate, that's what it's for. – Sebastian Olsen Oct 29 '16 at 13:32
ok . already flagged it as dup. – Isquare Oct 29 '16 at 13:33

score -1 · Answer 2 · answered Oct 29 '16 at 13:02

-1

You don't have to use forEach inside the loop function. Instead, you can access the array items using azj[i]

answered Oct 29 '16 at 13:02

Chris Lam

3,526
13
10

javascript loop with delay code fix

2 Answers2