jQuery - 在选择器不存在的页面上导致冲突问题的代码段

时间:2011-06-26 18:58:31

标签: javascript jquery plugins conflict

好的,我正在开发一个WordPress主题。在单个帖子页面上,我有一个注释div,它使用一些jquery在页面上浮动。我还运行一个模式弹出窗体来登录。当#commentWrapper(jquery浮动效果的选择器)存在时,这在单页面上完全没问题。但是,在没有#commentWrapper浮动的页面上,模态窗体不起作用。我在一般的jQuery调用中将问题固定到这一行(删除每一行并进行测试)。

调用general.js,最后一次电话:

jQuery('#commentWrapper').stickyfloat({ duration: 300, easing : 'easeInQuad' });

它引用的实际插件:

    $.fn.stickyfloat = function(options, lockBottom) {
            var $obj                = this;
            var parentPaddingTop    = parseInt($obj.parent().css('padding-top'));
            var startOffset         = $obj.parent().offset().top;
            var opts                = $.extend({ startOffset: startOffset, offsetY: parentPaddingTop, duration: 200, lockBottom:true }, options);

            $obj.css({ position: 'absolute' });

            if(opts.lockBottom){
                var bottomPos = $obj.parent().height() - $obj.height() + parentPaddingTop; //get the maximum scrollTop value
                if( bottomPos < 0 )
                    bottomPos = 0;
            }

            $(window).scroll(function () { 
                $obj.stop(); // stop all calculations on scroll event

                var pastStartOffset         = $(document).scrollTop() > opts.startOffset;   // check if the window was scrolled down more than the start offset declared.
                var objFartherThanTopPos    = $obj.offset().top > startOffset;  // check if the object is at it's top position (starting point)
                var objBiggerThanWindow     = $obj.outerHeight() < $(window).height();  // if the window size is smaller than the Obj size, then do not animate.

                // if window scrolled down more than startOffset OR obj position is greater than
                // the top position possible (+ offsetY) AND window size must be bigger than Obj size
                if( (pastStartOffset || objFartherThanTopPos) && objBiggerThanWindow ){ 
                    var newpos = ($(document).scrollTop() -startOffset + opts.offsetY );
                    if ( newpos > bottomPos )
                        newpos = bottomPos;
                    if ( $(document).scrollTop() < opts.startOffset ) // if window scrolled < starting offset, then reset Obj position (opts.offsetY);
                        newpos = parentPaddingTop;
                    $obj.animate({ top: newpos }, opts.duration );
                }
            });
        };

如果我添加一个if命令来查看选择器是否存在,那么一切正常。我想知道未来网站的问题是什么。

1 个答案:

答案 0 :(得分:1)

好吧,你的stickyfloat()方法假设很多东西,比如总是在包含至少一个元素的jQuery对象上调用,或者那个元素总是有父元素。例如,请考虑以下代码:

var $obj = this;
// ...
var startOffset = $obj.parent().offset().top;

如果调用方法的jQuery对象为空,或者其第一个元素没有父元素(该方法在$("html")上调用),则代码将失败,因为parent().offset()将为{{ 1}}。

如果你希望你的方法更健壮,你不应该假设任何关于它被调用的对象。一个好的第一步是使方法可链接,这对您的用户总是有益的,并将摆脱第一个问题。执行此操作的recommended way是:

null

由于代码现在通过匿名函数按顺序在每个元素(如果有)上运行,因此可以通过提前返回来处理对父元素存在的测试:

$.fn.stickyfloat = function(options, lockBottom) {
    return this.each(function() {
        var $obj = $(this);
        // The rest of your code.
    });
};