[jQuery] ClearInterval nie działa Opcje

[unnamedly]

[JAVASCRIPT] pobierz, plaintext 
	$(".bx-start").click(function() {
		if($(".bx-count").length)
		{
			clearInterval(interval);
			$(".bx-count").remove();
		}
		else
		{
			$(".bx-window").before("<span class='bx-count'>25</span>");
			var howmuch = 25;
			var interval = setInterval(function() {
				howmuch--;
				$(".bx-count").html(""+howmuch+"");
				if(howmuch == 0)
					howmuch = 25;
			}, 1000);
		}
	});
[JAVASCRIPT] pobierz, plaintext 

Czyli klikam w .bx-start i jeśli istnieje .bx-count, to ma czyścić mi timer i usuwać span`a. Jeśli nie istnieje, to tworzy .bx-count i ustawia timer, który odlicza od 25 do 0 i tak w kółko.

Problem w tym, że clearInterval() mi nie działa i nie czyści timeru (każdy kolejny timer przy kliknięciu w .bx-start o (chyba) id "interval" działa w tym samym czasie).

Any ideas?

[tolomei]

Witaj.

Nie jestem w stanie tego przetestować ale domyślam się że błąd tkwi w porozrzucanych deklaracjach var.
Spróbuj tego - jak coś to zobaczymy ponownie.

[JAVASCRIPT] pobierz, plaintext 
        $(".bx-start").click(function() {
		var howmuch, interval;
 
		if($(".bx-count").length)
		{
			clearInterval(interval);
			$(".bx-count").remove();
		}
		else
		{
			$(".bx-window").before("<span class='bx-count'>25</span>");
			howmuch = 25;
			interval = setInterval(function() {
				howmuch--;
				$(".bx-count").html(""+howmuch+"");
				if(howmuch == 0)
					howmuch = 25;
			}, 1000);
		}
	});
[JAVASCRIPT] pobierz, plaintext 

Ogólnie zalecanym zwyczajem przy pisaniu JS-a jest posiadanie jednej deklaracji var na początku każdej funkcji.
Kolejnym dobrym zwyczajem jest używanie setTimeout() w połączeniu z rekurencją.
Dobrze byłoby także przypisać $(".bx-count") do jakiejś zmiennej na początku funkcji. W tej chwili przy każdym wywołaniu skrypt przeszukuje całe drzewo DOM w poszukiwaniu tego elementu. W twoim skrypcie dzieje się to 3 razy, a potem co sekunde. Przy wspomnianej zmianie tylko raz i koniec.

Znowu gaduła mi się włączyła....

Pozdrawiam!

[PrinceOfPersia]

to co mu dałeś też raczej nie zadziała. Powinno się wywalić deklarację var... w ogóle poza funkcję click. Inaczej po prostu za każdym razem się będzie tworzyła nowa zmienna interval, a nie o to chodzi.
Chodzi o to, żeby zmienna była nieco "bardziej globalna".
o tak:

Kod

...
var interval;
$(".bx-start").click(function() {
....

EDIT:

Kolejnym dobrym zwyczajem jest używanie setTimeout() w połączeniu z rekurencją.

co masz na myśli? Rekurencja to ogólnie nie jest dobry zwyczaj.

Ten post edytował PrinceOfPersia 9.02.2013, 01:27:17