Ошибка JavaScript: ожидаемый объект

Question

У меня есть страница HTML, содержащая div с текстом прокрутки справа налево; следующий JavaScript находится между тегами HEAD документа.

function scroll(oid, iid) { 
      this.oCont = document.getElementById(oid); 
      this.ele = document.getElementById(iid); 
      this.width = this.ele.clientWidth; 
      this.n = this.oCont.clientWidth; 
      this.move = function() { 
       this.ele.style.left=this.n + "px" 
       this.n-- 
       if(this.n<(-this.width)){this.n=this.oCont.clientWidth} 
      } 
     } 
     var vScroll 
     function setup() { 
      vScroll = new scroll("oScroll", "scroll"); 
      setInterval("vScroll.move()", 20); 
      }   
     onload = function(){ 
      setup() 
     } 

     $("scroll").hover(function() { 
      $("scroll").stop(true, false) 
     }, function(){ 
      scroll(); 
     }); 
     scroll(); 

Прокручиваемый текст отлично работает; однако я хотел, чтобы прокрутка останавливалась при наведении мыши. Хотя текст перестает прокручиваться, когда курсор мыши проходит над div, я получаю ошибку javascript «Object expected». Я новичок в javascript и понятия не имею, где я ошибаюсь.

Любая помощь была бы принята с благодарностью.

Answer 1

Ваша проблема с вашим setInterval. Вы передаете ему строку! Это позволяет использовать eval! Это означает, что код запускается в глобальной области, поэтому vScroll не существует.

Вместо передать функцию setInterval:

setInterval(function(){ 
    vScroll.move(); 
}, 20); 

функция передается setInterval вызывается с «контекстом» (this значение), установленным в null, так что вы не можете передать vScroll.move непосредственно setTimeout. Вы можете, однако. do:

setInterval(vScroll.move.bind(vScroll), 20); 

но это не работает во всех браузерах.

P.S. Передача строки в setInterval - это плохая практика, вы должны всегда передать функцию.