Не срабатывает разобраться с setInterval

@Jasta · Регистрация: 20.02.2016

Студворк — интернет-сервис помощи студентам

Доброго всем времени суток. Мне нужно что бы функция в слайдере вызывающая следующую картинку, срабатывала через определённые промежутки времени и таким образом картинки менялись автоматически. Для этого я использую функцию setInterval, но проблема в том что она срабатывает всего один раз, причём сразу после загрузки. Подскажите что я делаю не так??? Заранее всем благодарен)

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
var slideIndex=1;
showSlides(slideIndex);
 
function plusSlides(n){
    showSlides(slideIndex += n);
}
 
function currentSlide(n){
    showSlides(slideIndex = n);
}
 
function showSlides(n){
    var i;
    var slides = document.getElementsByClassName("mySlides");
    var dots = document.getElementsByClassName("dot");
 
    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","");
    }
     slides[slideIndex-1].style.display = "block";
     dots[slideIndex-1].className+= " active";
  
}
 
   setInterval(plusSlides(slideIndex), 5000);

@Jewbacabra · 18.04.2017, 22:21

JavaScript
1
setInterval(plusSlides.bind(undefined, 1), 5000);

@Jasta · 19.04.2017, 00:21 **[ТС]**

Спасибо помогло) но у меня есть ещё один вопрос: как сделать что бы при срабатывании функции, таймер обнулялся, и до вызова функции вновь оставалось 10 секунда???
Попробовал добавить переменную time и уже в неё записать количество миллисекунд для отсчёта, а затем в главной функции, присваивать ей вновь значение 10000 миллисекунд. Но к сожалению это не работает, хотя и значение переменой после срабатывания функции меняется.

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
var slideIndex=1;
 var time = 10000;
showSlides(slideIndex);
 
function plusSlides(n){
    showSlides(slideIndex += n);
}
 
function currentSlide(n){
    showSlides(slideIndex = n);
}
 
function showSlides(n){
    var i;
    var slides = document.getElementsByClassName("mySlides");
    var dots = document.getElementsByClassName("dot");
 
    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","");
    }
    
     slides[slideIndex-1].style.display = "block";
     dots[slideIndex-1].className+= " active";
    time = 10000;
  
}
 
    
setInterval(plusSlides.bind(undefined, 1), time);

@Jewbacabra · 19.04.2017, 09:06

Сообщение от Jasta

как сделать что бы при срабатывании функции, таймер обнулялся, и до вызова функции вновь оставалось 10 секунда???

Сейчас так и должно быть
Можно через setTimeout, используя такую конструкцию

JavaScript
1
2
3
4
5
function foo() {
    // ...
    setTimeout(foo, 10000);
}
foo();

@Jasta · 20.04.2017, 07:29 **[ТС]**

Такая конструкция не очень работает в моем случае, всё дело в том что функция так же вызывается кнопками переключения между слайдами, и если нажать на кнопку то вызывается функция и setTimeout срабатывает, в результате слайды начинают сменять друг друга через разные интервалы, практически хаотично.
Видимо я в прошлый раз плохо объяснил в чем проблема, а она вот в чем: если кликнуть по кнопке следующий слайд скажем на 8 секунде после загрузки страницы, то слайд сменится, но уже через 2 секунды сработает, setInterval и он сменится снова. А мне нужно что бы каждый раз после срабатывания главной функции showSlides счётчик обнулялся, и от появления нового слайда, до вызова следующего вновь оставалось 10 секунд

@Balanaar · 20.04.2017, 09:25

Сообщение от Jasta

мне нужно что бы каждый раз после срабатывания главной функции showSlides счётчик обнулялся

Используйте clearInterval()

@Jasta · 21.04.2017, 07:11 **[ТС]**

Я знаю про эти функции, но не как не могу грамотно их встроить в работу. Пробовал делать вот так:

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
var slideIndex=1;
 
showSlides(slideIndex);
 
function plusSlides(n){
    showSlides(slideIndex += n);
   
}
 
function currentSlide(n){
    showSlides(slideIndex = n);
}
 
function showSlides(n){
    var i;
    var slides = document.getElementsByClassName("mySlides");
    var dots = document.getElementsByClassName("dot");
 
    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","");
    }
    
     slides[slideIndex-1].style.display = "block";
     dots[slideIndex-1].className+= " active";
    
    
    clearTimeout(timeId);
    
timeId();
}
 
function timeId() {
 setTimeout(plusSlides.bind(undefined, 1), 10000);
}

