Question

我正在处理一个带有固定菜单的页面，该菜单在用户从顶部滚动一定距离后拾取，当它们向下滚动页面时，菜单中的不同链接会被赋予一个更改颜色的类。所有这一切似乎都适用于Chrome和Safari，但在Firefox中，页面冻结在顶部。我想知道它是否在不断地循环一些代码......基本上冻结了窗口。

这是我的代码。

$.localScroll({
    onBefore: function() {
        $('body').data('scroll-executing', true);

    },
    onAfter: function() {
        $('body').data('scroll-executing', false);
        $(window).trigger("scroll");
    }
});

$(window).scroll(function () {
    if ($(this).scrollTop() > 259) {
        $('.nav').addClass("f-nav");
    } else {
        $('.nav').removeClass("f-nav");
    }
});

$(window).scroll(function() { 
    if ($('body').data('scroll-executing')) {
        return;
    }
    // find the a with class 'active' and remove it
    $("a").removeClass('active');
    // get the amount the window has scrolled
    var scroll = $(window).scrollTop();
    // add the 'active' class to the correct #nav based on the scroll amount

    if (scroll > 2150) {
        $("#nav_3").removeClass('active');
        $("#nav_5").attr('class', 'active');
        setHash("contact");

    } else if (scroll > 1300) {
        $("#nav_2").removeClass('active');
        $("#nav_3").attr('class', 'active');
        setHash("portfolio");

    } else if (scroll > 400) {
        $("#nav_2").attr('class', 'active');
        setHash("about");

    } else if (scroll <= 380) { //when I remove this section, the problem goes away.
        $("#nav_1").attr('class', 'active');
        setHash("home");
    }

});

我忘了添加setHash定义。在这里。

setHash = function(hash) {
    var scrollmem = $('body').scrollTop();
    window.location.hash = hash;
    $('html,body').scrollTop(scrollmem);
}

我还注意到CPU上升到100％，我似乎无法弄清楚原因。

问题在于以else开头的代码的第三部分if（scroll＆lt; = 380）。我通过消除过程想出来了。任何人都可以看到它循环或做一些永远不会结束的事情......或者会解释为什么firefox冻结在页面顶部？

我对这一切都很陌生......过去几天我刚拿起jquery，我基本上都在谷歌搜索并调整代码，以便它符合我的需要。

任何帮助都会非常感激。

Answer 1

在滚动事件上执行太多代码是过度的，在每个滚动上，浏览器触发滚动事件一百次，您可以考虑使用具有throttle或debounce等方法的库。

http://documentcloud.github.com/underscore/#throttle

将处理程序附加到窗口滚动事件是一个非常非常糟糕的想法。根据浏览器的不同，滚动事件可以激活很多，将代码放入滚动回调会减慢任何尝试滚动页面（不是一个好主意）。因此，滚动处理程序中的任何性能下降只会影响整体滚动的性能。相反，最好使用某种形式的计时器来检查每X毫秒或附加一个滚动事件，并且只在延迟后运行代码（或者甚至在给定次数的执行之后 - 然后延迟）。 http://ejohn.org/blog/learning-from-twitter/

Answer 2

基本上你在滚动事件中执行的太多了。正如@undefined所说，浏览器对此的调用比你想象的要多得多。一些提示：

当在一个将被多次调用的函数内部时，已经创建了jQuery对象。这样，每次调用函数时，都不会重新创建相同的jQuery对象。

var nav2 = $("#nav_2");
$(window).scroll(function() {
   ...
   nav2.removeClass('active');
   ...
});

类似地，一次又一次地调用时，添加和删除类会占用大量的处理周期 - 尤其是当您访问整个类的元素时，例如在$('.nav').addClass("f-nav");中。

相反，只有在不存在的情况下才尝试添加/删除类。类似的东西：

var alreadyAdded = false;
var alreadyRemoved = false;
$(window).scroll(function () {
    if ($(this).scrollTop() > 259) {
      if(!alreadyAdded){
        $('.nav').addClass("f-nav");
        alreadyAdded = true;
        alreadyRemoved = false;
      }
    } else if(!alreadyRemoved) {
        $('.nav').removeClass("f-nav");
        alreadyAdded = false;
        alreadyRemoved = true;
    }
});

所有这一切，它仍然可能会慢慢滚动，所有这些代码附加到滚动事件。也许还有另一种方法可以获得相同的效果。如果firefox冻结，我只能想象它在IE中的速度有多慢。

window.scroll函数冻结了firefox

2 个答案: