Как поставить задержку в цикле for?

Question

[smarisov666]

function change(params) {
            vk.call('groups.edit', {
                group_id: 196543401,
                screen_name: params
            }).then((response) => {
                console.log(response)
                console.log(response.getFullResponse())
            }).catch((error) => {
                console.log(error);
            });
        }
        var array = fs.readFileSync('file.txt').toString().split("\n");
            for(i in array) {
                console.log(array[i]);
                setTimeout(function() {
                    change(array[i])
                }, 2000);
            }

Имею такой код, что только уже не перепробовал, суть проблемы, когда ставлю интервал, то console log выполняется спокойно, а потом просто флудит функцией settimeout уже с неактуальной информацией
мне нужно, чтобы при каждом новом значении i с задержкой в 5 секунд выполнялась функция change, как решить?

Comments

[Александр Шипин]

Ну так логика у вас идёт такая, что сначала читается файл, потом запускается цикл for по контенту файлов, а только потом запускается выполнение функции change с интервалом в 2 секунды для каждой строки текста.
Вы пробовали обернуть ваш fs.readFileSync в setTimeout, что раз в n-секунд выполнялась логика чтения?

setTimeout( function() {
   let array = fs.readFileSync('file.txt').toString().split("\n");
   array.forEach(string => {
     change(string)
   })
}, 5000)

Answer 1

[GrayHorse]

В очередной раз:

function sleep(ms) {
    return new Promise(resolve => setTimeout(resolve, ms));
}

for (let i = 0; i < 10; i++) {    
    console.log(i);
    await sleep(500);
}

---

async function change(params) {
    try {
        const response = await vk.call("groups.edit", {
            group_id: 196543401,
            screen_name: params
        });
        console.log(response);
        console.log(response.getFullResponse());
    } catch (error) {
        console.log(error);
    }
}
        
const array = fs.readFileSync("file.txt").toString().split("\n");
for (const item of array) {
    console.log(item);
    await change(item);
    await sleep(1000);
}

Comments

[GlazOtca]

await уже разрешили работать без обертки в async?

Answer 2

[black1277]

Вы умудрились наступить сразу на 2 грабли.
1 Замыкания
2 Асинхронный код в теле цикла

for (i in array){
    (function (m) {
        setTimeout(function () {
            change(array[m])
        }, 2000)
    })(i)
}

Код который я привел - разрулит замыкание, а в предыдущем ответе возьмите ф-ю для паузы и почитайте по ссылке про циклы.

Comments

[Квазиморд Сидорович]

Про асинхронность в циклах есть еще одна интересная статья Простейшее руководство по использованию Async/Awai...

[Николай]

а разве обычный let не решит проблему замыкания?

[black1277]

Николай, Решит, но не даст понимания проблемы автору вопроса.

Answer 3

[Crone1331]

Создай функцию, например timer

function timer(ms) {
    return new Promise(res => setTimeout(res, ms));
}

и вызывай её в асинхронной функции, где тебе нужна задержка
в твоем случае добавь async перед названием функции

async function change(params) {
...
await timer(1000);
}

подробнее можно прочитать тут

Comments

[smarisov666]

не работает, все так же флудит функциями, хотя ваш вариант самый полезный

function timer(ms) {
            return new Promise(res => setTimeout(res, ms));
        }

        async function change(params) {
            vk.call('groups.edit', {
                group_id: 196543401,
                screen_name: params
            }).then((response) => {
                console.log(response)
                console.log(response.getFullResponse())
            }).catch((error) => {
                console.log(error);
            });
            await timer(5000);
        }
        
        var array = fs.readFileSync('file.txt').toString().split("\n");
        for(i in array) {
            console.log(array[i]);
            change(array[i])
        }

Answer 4

[dollar]

Если i - это обычный индекс, то вместо 2000 просто напишите 2000*i

как-то так

for(i in array) {
	console.log(array[i]);
	let j = i;
	setTimeout(function() {
		change(array[j])
	}, 2000*i);
}

Хотя если массив слишком большой, то лучше вообще убрать цикл for и переписать его на setInterval(), например.

Comments

[GlazOtca]

Это будет работать неправильно!!! Функция в setTimeout будет вызвана всегда с одним и тем же элементом array[i], в котором i во всех вызовах будет равен последнему индексу в массиве.

[dollar]

GlazOtca, согласен. Исправим же это:

array = [1,2,3,4,5]
for(i in array) {
	console.log(array[i]);
	let j = i
	setTimeout(function() {
		console.log('inside timeout:', array[j]);
	}, 2000*i);
}

(адаптировал для быстрого копипаста в консоль для проверки)

[GlazOtca]

или так

array = [1,2,3,4,5]
for(let i in array) {
  console.log(array[i]);
  setTimeout(function() {
    console.log('inside timeout:', array[i]);
  }, 2000*i);
}

[dollar]

Да, верно.