如何覆盖 JQuery 的 .show() 和 .hide() 以触发前后的事件？

Question

我正在尝试覆盖 JQuery 的 .show 和 .hide 方法以在使用以下代码调用它们之前和之后启动触发器事件。

$(document).ready(function () {
    $('#dataBox').bind('afterShow', function () {
        alert('afterShow');
    });
    $('#dataBox').bind('afterHide', function () {
        alert('afterHide');
    });
    $('#dataBox').bind('beforeShow', function () {
        alert('beforeShow');
    });
    $('#dataBox').bind('beforeHide', function () {
        alert('beforeHide');
    });
    $('#toggleButton').click(function(){
        if($('#dataBox').is(':visible')) {
            $('#dataBox').hide ();
        } else {
            $('#dataBox').show();
        }
    });
});

jQuery(function ($) {
    var _oldShow = $.fn.show;
    //Override jquery's 'show()' method to include two triggered events before and after
    $.fn.show = function (speed, oldCallback) {
        return $(this).each(function () {
            var obj = $(this),
                newCallback = function () {
                    if ($.isFunction(oldCallback)) {
                        oldCallback.apply(obj);
                }
                obj.trigger('afterShow');
            };
        obj.trigger('beforeShow');
            _oldShow.apply(obj, [speed, newCallback]);
        });
    }
});

jQuery(function ($) {
    var _oldHide = $.fn.hide;
    //Override jquery's 'hide()' method to include two triggered events before and after
    $.fn.hide = function (speed, oldCallback) {
        return $(this).each(function () {
            var obj = $(this),
                newCallback = function () {
                    if ($.isFunction(oldCallback)) {
                        oldCallback.apply(obj);
                }
                obj.trigger('afterHide');
            };
        obj.trigger('beforeHide');
            _oldHide.apply(obj, [speed, newCallback]);
        });
    }
});

我有以下标记：

<input type='text' id='dataBox'/>
<input type='button' value='toggle' id='toggleButton' />

当我单击切换按钮时，会触发“beforeHide”和“beforeShow”事件，而“afterShow”和“afterHide”则不会。谁能告诉我我做错了什么？

Answer 1

请检查demo fiddle。希望有效

$.show / $.hide - 覆盖函数：

(function($){
$.override ={'show': $.fn.show, 'hide': $.fn.hide};

$.each($.override,function(M,F){

        var m=M.replace( /^\w/, function(r){ return r.toUpperCase(); });

        $.fn[M] = function(speed, easing, callback) {

            var args=[speed||0,easing||'',callback||function(){}];

            if( $.isFunction(speed)){
                args[2]=speed;
                args[0]=0;
            }                   
            else if( $.isFunction(easing)){
                args[2]=easing;
                args[1]='';
            }                   

            if(!this.selector){
                F.apply(this, arguments);
                return this;
            }

            return this.each(function () {
                var obj = $(this),
                    oldCallback = args[args.length-1],
                    newCallback = function () {
                        if ($.isFunction(oldCallback)){
                            oldCallback.apply(obj);
                        }
                        obj.trigger('after'+m);
                };

                obj.trigger('before'+m);
                args[args.length-1]=newCallback;

                //alert(args);
                F.apply(obj,args);

            });
        }
});
})(jQuery);

用法

jQuery(function($){
var $dataBox=$('#dataBox').bind('beforeHide afterHide beforeShow afterShow', function(e) {
    alert(e.type);
});

$('#toggleButton').bind('click',function(){
    $dataBox[$dataBox.is(':visible')?'hide':'show'](100,'swing',function(){ alert('end of animation');});
    return false;
});
});

Answer 2

