JQuery 插件在页面中多次使用时无法正常工作

Question

我正在尝试编写一个名为 grid2carousel 的 JQuery 插件，它在桌面设备上采用 Bootstrap 样式的网格中的一些内容，并在较小的屏幕上变成轮播。

如果它是一个页面上的唯一实例，该插件可以正常工作，但如果有多个实例，则会遇到一些问题。我在这里创建了一个 Codepen 来演示这个问题：

http://codepen.io/decodedcreative/pen/BzdBpb

尝试注释掉 codepen HTML 部分中的一个组件，调整浏览器的大小直到它变成轮播，然后在取消注释的情况下再次重复此过程

每当浏览器宽度低于 HTML 中的数据属性中指定的断点时，插件都会通过运行一个名为 SetupPlugin 的内部函数工作。如果浏览器宽度超过此断点，一个名为 DestroyPlugin 的函数会将 HTML 恢复到其原始状态。像这样：

        checkDeviceState = function(){
            if($(window).width()>breakpointValue){
                destroyPlugin();
            }else{
                if(!$element.hasClass('loaded')){
                    setupPlugin();
                }
            }
        },

下面是我的完整插件代码。有人可以指出我在这里做错了什么吗？

(function (window, $){
$.grid2Carousel = function (node, options){
    var
    options = $.extend({slidesSelector: '.g2c-slides', buttonsSelector: '.g2c-controls .arrow'}, {},options),
    $element = $(node),
    elementHeight = 0,
    $slides = $element.find(options.slidesSelector).children(),
    $buttons = $element.find(options.buttonsSelector),
    noOfItems = $element.children().length + 1,
    breakpoint = $element.data("bp"),
    breakpointValue = 0;

    switch(breakpoint){
        case "sm":
            breakpointValue = 767;
            break;
        case "md":
            breakpointValue = 991;
            break;
        case "lg":
            breakpointValue = 1199;
            break;
    }

    setupPlugin = function(){

        // Add loaded CSS class to parent element which adds styles to turn grid layout into carousel layout
        $element.addClass("loaded");

        // Get the height of the tallest child element
        elementHeight = getTallestInCollection($slides)

        // As the carousel slides are stacked on top of each other with absolute positioning, the carousel doesn't have a height. Set its height using JS to the height of the tallest item;
        $element.height(elementHeight);

        // Add active class to the first slide
        $slides.first().addClass('active');


        $buttons.on("click", changeSlide);

    },

    destroyPlugin =  function(){
        $element.removeClass("loaded");
        $element.height("auto");
        $buttons.off("click");
        $slides.removeClass("active");
    },

    checkDeviceState = function(){
        if($(window).width()>breakpointValue){
            destroyPlugin();
        }else{
            if(!$element.hasClass('loaded')){
                setupPlugin();
            }
        }
    },

    changeSlide = function(){
        var $activeSlide = $slides.filter(".active"),
            $nextActive = null,
            prevSlideNo = $activeSlide.prev().index() + 1,
            nextSlideNo = $activeSlide.next().index() + 1;

        if($(this).hasClass('left')){


            if(prevSlideNo !== 0){
                $nextActive = $activeSlide.prev();
                $nextActive.addClass('active');
                $slides.filter(".active").not($nextActive).removeClass("active");
            }else{
                $nextActive = $slides.last();
                $nextActive.addClass('active');
                $slides.filter(".active").not($nextActive).removeClass("active");
            }

        }else if($(this).hasClass('right')){


            if(nextSlideNo !== 0){
                $nextActive = $activeSlide.next();
                $nextActive.addClass('active');
                $slides.filter(".active").not($nextActive).removeClass("active");
            }else{
                $nextActive = $slides.first();
                $nextActive.addClass('active');
                $slides.filter(".active").not($nextActive).removeClass("active");
            }

        }
    },

    getTallestInCollection = function(collection){

        $(collection).each(function(){
            if($(this).outerHeight() > elementHeight){
                elementHeight = $(this).outerHeight();
            }
        });

        return elementHeight;

    };

    setupPlugin();
    checkDeviceState();
    $(window).on("resize", checkDeviceState);

}




$.fn.grid2Carousel = function (options) {
    this.each( function (index, node) {
        $.grid2Carousel(node, options)
    });

    return this
}
})(window, jQuery);

非常感谢，

詹姆斯

Answer 1

请检查插件代码中的第 30 行，您似乎只是忘记添加 var 关键字，因此没有创建局部变量来存储函数 setupPlugin、destoryPlugin 和所以你已经创建了全局变量，然后你的插件的每个新初始化都会重写这个函数以指向一个新创建的滑块。

setupPlugin = function(){

应该是

var setupPlugin = function(){

更新代码 here。

JQuery 插件在页面中多次使用时无法正常工作

JQuery plugin not working when used multiple times in a page

javascript

jquery

multiple-instances

jquery-plugins