javascript не работает jquery

Question

Не знаете, как это назвать, но проблема в том, что ... У меня есть несколько строк jQuery, которые работают нормально, они активируются сначала, когда страница загружается, а затем щелчком мыши внутри div.

Я сейчас пытаюсь иметь один и тот же код работает на временных интервалах, но он не работает вообще ни выдает какую-либо ошибку ....

Любая идея? сайт построен на WordPress и сценарий работает над картиной «слайд-шоу» вы можете проверить на http://demo.lavenderwp.com/nowland/

код, который я использовал для интервалов времени:

setInterval("myFunc()",100); 

function myFunc() 
{ 
$('#slider, .leftNav, .rightNav, .bottomNav').click(function() 
     { 
      $(".contentHolderUnit").filter(function(i, el) 
      { 
        return el.style.zIndex <= 100; 
       }).find('img').wrap('<div class="blue" />') .css({opacity: 0.75, border: "0px white solid"}); 

      $(".contentHolderUnit").filter(function(i, el) 
      { 
        return el.style.zIndex <= 100; 
       }).find('div.car').css({display: "none"}); 

      $(".contentHolderUnit").filter(function(i, el) 
      { 
        return el.style.zIndex >= 100; 
       }).find('img').css("background-color","") .css({opacity: 1, border: "15px white solid"}); 

      $(".contentHolderUnit").filter(function(i, el) 
      { 
        return el.style.zIndex >= 100; 
       }).find('div.car').css({display: "block"}); 
     }); 
} 

Answer 1

Поместите свой setInterval (и любой jQuery) в пределах $(function(){ });, поэтому он запускается только при загрузке документа. И как указывает @tymeJV, вы должны передать функцию без скобок.

Также вы понимаете, что интервал времени равен миллисекундам? Ваш myFunc работает 10 раз в секунду. Вам также не нужно повторно связывать событие click. Просто сделайте это один раз, и все будет в порядке.

Answer 2

Удалите кавычки вокруг вашей функции, а также ():

setInterval(myFunc , 100); 

Кроме того, не забудьте включить эту функцию после того, как DOM готова.

Answer 3

Я считаю, что вы можете обернуть свой код в $(document).ready()

Вы можете прочитать об этом здесь: http://learn.jquery.com/using-jquery-core/document-ready/

Хотелось бы надеяться, что помогает.

Answer 4

Ответ: не делайте этого!

Почему вы неоднократно присоединяете слушателей событий к событию десять раз в секунду?!? Это быстро исчерпает память, если jQuery достаточно умен, чтобы прикрепить их только один раз.

Вам нужно только один раз прикрепить события кликов. Если причина, по которой вы делаете это, заключается в том, что элементы DOM, с которыми вы подключаете слушателей, заменяются во время слайд-шоу, а затем используйте делегирование событий.

Или если проблема в том, что прослушиватели событий не подключаются при первой загрузке страницы, то, как и другие, используйте $(document).ready(myFunc); или $(myFunc);.

Показать больше вашего кода - в частности, соответствующего HTML - и кто-то может дать более конкретные предложения.

Подробнее о коде кода myInit, код в этой функции имеет немного повторения, которое было бы лучше всего удалено, а также есть ошибка «один за другим». Что произойдет, если один из ваших элементов имеет z-index ровно 100? Он проходит все.filter() тестов. А также отступы немного запутывают.

Вот пример того, как можно очистить этот код:

function myFunc() { 
    $('#slider, .leftNav, .rightNav, .bottomNav').click(function() { 

     var $all = $(".contentHolderUnit"); 
     var $lower = $all.filter(function(i, el) { 
      return el.style.zIndex <= 100; // or should it be < 100 ? 
     }); 
     var $upper = $all.not($lower); 

     $lower.find('img').wrap('<div class="blue" />').css({ 
      opacity: 0.75, 
      border: "0px white solid" 
     }); 

     $lower.find('div.car').css({ 
      display: "none" 
     }); 

     $upper.find('img').css({ 
      backgroundColor: "", 
      opacity: 1, 
      border: "15px white solid" 
     }); 

     $upper.find('div.car').css({ 
      display: "block" 
     }); 
    }); 
} 

Answer 5

Удивительные ребята! Получил это, спасибо большое!

Окончательный код:

jQuery(function(){ 
setInterval(myFunc,100); 
}); 

function myFunc() 
{ 
jQuery(".contentHolderUnit").filter(function(i, el) 
      { 
        return el.style.zIndex <= 100; 
       }).find('img').wrap('<div class="blue" />') .css({opacity: 0.75, border: "0px white solid"}); 

      jQuery(".contentHolderUnit").filter(function(i, el) 
      { 
        return el.style.zIndex <= 100; 
       }).find('div.car').css({display: "none"}); 

      jQuery(".contentHolderUnit").filter(function(i, el) 
      { 
        return el.style.zIndex >= 100; 
       }).find('img').css("background-color","") .css({opacity: 1, border: "15px white solid"}); 

      jQuery(".contentHolderUnit").filter(function(i, el) 
      { 
        return el.style.zIndex >= 100; 
       }).find('div.car').css({display: "block"}); 
     }