Автослайдер на JavaScript. Где ошибка в слайдере?

Question

[Dmitry2307]

При нажатии кнопок слайдера (next / prev), функция setTimeout - начинает работать не корректно. Заданное время для функции сбивается. Сами слайды перелистываются через один.

var slideIndex = 0;
showSlides(slideIndex);
function plusSlide() {
    showSlides(slideIndex += 1);  
}
function minusSlide() {
    showSlides(slideIndex - 1); 
}
function currentSlide(n) {
    showSlides(slideIndex = n);
}

function showSlides(n) {
    var i;
    var slides = document.getElementsByClassName("mySlides");
    var dots = document.getElementsByClassName("dot");
    var time = 5000;

    if (n > slides.length) {
      slideIndex = 1
    }
    if (n < 1) {
        slideIndex = slides.length
    }
    for (i = 0; i < slides.length; i++) {
        slides[i].style.display = "none";
    }
    for (i = 0; i < dots.length; i++) {
        dots[i].className = dots[i].className.replace(" active_slide", "");
    }

    slideIndex++;
    if (slideIndex < 1) {slideIndex = slides.length};
    if (slideIndex > slides.length) {slideIndex = 1};

    slides[slideIndex - 1].style.display = "block";
    dots[slideIndex - 1].className += " active_slide", "";
    setTimeout(showSlides, time); // Change image every 5 seconds  
};

Answer 1

[Александр]

Доброго дня!

Бросилось в глаза вот это (в паре мест):

if (n > slides.length) {
slideIndex = 1
}
if (n < 1) {
slideIndex = slides.length
}

В JavaScript допустимые индексы массива - от 0 до someArray.length-1 включительно.

Может в этом дело?

Comments

[Dmitry2307]

комментирование не помогло ((( ни чего не изменилось https://ventkanal.store

[Александр]

Dmitry2307,
Каждый showSlide запускает свой таймер, который переключает слайды? Получается путаница.

Я бы смену текущего отображаемого слайда вынес в отдельную функцию: setCurrentSlide(n: number)
И соответственно в showSlide, plusSlide и minusSlide вызывал бы эту функцию с аргументами

++slideIndex,
++slideIndex,
––slideIndex.

Проверку на границы массива тоже надо вынести в setCurrentSlide

[Dmitry2307]

Александр, не очень понятно как вынести в отдельную функцию setCurrentSlide, не хватает опыта работы с JS

[Dmitry2307]

спасибо за помощь, вроде все заработало