Как остановить interval в переменной с динамическим именем?

Question

[paulvales]

Приветствую! Устал бороться, создаю интервалы
window[intername] = setInterval(function () {...})
intername- динамическое имя
Потом по кнопке нужно его завершить
clearInterval( window[intername]);
но этого не происходит
Это используется для аренды велосипедов, т.е. при начале создается таймер с обратным отсчетом и уведомлением об окончании , но если поездку завершили досрочно то уведомление все равно срабатывает т.к. работает таймер

Comments

[Kovalsky]

Если у вас интервал не останавливается, значит вы передаёте неверный ID, тут вариантов быть не может. Вероятно у вас просто несколько интервалов инициализировано. Вы конечно вольны делать как вам нравится, но всё же гораздо проще будет хранить время начала аренды в базе данных, а на клиенте только проверять его.

Answer 1

[twobomb]

Все должно работать. Единственное в чем может быть проблема, это имя itername в
clearInterval( window[intername]); не совпадает с именем которое было задано в window[intername] = setInterval(function () {...}) либо оно где-то заново инициализируется после clearInterval

Comments

[paulvales]

делал проверку, совпадает. делал console.log(window) после начала и завершения интервала, записывается но не удаляется(

[twobomb]

paulvales: А нельзя создать одну глобальную переменную с константным именем? Желательно бы еще увидеть участок кода с проблемой.

[paulvales]

twobomb: получается так: при начале аренды создается счетчик с именем 'time'+номер велосипеда, чтобы потом можно было завершить именно этот таймер

[twobomb]

paulvales: Не совсем понимаю что там за аренда такая и зачем там таймеры. Возможно вам стоит в целом пересмотреть алгоритм работы. Мне кажется там можно обойтись либо без таймеров setInterval или обойтись одним таймеров. Ну я понятия не имею как и что у вас там работает.

Answer 2

[Ernest Fayzullin]

что то похожее было у меня, может поможет:

(function(w) {
    var oldST = w.setTimeout;
    var oldSI = w.setInterval;
    var oldCI = w.clearInterval;
    var timers = [];
    w.timers = timers;
    w.setTimeout = function(fn, delay) {
        var id = oldST(function() {
            fn && fn();
            removeTimer(id);
        }, delay);
        
        return id;
    };
    w.setInterval = function(fn, delay) {
        var id = oldSI(fn, delay);
        timers.push(id);
        return id;
    };
    w.clearInterval = function(id) {
        oldCI(id);
        removeTimer(id);
    };
    w.clearTimeout = w.clearInterval;

    function removeTimer(id) {
        var index = timers.indexOf(id);
        if (index >= 0)
            timers.splice(index, 1);
    }
}(window));

после применения этого фикса setTimeout отработает и удалит остальные