Question

在以下示例中将这些值存储为变量（.scroll（）之外）是否是一个好主意？

$(window).scroll(function(){
  if ( $(document).height() <= ( $(window).height() + $(window).scrollTop() ) ) {
    // position stuff
  } else {
    // other position stuff
  }
});

我看到的唯一缺点是，在窗口调整大小的情况下，应再次检查的高度。或者我只需要自己存储对象？

非常感谢任何反馈！

Answer 1

至少，肯定一个好主意是在$(document)回调之外存储对$(window)和scroll的引用，因为滚动事件可能会反复快速地被解雇。

var $win = $(window),
    $doc = $(document);

$win.scroll(function(){
  if ( $doc.height() <= ( $win.height() + $win.scrollTop() ) ) {
    // position stuff
  } else {
    // other position stuff
  }
});

按照您的要求进行操作并不会造成伤害，并将文档和窗口高度存储在回调之外 - 只要您根据需要更新它们。处理所有这些的一个好方法是使用优秀的jQuery throttle/debounce plugin，它为您提供了一个简单的界面，以确保函数不会过于频繁地运行。

Answer 2

我做同样的事情，但我添加其他东西：

$(window).scroll(function(){
  if ( $(document).height() <= ( $(window).height() + $(window).scrollTop() ) ) {
    // position stuff
  } else {
    // other position stuff
  }
}).resize(function(){$(this).scroll()});//will call the scroll method on resize

或者你可以做的是：

function pageSizer(){
      if ( $(document).height() <= ( $(window).height() + $(window).scrollTop() ) ) {
        // position stuff
      } else {
        // other position stuff
      }
}

$(window).scroll(pageSizer).resize(pageSizer);

Answer 3

存储$(document)和$(window)，但重新检查height（）和scrollTop（），因为这可能会根据用户互动进行更改。

Answer 4

每次使用jQuery包装Element时，您都在创建一个jQuery对象。如果您打算多次使用相同的jQuery对象，那么缓存它是明智之举。这是通过将其存储在变量中并在每次要使用它时调用变量来完成的。这将不再创建jQuery对象，而是使用缓存版本。

前：

var wh = $(window).height();

我应该将$（document）.height（）和/或$（window）.height（）存储为变量吗？

4 个答案: