Wyciek pamięci (jQuery), jedna zakładka zajmuje 1GB pamięci w chrome Opcje

[kamloo21]

Hej
Mam problem z jquery , napisałem kilka animacji , wszystko ładnie działa lecz po pewnym czasie (jakieś 10min) drastycznie zwiększa się użycie pamięci które dochodzi do 1GB na 1 zakładke w Chrome.
Domyślam się że animacja jest źle zloopowana ale wydaję mi sie że jest OK. Próbowałem kilku różnych rzeczy

http://wytestuj.cba.pl/tyla/send.php Chodzi mi o tą animacje konkretnie (na reszcie strony jest inna więc nie zmieniajcie zakładki)

[HTML] pobierz, plaintext 
$(document).ready(
			function animuj(){
				$("#loga").animate(
					{ backgroundPosition: "+=2000" }, 19000  //animacja paska na dole
				).animate(
					{ backgroundPosition: "-=2000" }, 19000  //animacja paska na dole
				);
 
			$("p.active").prev("p").css("border-bottom-color","#363732"); //zmiana koloru ramki nad .active
 
			$("#menu p").hover( 									//animacja menu
				function(){
					$(this).prev().css("border-bottom-color","#363732");
					$(this).addClass("hover");
				},
				function(){
					if(!$(this).hasClass('active'))
						$(this).prev().css("border-bottom","#666 dotted 2px");
 
					$(this).removeClass("hover");
				}
			);
 
			setInterval(animuj,40000);
			}
		);
[HTML] pobierz, plaintext 

[kamloo21]

Poczytałem troche o setInterval i setTimeout i zastosowałem sie do twoich rad.
Sama zamiana z setInterval na setTimeout rozwiązała problem i teraz strona sie nie zawiesza i nie pobiera 1GB pamięci , lecz nadal bierze ~140MB co jest dużą liczbą
Myśle że drugie rozwiązanie które mi podesłałeś będzie OK, jak przerobie to napisze co i jak. Dzięki ! (IMG:style_emoticons/default/smile.gif)


PS. W drugim rozwiązaniu jest var jLoga;
Czemu odrazu nie ustawiłeś wartości jLoga = $("#loga") ?

Ten post edytował kamloo21 2.05.2013, 18:02:51

[zegarek84]

PS. W drugim rozwiązaniu jest var jLoga;
Czemu odrazu nie ustawiłeś wartości jLoga = $("#loga") ?

tylko to zależy, kiedy zaczyna się o tym myśleć ;p - a to dosyć ważne, gdyż jest to dostępne dopiero po documentready ;p, czyli jak cały DOM załadowany, po to z tej metody kożystałeś... ;p... trochę inaczej zapisane a to samo znaczące ;p - po prostu wcześniej tego elementu anie znajdziesz...

[JAVASCRIPT] pobierz, plaintext 
(function(){
  var jLoga;
  function animuj(){
    jLoga.animate(
    {
      backgroundPosition: "+=2000"
    }, 19000  //animacja paska na dole
    ).animate(
    {
      backgroundPosition: "-=2000"
    }, 19000  //animacja paska na dole
    );
 
    $("p.active").prev("p").css("border-bottom-color","#363732"); //zmiana koloru ramki nad .active
  }
  function start(){
    jLoga = $("#loga");
    $("#menu p").hover( 									//animacja menu
      function(){
        $(this).prev().css("border-bottom-color","#363732");
        $(this).addClass("hover");
      },
      function(){
        if(!$(this).hasClass('active'))
          $(this).prev().css("border-bottom","#666 dotted 2px");
 
        $(this).removeClass("hover");
      }
      );
    setInterval(animuj,40000);
    animuj();
  }
  $(document).ready(
    start
    );
})();
[JAVASCRIPT] pobierz, plaintext