带有大量'if'语句的jQuery。必须有一个更好的方法来做到这一点？

我写了一个函数，它会在用户滚动后将我的页面滚动到最近的定义点（图层的顶部）。带有大量'if'语句的jQuery。必须有一个更好的方法来做到这一点？

我对自己感到非常满意，但我看着代码，它看起来很笨重和可怕。没有太多的闲言：

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

来源

2014-01-05 SuperKim

这和jQuery有什么关系？ – Christian

@Christian scrollTop（）是一种jQuery方法。这是我看到的唯一的东西。 –

是的，只有90％左右的代码是纯javascript ...... – Christian

或许像

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

    goToByScroll(target); 
}

来源

2014-01-05 23:42:18

我即将张贴相同。然后'压裂'是无关紧要的。 –

@Tim，非常正确..删除它.. –

我已经测试过这个功能，但是它不能正常工作，页面会从第三张幻灯片滑到最上面的幻灯片，有时它会上升两个，也是它的重点选择是不同的每张幻灯片，嗯，这是一个不错的主意，但我认为数学不一样。 – SuperKim

带有大量'if'语句的jQuery。必须有一个更好的方法来做到这一点？

回答

相关问题