简化jQuery / Javascript代码

时间:2013-10-16 13:53:01

标签: javascript jquery algorithm simplify

我一直在尝试使用'$(输入[数据代码]))每个'函数来简化我的代码以简化代码。代码只是在用户点击锚链接时处理,它将它动画到那个位置,它看起来像:

$('a.welcome').click(function(){
    $('html, body').animate({scrollTop:0}, 2000, function() {
        parallaxScroll(); 
    });
    return false;
});

$('a.step1').click(function(){
    $('html, body').animate({scrollTop: $('#step1').offset().top - 70}, 2000, function() {
        parallaxScroll(); 
    });
    return false;
});

$('a.step2').click(function(){
    $('html, body').animate({scrollTop: $('#step2').offset().top - 70}, 2000, function() {
        parallaxScroll(); 
    });
    return false;
});

$('a.step3').click(function(){
    $('html, body').animate({
        scrollTop: $('#step3').offset().top - 70
    }, 2000, function() {
        parallaxScroll(); 
    });
    return false;
});

到目前为止我想出的简化它看起来像这样:

$("input[data-code]").each(function() {
    $(this).click(function(){
        $('html, body').animate({
            scrollTop: $($(this).data("code")).offset().top - 70
        }, 2000, function() {
            parallaxScroll(); 
        });
        return false;
    });
}); 

但它似乎没有起作用:(

我的理论是$(this).click(function(){实际上没有正确地调用样式中的元素$(a.welcome).click(function(){

链接的实际HTML如下所示:

<nav id="primary">
    <ul>
        <li>
            <h1>Welcome</h1>
            <a class="welcome" href="#welcome" data-code="0">View</a>
        </li>
        <li>
            <h1>Step 1: Setup</h1>
            <a class="step1" href="#step1" data-code="#step1">Step 1</a>
        </li>
        <li>
            <h1>Step 2: Data Management</h1>
            <a class="step2" href="#step2" data-code="#step2">Step 2</a>
        </li>
        <li>
            <h1>Step 3: Configure Cameras</h1>
            <a class="step3" href="#step3" data-code="#step3">View</a>
        </li>
    </ul>
</nav>

有什么想法吗?

4 个答案:

答案 0 :(得分:0)

您可以在点击功能

中使用此功能
$('#primary a').on('click', function () {
    var offSet = $($(this).data('code')).offset().top - 70;
    if ($(this).is('.welcome')) {
        offSet = 0;
    }
    $('html, body').animate({
        scrollTop: offSet
    }, 2000,

    function () {
        parallaxScroll();
    });
    return false;
});

答案 1 :(得分:0)

一个简单的

$('#scroll a').click(function(){
    $('html, body').animate({scrollTop: $(this).offset().top - 70}, 2000, 
    function() {
        parallaxScroll(); 
    });
    return false;
});

并将id = "scroll"添加到ul

<nav id="primary">
        <ul id = "scroll">
            <li>
                <h1>Welcome</h1>
                <a class="welcome" href="#welcome" data-code="0">View</a>
            </li>
            <li>
                <h1>Step 1: Setup</h1>
                <a class="step1" href="#step1" data-code="#step1">Step 1</a>
            </li>
            <li>
                <h1>Step 2: Data Management</h1>
                <a class="step2" href="#step2" data-code="#step2">Step 2</a>
            </li>
            <li>
                <h1>Step 3: Configure Cameras</h1>
                <a class="step3" href="#step3" data-code="#step3">View</a>
            </li>
        </ul>
</nav>

会起作用。

答案 2 :(得分:0)

您的jQuery代码定位的是HTML输入标记,而不是锚标记。

变化:

$("input[data-code]").each(function() {

为:

$("a[data-code]").each(function() {

答案 3 :(得分:0)

我认为,我会这样做。 您想要定位锚标签。

$("a[data-code]").click(function(){
    var val = $(this).data("code") ? $($(this).data("code")).offset().top - 70 : 0;
    $('html, body').animate({
        scrollTop: val
    }, 2000, function() {
        parallaxScroll(); 
    });
return false;
});