Почему функция выполняется, когда её не приглашают?

Question

[Uzumeti]

Привет!

Только-только начал вникать в JS. Вот пробую сделать слайдер картинок. Столкнулся с интересной проблемой (и уже не первый раз). Функция сама выполняется, даже если к ней нет обращения. Почему так происходит? Помогите пожалуйста понять.

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

Пробовал выносить обнуление интервала в отдельную функцию и включать её в теле основной функции - результат тот же.

Пример кода:

var MySlider = document.querySelectorAll('#slides .slide');
var currentPicture = 2;
var IntervalForChange = setInterval(nextSlide,3500);



function nextSlide() {
    goToSlide(currentPicture+1);
	clearInterval(IntervalForChange); // Вот этот интервал срабатывает сам и слайдер просто стоит на месте.
	
}

function backSlide() {
    goToSlide(currentPicture-1);
}

function goToSlide(n) {
 MySlider[currentPicture].className = 'slide';
 currentPicture = (n+MySlider.length)%MySlider.length;
 MySlider[currentPicture].className = 'slide showing';
}

Подобные проблемы наблюдаю не первый раз. Подскажите, что почитать, где поискать? Спасибо.

Comments

[Lynn «Кофеман»]

Казалось бы, а зачем тут вообще setInterval?

[Uzumeti]

Алексей Тен, setInterval запускает функцию смены слайда. Хм...гениально :) Вы правы, она сама себя запускает и сама себя тормозит. Спасибо. Но почему происходит это сразу при загрузке? Он же поидее один раз должен успеть щелкнуть...

А как тогда можно реализовать замирание при использовании кнопок? Ещё одну функцию на кнопку клеить?

[Stalker_RED]

Алексей Тен, вангую, что этот слайдер должен сам листаться.

[Uzumeti]

Всё вышло, спасибо :) Косяк понят. Стыдюсь)

Answer 1

[Stalker_RED]

В третьей строке вы какбы пишете:

движок, запускай функцию nextSlide каждые 3.5 секунды

Внутри функции nextSlide вы пишете:

движок, отмени тот приказ, который был в третьей строке

Поэтому nextSlide выполняется всего один раз.

Не совсем понятно, почему вы пишете "срабатывает сам", если сами же написали cleraInterval.
Или это диверсанты проникли к вам в код? :)

Comments

[Uzumeti]

Да, тупанул жёстко :) Это Вы правы...

А как бы это правильно обыграть?) Вынести в отдельную функцию и запускать две функции кнопкой?

[Uzumeti]

Всё вышло, спасибо!