我在您的代码中设置了一些断点，发现您的 newCallback 函数永远不会被调用。我认为那是因为您在调用该函数时没有使用速度参数。 从jQuery的原始函数文档$.show可以看出，你可以用no、one、two或`三个参数调用$.show

.show( 持续时间 [, 回调] )

duration 一个字符串或数字，确定动画将运行多长时间。

callback 动画完成后调用的函数。

添加的版本：1.4.3

.show( [duration] [,easing] [, callback] )

duration 一个字符串或数字，确定动画将运行多长时间。

easing 一个字符串，指示用于过渡的缓动函数。

callback 动画完成后调用的函数。

所以你可能会也可能不会传递一堆不同的参数。所以我认为问题出在这一行

 _oldHide.apply(obj, [speed, newCallback]);

在这种特定情况下，speed 未定义。

你可以像这样调用$.fn.show函数

$(..).show(undefined,callBack)
//or
$(..).show(undefined,undefined, callBack)

如果没有指定其他参数，回调将始终是第一个参数，在这种情况下，jQuery 将使用它的默认 400ms 动画速度/持续时间。你会看到一个淡入/淡出类型的效果。使用此回调使其表现得像显示/隐藏

你必须打电话给_oldShow/_oldHide 喜欢

_oldShow.apply(obj, [0,newCallback]);

//or

_oldHide.apply(obj, [0,newCallback]);

在快速搜索中，我在互联网上找到了一个代码，它似乎工作正常，并且对您可以传递给 $.show 和 $.hide 的所有参数都有特殊处理。检查下面的小提琴来看看

Working Fiddle

Answer 3

为什么没有触发 afterShow 和 afterHide 事件

如果显示/隐藏动画，jQuery 只会调用回调函数（代码中的newCallback）。如果.show()/.hide()没有传入参数（或者speed是undefined），jQuery会直接显示/隐藏元素，回调函数不会被执行。

为什么覆盖 .show() / .hide() 不是最好的主意

• jQuery 在内部调用.show() / .hide()（例如，作为.fadeTo() 的一部分）和递归调用（在进行动画显示/隐藏时），因此您的自定义事件将在您不期望的时候触发他们去。

• 恕我直言，最好将“库存”jQuery 和自定义（作为插件）分开，使用单独的方法名称/命名空间：

  • 当您的开发人员在 http://api.jquery.com 中查找库存方法时，他们会知道文档完整地描述了这些方法的行为。
  • 如果您的开发人员使用其他地方的代码，他们不会调用常规方法并期望特定于您公司代码库的行为。

.eventedShow() / .eventedHide()

这增加了.eventedShow() 和.eventedHide()（它采用与.show() / .hide() 相同的参数）并且应该做你想做的事：

(function( $ ) {

$.each( [ 'Show', 'Hide' ], function( i, method ) {
    $.fn[ 'evented' + method ] = function( speed ) {
        var originalMethod = method.toLowerCase(),
            hasCallback = false,
            args;

        // animate
        if ( speed || speed === 0 ) {
            args = $.makeArray( arguments );
            $.each( args, function( i, arg ) {
                if ( $.isFunction( arg ) ) {
                    args[ i ] = function() {
                        arg.apply( this, arguments );
                        $( this ).trigger( 'after' + method );
                    };
                    hasCallback = true;
                    return false;
                }
            } );
            if ( !hasCallback ) {
                args.push( function() {
                    $( this ).trigger( 'after' + method );
                } );
            }

            this.each( function() {
                var el = $( this );
                el.queue( function() {
                    $( this )
                        .trigger( 'before' + method )
                        .dequeue();
                } );
                $.fn[ originalMethod ].apply( el, args );
            } );

        // straight show/hide
        } else {
            this.trigger( 'before' + method );
            $.fn[ originalMethod ].apply( this, arguments );
            this.trigger( 'after' + method );
        }

        return this;
    };
} );

})( jQuery );

您可以在这里尝试代码：http://jsfiddle.net/jefferyto/bv8D2/

您仍然需要在现有代码中将 show / hide 更改为 eventedShow / eventedHide，但这可以通过全局搜索和替换来完成。