javascript - jQuery 包含大量 'if' 语句。必须有一个更简洁的方法来做到这一点？

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 );
}