带有大量“if”语句的 jQuery。必须有一个更整洁的方法来做到这一点？

Question

我编写了一个函数，可以在用户滚动后将我的页面滚动到最近的定义点（图层的顶部）。

我对自己非常满意，但我查看了代码，但它看起来很笨拙和糟糕。事不宜迟：

enter code here

sli1 = 0;
sli2 = slideheight;
sli3 = (slideheight * 2);
sli4 = (slideheight * 3);
sli5 = (slideheight * 4);
sli6 = (slideheight * 5);
sli7 = (slideheight * 6);
frac = (slideheight / 2);

function autoScroll() {
    cl = mywindow.scrollTop();
    if(cl < sli2-frac && cl < sli1+frac){goToByScroll(1);}
    if(cl > sli2-frac && cl < sli2+frac){goToByScroll(2);}
    if(cl > sli3-frac && cl < sli3+frac){goToByScroll(3);}
    if(cl > sli4-frac && cl < sli4+frac){goToByScroll(4);}
    if(cl > sli5-frac && cl < sli5+frac){goToByScroll(5);}
    if(cl > sli6-frac && cl < sli6+frac){goToByScroll(6);}
    if(cl > sli7-frac && cl < sli7+frac){goToByScroll(7);}
}

我觉得应该更像这样……

function aS() {
/*loop to create array like 
array = ('sl[0] = 0', sl[1] = sh.... there must be a way of automating this part*\
     cL = mW.scrollTop();
     var cH = (cl < sl[1]-f && cl < sl[0]+f)?'1':(cl > sl[1]-f && cl < sl[1]+f)?'2':
     (cl > sl[2]-f && cl < sl[2]+f)?'3':(cl > sl[3]-f && cl < sl[3]+f)?'4':
     (cl > sl[4]-f && cl < sl[4]+f)?'5':(cl > sl[5]-f && cl < sl[5]+f)?'6':
     (cl > sl[6]-f && cl < sl[6]+f)?'7'goToByScroll(cH);

但是现在看看这甚至看起来很丑，有没有办法用某种天才逻辑很好地做到这一点。我希望我在这里有意义，代码有效，但我认为它可以以更优雅的方式完成，但我是 jQuery 的新手，虽然我现在在这个问题上遇到了一堵砖墙，但它很有趣，请帮助我.

我的解决方案可能是 javascript，但 .js 文件开始：

 jQuery(document).ready(function ($) {

goToByScroll() 函数是 jQuery....

 function goToByScroll(dataslide) {
htmlbody.animate({
    scrollTop: $('.slide[data-slide="' + dataslide + '"]').offset().top
}, 2000, 'easeInOutQuint');
 }

Answer 1

可能是这样的

function autoScroll() {
    var cl = mywindow.scrollTop(),
        target = Math.floor((cl / slideheight)+0.4999) + 1 || 1;

    goToByScroll( target );
}