setTimeout() не вызывает функцию после указанного времени

Question

У меня есть код в файле js.

function makeScrollable(wrapper, scrollable){ 
    ... 
    // Hide images until they are not loaded 
    scrollable.hide(); 

    var loading = $('<div class="loading">Loading...</div>').appendTo(wrapper); 

    // Set function that will check if all images are loaded 
    var interval = setInterval(function(){ 
     var images = scrollable.find('img'); 
     var completed = 0; 

     if (completed == images.length) { 
      clearInterval(interval); 

      // Timeout added to fix problem with Chrome 
      setTimeout(function(){ 
       loading.hide(); 
       .... 
      }, 1000); //end of setTimeout(func, delay) 

     } //end of if (completed == images.length) 

    }, 100); //end of var interval = setInterval(fn, 100) 

    ... 
} 

(function(){ 
    makeScrollable("div.sc_menu_wrapper", "div.sc_menu"); 
})(jQuery); 

Но моя функция таймаута не звонит. Я все еще получаю загрузку Div. Функция должна быть вызвана через 1 секунду, после чего скрыть контейнер загрузки и выполнить код. Но этого не происходит. Что я делаю неправильно? Thanks

Answer 1

Неверное положение кода.

var completed = 0; 

    if (completed == images.length) { // This will be true only when there are no images 
     clearInterval(interval); 


     setTimeout(function(){ 
      loading.hide(); 
      .... 
     }, 1000); 

    } 

поставить его вне

var completed = 0; 
var interval = setInterval(function(){ 
    var images = scrollable.find('img'); 


    if (completed == images.length) { 
     clearInterval(interval); 

     // Timeout added to fix problem with Chrome 
     setTimeout(function(){ 
      loading.hide(); 
      .... 
     }, 1000); //end of setTimeout(func, delay) 

    } //end of if (completed == images.length) 

}, 100); //end of var interval = setInterval(fn, 100) 

И надеюсь, что вы приращением completed где-то в коде