Но это не помогает, таймер работает всё так же(((

@Balanaar · 21.04.2017, 09:11

Функция setTimeout() возвращает id таймера, который и нужно будет поместить в функцию clearTimeout() для выключения таймера. В общем случае, это выглядит так:

JavaScript
1
2
var timerId = setTimeout(someFunction, duration);
clearTimeout(timerId);

@Jasta · 23.04.2017, 15:44 **[ТС]**

Да знаю я как эти функции работают, проблема в том что я не знаю как их встроить в код что бы обнулять счётчик((

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
Пробовал вот так:
var slideIndex=1;
 
showSlides(slideIndex);
 
function plusSlides(n){
    showSlides(slideIndex += n);
   
}
 
function currentSlide(n){
    showSlides(slideIndex = n);
}
 
function showSlides(n){
    var i;
    var slides = document.getElementsByClassName("mySlides");
    var dots = document.getElementsByClassName("dot");
 
    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","");
    }
    
     slides[slideIndex-1].style.display = "block";
     dots[slideIndex-1].className+= " active";
clearInterval(timeId);
var timeId = setInterval(plusSlides.bind(undefined, 1), 10000);
}

Но опять же выходит полная фигня, при многократном вызове функции showSlides кликом на кнопки, setInterval так же многократно повторяется, в результате слайды начинают хаотично сменять друг друга(( К тому же если встроить setInterval в саму функцию showSlides, перестаёт верно передаваться аргумент, в результате чего, на второй итерации смена картинок прекращается, и начинает моргать одна вторая картинка((

@Balanaar · 23.04.2017, 16:07

На 35й строке переменная timeId ещё не определена.

@Jasta · 23.04.2017, 16:13 **[ТС]**

Да в курсе я, в курсе, идея была в том что бы отменить счётчик, после первого срабатывания, а потом сразу вызвать его ещё раз, но..... не судьба Ваш ответ как всегда "бесценен")

@Balanaar · 23.04.2017, 16:18

Сообщение от Jasta

Да в курсе я, в курсе

А зачем тогда так пишете? На 35й строке интерпретатор неизбежно столкнётся с неопределённой переменной, выдаст ошибку "Uncaught ReferenceError: timeId is not defined" и остановит выполнение сценария. Всё, что ниже 35й строки просто не отработает.

@Jasta · 23.04.2017, 21:48 **[ТС]**

Ok не работает, но почему же тогда setInterval, продолжает срабатывать и менять слайды???? Чудо не иначе

Добавлено через 5 часов 4 минуты
Попробовал вот такую конструкцию, создать clearInterval, потом очистить его, и снова создать, но это так же не помогает((

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
var slideIndex=1;
 
showSlides(slideIndex);
 
function plusSlides(n){
    showSlides(slideIndex += n);
   
}
 
function currentSlide(n){
    showSlides(slideIndex = n);
}
 
function showSlides(n){
    var i;
    var slides = document.getElementsByClassName("mySlides");
    var dots = document.getElementsByClassName("dot");
 
    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","");
    }
    
     slides[slideIndex-1].style.display = "block";
     dots[slideIndex-1].className+= " active";
    
    var timeId = setInterval(plusSlides.bind(undefined, 1), 10000);
clearInterval(timeId);
var timeId = setInterval(plusSlides.bind(undefined, 1), 10000);
 
}

Новые блоги и статьи Все статьи Все блоги /
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 19.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .

@Jasta 1516 / 573 / 137 Регистрация: 20.02.2016 Сообщений: 3,389
	20.04.2017, 07:29 [ТС]
	Такая конструкция не очень работает в моем случае, всё дело в том что функция так же вызывается кнопками переключения между слайдами, и если нажать на кнопку то вызывается функция и setTimeout срабатывает, в результате слайды начинают сменять друг друга через разные интервалы, практически хаотично. Видимо я в прошлый раз плохо объяснил в чем проблема, а она вот в чем: если кликнуть по кнопке следующий слайд скажем на 8 секунде после загрузки страницы, то слайд сменится, но уже через 2 секунды сработает, setInterval и он сменится снова. А мне нужно что бы каждый раз после срабатывания главной функции showSlides счётчик обнулялся, и от появления нового слайда, до вызова следующего вновь оставалось 10 секунд 0

@Balanaar 2463 / 1769 / 625 Регистрация: 11.07.2016 Сообщений: 4,067
	23.04.2017, 16:07
	На 35й строке переменная timeId ещё не определена. 0

@Jasta 1516 / 573 / 137 Регистрация: 20.02.2016 Сообщений: 3,389
	23.04.2017, 16:13 [ТС]
	Да в курсе я, в курсе, идея была в том что бы отменить счётчик, после первого срабатывания, а потом сразу вызвать его ещё раз, но..... не судьба Ваш ответ как всегда "бесценен") 0