Как остановить setTimeout?

Question

[Артур Кудашев]

Здравствуйте, делаю автоскролл для dnd. Когда перетаскиваемый элемент доходит до края экрана, начинается автоскролл.

Вот код:

if (item.getBoundingClientRect().right + 20 > document.body.getBoundingClientRect().width) {
    this.scroll(1, 'horizontal');
} else {
    this.scroll(0, 'horizontal', true);
}
     
scroll = (step, direction, stop = false) => {
    if (stop) clearTimeout(this.scrollId); // Не удаляет, т.к scrollId не совпадает
		
    if (direction === 'horizontal') { 
        document.documentElement.scrollLeft += step;
        this.scrollId = setTimeout(() => this.scroll(step, direction), 100); 
    }
};

Проблема в том, что scrollId при проверке stop не совпадает со scrollId, который я задаю.
P.S событие mousemove

Answer 1

[Антон Спирин]

У вас метод scroll вызывается как снаружи, так и изнутри. Это значит, что scroll может быть вызван несколько раз с direction: 'horizontal' и scroll: undefined и таймаутов у вас будет несколько. В scrollId будет записан последний идентификатор.

Answer 2

[Шохрух Шаймардонов]

Почитайте про области видимости и про this в js.
У вас this.scrollId не туда обращается

Лучше id присвойте к переменной и через него удаляйте таймаут

Comments

[Артур Кудашев]

Шохрух Шаймардонов, область видимости я знаю. Тут проблема не в этом. This ссылается на мой класс DragManager в обоих случаях я проверял.

[Шохрух Шаймардонов]

Артур Кудашев, так выложите код всего класса (или хотя бы подробней с частью, связанной с scrollId), а также айдишники которые у вас получаются (при установке таймаута, и при попытке его удаления)