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

jQuery有很多'if'语句。必须有一个更简洁的方法来做到这一点？

1 个答案: