JavaScript setTimeout Проблема с памятью - PullRequest
Новая
       15

JavaScript setTimeout Проблема с памятью

1 голос
/ 11 января 2012

Я отлаживаю код, который, как мне кажется, имеет некоторую утечку памяти. При запуске кода в Google Chrome через некоторое время происходит сбой браузера. Я твердо верю, что следующий код (прикрепление только шаблона, а не весь код для простоты) вызывает проблему. Когда я использовал вкладку «Временная шкала» инструментов разработчика Chrome и наблюдал за использованием памяти и большим скачком памяти в реальном приложении. Если вы заметили код, игра вызывает transitionTo, а transistionTo имеет функцию setTimeout, которая снова вызывает play. Я думаю, что память не освобождается из-за этого. Может ли кто-нибудь помочь мне решить эту проблему? 

    var htmlRotatorTimer = '';

    function play(src){
      // some code here
      transitionTo("gallery", "counter");
      // some code here
    }

    function transitionTo(gallery,index) {      
        // some code here      
       clearTimeout(htmlRotatorTimer);
       htmlRotatorTimer = setTimeout( function(){ play(); }, 1000 );    
       // some code here
     }

play();

Вот более длинная версия кода 

// utility for loading slides
        function transitionTo(gallery,index) {

            // preloader (optional) 
            var counterMin = 0;
            if (config.preloader) {
                counterMin = 1;
                if (gallery.length < 3) pause('');                  
            }

            var oldCounter = counter;
            if((counter >= gallery.length) || (index >= gallery.length)) { counter = counterMin; var e2b = true; }
            else if((counter < counterMin) || (index < 0)) { counter = gallery.length-1; var b2e = true; }
            else { counter = index; }



   // added attr speed in div's
//htmlRotatorTimer = ''; 
         var itemSpeed = Number($(gallery[counter]).attr("speed") != undefined ?   $(gallery[counter]).attr("speed") : config.speed);
            //htmlRotatorTimer = setInterval(function(){ play(); },itemSpeed);
            clearTimeout(htmlRotatorTimer);
            htmlRotatorTimer = setTimeout( function(){ play(); }, itemSpeed );

            var rmkName = $(gallery[counter].children).attr("id") != undefined ? 'RMK_' + $(gallery[counter].children).attr("id") : '';
            var isHtml5 = false;
            if (rmkName != '' && eval ('typeof '+ rmkName) == 'object') {                   
                rmkObj = eval(rmkName);     
                isHtml5 = rmkObj.rmkType == 'html5' ? true : false;
                //console.log('html5 is' + rmkObj.rmkType,'obj name' + rmkName, 'typeof:' +(typeof rmkObj));            
            }

    if (config.effect=='fade') {    
        $(gallery[counter])
        .clone()
        .appendTo($cont)
        .hide()                     
        .fadeIn(config.changeSpeed,function(){$('#showbanners.rmkbanner').css({ 'visibility': 'visible'});if($.browser.msie)this.style.removeAttribute('filter');});                        
                if($cont.children().length>1){
                    $cont.children().eq(0).css('position','absolute').fadeOut(config.changeSpeed,function(){$(this).remove();});
                };
            } else if (config.effect=='none') {
                $(gallery[counter])
                    .appendTo($cont);
                if($cont.children().length>1){
                    $cont.children().eq(0).remove();
                };
            };
            // update active class on slide link
            if(config.links){
                $('.'+uniqueClass+'-active').removeClass(uniqueClass+'-active jshowoff-active');
                $('.'+uniqueClass+'-slidelinks a').eq(counter).addClass(uniqueClass+'-active jshowoff-active');
            };

            // reset for html5 objects only
            if (isHtml5) {
                    rmkObj.preload = 'nopreload';
                    rmkObj.Reset();
            }
        };// end function transistionTo

        // is the rotator currently in 'play' mode
        function isPlaying(){
            return $('.'+uniqueClass+'-play').hasClass('jshowoff-paused') ? false : true;
        };

        // start slide rotation on specified interval
        function play(src) {
            if (!finalStop) {
                if (!isBusy()) {
                    counter++;
                    transitionTo(gallery, counter);
                    if (src == 'hover' || !isPlaying()) {
                        //htmlRotatorTimer = setInterval(function(){ play(); },config.speed);
                        clearTimeout(htmlRotatorTimer);
                        htmlRotatorTimer = setTimeout(function(){
                            play();
                        }, config.speed);
                    }
                    if (!isPlaying()) {
                        $('.' + uniqueClass + '-play').text(config.controlText.pause).removeClass('jshowoff-paused ' + uniqueClass + '-paused');
                    }
                };
            };
        };

1 Ответ

0 голосов
/ 11 января 2012

Я считаю, что ваш призыв к игре неверен внутри функции transitionTo. Вы не предоставляете аргумент. Кстати, почему вы также вызываете play () снаружи? Если возможно, пожалуйста, опубликуйте весь код. Я не уверен, почему вы передаете встречную строку для transitionTo. Одно из предложений - использовать метод setInterval в методе play, подобном этому 

function play(){
    //existing logic
    setInterval(function(){transitionTo("gallery","counter");},1000); 
}
...