Вызов метода класса с помощью setInterval

Question

Здравствуйте, вопрос у меня такой, ну изменять позицию экземпляра класса, есть метод Move, который изменяет позицию, хочу вызвать его с определённой задержкой в цикле

while (car1.currentPosition < screen.width - 10){
 div.style.left = car1.currentPosition + 'px';
 setInterval(car1.move(), 1000);
}

Car1 тестовый экземпляр, setInterval(car1.move(), 1000) - Данная конструкция не работает, вылетает всё разом без задержки. Вот и вопрос как изменить вызов setIntervalдля того чтобы всё работало

К сведению: Потеря контекста вызова, Как куда-нибудь передать функцию // почему событие вызывается сразу же? — Pavel Mayorov, Dec 04 '17 at 14:12
Возможный дубликат вопроса: Как куда-нибудь передать функцию // почему событие вызывается сразу же? — Pavel Mayorov, Dec 04 '17 at 14:12

score 1 · Answer 1 · answered Dec 05 '17 at 19:01

Проблему решила данная конструкция

function initialization(vechicle) {

setInterval(function () {
    vechicle.move();
    div.style.left = vechicle.currentPosition + 'px';
    if (vechicle.currentPosition === screen.width - 100) {
        vechicle.setCurrentPosition(0);
        div.style.left = '0px';
        count++;
    }
}, 5);

score -1 · Answer 2 · answered Dec 03 '17 at 22:45

-1

while (car1.currentPosition < screen.width - 10){
 div.style.left = car1.currentPosition + 'px';
 setInterval(function(){car1.move()}, 1000);
}

answered Dec 03 '17 at 22:45

L11VADIS

608

пожалуйста, постарайтесь оставлять чуть более развёрнутые ответы. дополнить ответ можно, нажав [edit] – aleksandr barakin Dec 04 '17 at 00:16

score -1 · Answer 3 · answered Dec 04 '17 at 13:51

Вам нужно прочитать про стек трейс и понять как работает javascript внутри движка. Метод while синхронный и будет выполняться пока не завершится в то время как setInterval асинхронный - он упадет в web APIs и будет ждать пока event loop его не вызовет (иными словами пока не пройдет весь стек трейс синхронных функций). Попробуйте использовать самовызывающиеся setTimeout (что-то типа этого):

setTimeout(function timeout() {
        if (car1.currentPosition < screen.width - 10) {
            div.style.left = car1.currentPosition + 'px';
            setTimeout(timeout, 1000);
        }
    }, 1000)

emvcoder · Answer 4 · 2017-12-03T18:39:15.613

-2

Необходимо убрать скобки. Вот так:

while (car1.currentPosition < screen.width - 10){
 div.style.left = car1.currentPosition + 'px';
 setInterval(car1.move, 1000);
}

UPDATE:

Тогда в вышем случае лучше так:

while (car1.currentPosition < screen.width - 10){
  div.style.left = car1.currentPosition + 'px';
  setTimeout(function call() {
    car1.move();
    setTimeout(call, 1000);
  }, 1000);
}

Данный код ждет, пока выполнется перемещение элемента.

edited Dec 03 '17 at 18:39

answered Dec 03 '17 at 18:26

emvcoder

31

Тоже не работает, страница вообще зависла – Kirill Dec 03 '17 at 18:29
Тот же эффект, зависает всё – Kirill Dec 03 '17 at 19:00
Опишите проблему подробнее. – emvcoder Dec 03 '17 at 19:13

Вызов метода класса с помощью setInterval

4 Answers4