自定义滚动jquery只适用于手动编码

Question

我一直坚持这一段时间了。我正在尝试制作自己的Scrollspy。事情是，它只能硬编码的时候，如果我尝试做它通过变量它不断给我

$(document).ready(function() { 
    setInterval(check, 100); 
}); 

function isScrolledIntoView() 
{ 
    var docViewTop = $(window).scrollTop(); 
    var docViewBottom = docViewTop + $(window).height(); 
    var year = $('.year').eq(2).attr("class"); 
    var test = year.split(' '); 
    test = test[1]; 
    var elemTop = $(".item."+test).offset().top; 
    var elemBottom = elemTop + $(".item."+test).height(); 

    return ((elemBottom <= docViewBottom) && (elemTop >= docViewTop)); 
} 

var check = function(){ 
    var itemslength = $('.year').length; 
    for (var i = itemslength-1; i >= 0; i--) { 
     var year = $('.year').eq(i).attr("class"); 
     var yearsplitted = year.split(' '); 
     if(isScrolledIntoView()) 
     { 
      $(".year.2016").addClass("item-active"); 
     } 
     else 
     { 
      $(".year.2016").removeClass("item-active"); 
     } 
    } 
}; 

您可以在第一功能

var elemTop = $(".item."+test).offset().top; 

看到如果我试图做到这一点通过变量，它不起作用。

http://jsfiddle.net/DFh7z/68/

编辑：我使用了错误的循环。

Answer 1

在Javascript中，只有变量声明被提起，而不是初始化。因此，当您在document.ready函数中调用“setInterval”时，“check”函数变量将不确定。

如果你想使用一个变量的“检查”功能，试试你设定的时间间隔的呼叫转移到你的脚本的末尾，像这样：

function isScrolledIntoView() 
{ 
    var docViewTop = $(window).scrollTop(); 
    var docViewBottom = docViewTop + $(window).height(); 
    var year = $('.year').eq(2).attr("class"); 
    var test = year.split(' '); 
    test = test[1]; 
    var elemTop = $(".item."+test).offset().top; 
    var elemBottom = elemTop + $(".item."+test).height(); 

    return ((elemBottom <= docViewBottom) && (elemTop >= docViewTop)); 
} 

var check = function(){ 
    var itemslength = $('.year').length; 
    for (var i = itemslength-1; i >= 0; i--) { 
     var year = $('.year').eq(i).attr("class"); 
     var yearsplitted = year.split(' '); 
     if(isScrolledIntoView()) 
     { 
      $(".year.2016").addClass("item-active"); 
     } 
     else 
     { 
      $(".year.2016").removeClass("item-active"); 
     } 
    } 
}; 

$(document).ready(function() { 
    setInterval(check, 100); 
}); 

这应该让你的脚本射击，但我怀疑你需要更多的工作才能让你的滚动间谍扩展功能发挥作用。