Slider javascript, clearTimeout nie działa :c

Question

var slideIndex = 1;
var myVar = 0;
var myVars = 0;
showDivs(slideIndex);
carousel();

function carousel() {
	carousell(1);
    myVar = setTimeout(carousel, 2000);
}

function carousell(n) {
  showDivs(slideIndex += n);
}

function plusDivs(n) {
  showDivs(slideIndex += n);
  clearTimeout(myVar);
  setTimeout(carousel, 2000);
}

function showDivs(n) {
  var i;
  var x = document.getElementsByClassName("slide");
  if (n > x.length) {slideIndex = 1}    
  if (n < 1) {slideIndex = x.length}
  for (i = 0; i < x.length; i++) {
     x[i].className = "slide";
  }
	  x[slideIndex-1].className += " fadeIn";
}

Po kliknięciu strzałki w lewo, lub w prawo, "naturalny czas przesuwania slajdu" nie zatrzymuje się.
Slider można póki co zobaczyc na http://aletu.ideya.pl/
Liczę, że ktoś ma pomysł co poprawić

Answer 1

jak napisales tak dziala :P nie zrobie roboty za Ciebie ale powiem gdzie jest blad

function carousel() {
    carousell(1);
    myVar = setTimeout(carousel, 2000);
}

w funkcji carousel wolasz funkcje carousel, zrobiles tak zwane wywolanie rekurencyjne

offtop
nadawaj zmiennym nazwy ktore cos mowia, jak wrocisz za 2 lata do kodu to bedziesz musial sie zastanawiac co miales na mysli :)

var myVar = 0;
var myVars = 0;

 

Comments

W zasadzie to rekurencji tu nie ma, funkcja `carousel()` nie wywołuje siebie, tylko inną funkcję. Różnicą jest... jedna litera:

carousel() {
carousell(1);

 

---

jest, popatrz na 1 i 3 linie w tym co podalem wczesniej