調用函數內部的原型方法不起作用

Question

我正在嘗試使用Javascript和jQuery 創建一個庫類 。

我為該庫類有一些原型函數 ，但是當我嘗試在類中訪問這些函數時，它顯示錯誤。

Uncaught TypeError: Gallery.start is not a function

我的JSfiddle鏈接

這是我的Javascript：

/**
 *
 * @param options should be object of following
 * options.images Array of gallery images URL
 * options.start slide starting point
 * options.autoPlay This option will be false by default
 *
 */
function Gallery(options) {
  var _this = this;
  var galleryOptions = options;

  function randomString(len, charSet) {
    var ranString = '';
    var randomPoz;
    charSet = charSet || 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789';
    for (var i = 0; i < len; i++) {
      randomPoz = Math.floor(Math.random() * charSet.length);
      ranString = ranString + charSet.substring(randomPoz, randomPoz + 1);
    }
    return ranString;
  }

  function addThumbnail(imgSrc) {
    return '<li class="thumbnail__list"><img src="' + imgSrc + '" class="thumbnail__list-img" /></li>';
  }

  function renderThumbnail(imgArr) {
    var html = [];
    if (imgArr) {
      $.each(imgArr, function(i, v) {
        html.push(addThumbnail(v));
      });
    }
    return '<ul class="gallery__thumbnail__container">' + html.join('') + '</ul>';
  }

  function getPlaceholder() {
    return 'dummy.jpg';
  }

  function disableNext(thisObj) {
    thisObj.galleryElement.find('.next').addClass('disabled');
  }

  function disablePrev(thisObj) {
    thisObj.galleryElement.find('.prev').addClass('disabled');
  }

  function enableNext(thisObj) {
    thisObj.galleryElement.find('.next').removeClass('disabled');
  }

  function enablePrev(thisObj) {
    thisObj.galleryElement.find('.prev').removeClass('disabled');
  }

  function togglePrevNext(self) {
    if (self._opt.currIndex === (self._opt.imagesLength)) {
      disableNext(self);
    } else {
      enableNext(self);
    }
    if ((self._opt.currIndex - 1) === 0) {
      disablePrev(self);
    } else {
      enablePrev(self);
    }
  }

  function controls() {
    var html = [];
    html.push('<div class="prev sz-icon-arrow-left"></div>');
    html.push('<div class="next sz-icon-arrow-right"></div>');
    return '<div class="gallery__controls">' + html.join('') + '</div>';
  }

  function bindClickEvents(galleryElement) {
    galleryElement.on('click', '.start', function() {
      _this.start();
    });
    galleryElement.find('.stop').on('click', function() {
      _this.stop();
    });
    galleryElement.find('.prev').on('click', function() {
      _this.prev();
    });
    galleryElement.find('.next').on('click', function() {
      _this.next();
    });
    galleryElement.find('.thumbnail__list').on('click', function() {
      _this.goTo(Number($(this).index()) + 1);
    });
  }

  function checkOptions(option) {
    var opt = option;
    opt.images = (opt.images.length) ? opt.images : getPlaceholder();
    opt.thumbnail = (opt.thumbnail) ? true : false;
    opt.fullScreen = (opt.fullScreen) ? true : false;
    opt.container = (opt.fullScreen) ? 'body' : opt.container;
    opt.container = (opt.container) ? opt.container : 'body';
    opt.autoPlay = (opt.autoPlay) ? true : false;
    opt.start = (opt.start && opt.start <= Number(opt.images.length)) ? opt.start : 1;
    return opt;
  }

  Gallery.init = function() {
    var html;
    _this._opt = checkOptions(galleryOptions);
    _this._opt.imagesLength = Number(_this._opt.images.length);
    _this._opt.currIndex = Number(_this._opt.start);
    _this._opt.elementName = 'gallery--' + randomString(5, 'SZgallery');
    if (_this._opt.fullScreen) {
      html = '<div class="pop-up__model ' + _this._opt.elementName + '">' +
        '<div class="pop-up__model-table">' +
        '<div class="pop-up__model-cell">' +
        '<div class="gallery"><div class="pop-up__model-content">' +
        '<div class="gallery__inner-wrapper"></div>' +
        '</div></div>' +
        '</div></div>' +
        '</div>';
      $(_this._opt.container).append('', html);
      if (_this._opt.thumbnail) {
        $('.pop-up__model-content').append('', '<div class="thumbnail__list-wrapper">' + renderThumbnail(options.images) + '</div>').append('', controls());
      }
    } else {
      $(_this._opt.container).append('', '<div class="gallery gallery--hidden ' + _this._opt.elementName + '"></div>');
    }
    _this.galleryElement = $('.' + _this._opt.elementName);
    if (_this._opt.fullScreen) {
      _this.galleryElement.find('.gallery__inner-wrapper').append('', '<div class="gallery__img-holder">' +
        '<img class="gallery__img-preview" src="' + _this._opt.images[_this._opt.start - 1] + '"/>' +
        '</div>');
    } else {
      _this.galleryElement.append('', '<div class="gallery__img-holder">' +
        '<img class="gallery__img-preview" src="' + _this._opt.images[_this._opt.start - 1] + '"/>' +
        '</div>');
    }
    if (_this._opt.thumbnail) {
      _this.galleryElement.append('', renderThumbnail(options.images)).append(controls());
    } else {
      _this.galleryElement.append('', controls());
    }

    if (_this._opt.autoPlay) {
      Gallery.start();
    }
    bindClickEvents(_this.galleryElement);
  };

  Gallery.prototype = {
    start: function() {
      _this.goTo(_this._opt.currIndex);
      _this.galleryElement.removeClass('gallery--hidden');
      console.log('started', _this._opt);
      if (_this._opt.fullScreen) {
        $('.pop-up__model').addClass('is_visible');
      }
    },
    stop: function() {
      _this.galleryElement.addClass('gallery--hidden');
    },
    next: function() {
      if (_this._opt.currIndex <= (_this._opt.imagesLength - 1)) {
        _this._opt.currIndex++;
        _this.goTo(_this._opt.currIndex);
      }
    },
    prev: function() {
      if ((_this._opt.currIndex - 1) !== 0) {
        _this._opt.currIndex--;
        _this.goTo(_this._opt.currIndex);
      }
    },
    goTo: function(imgNo) {
      var thumbnail;
      _this._opt.currIndex = Number(imgNo);
      if (_this._opt.images[imgNo - 1]) {
        _this.galleryElement.find('.gallery__img-preview').attr('src', _this._opt.images[imgNo - 1]);
      }
      if (_this._opt.thumbnail) {
        thumbnail = _this.galleryElement.find('.thumbnail__list');
        thumbnail.removeClass('active');
        thumbnail.eq(imgNo - 1).addClass('active');
      }
      togglePrevNext(_this);
    },
    destroy: function() {
      console.log('destroyed');
    }
  };

  return Gallery;
}

var imgArr = ['images/placeholder-1.png', 'images/placeholder-2.png', 'images/placeholder-3.jpg', 'images/placeholder-4.jpg'];


var hotelPhotosGallery = new Gallery({
  images: imgArr,
  autoPlay: true,
  container: '.photo-list'
});
hotelPhotosGallery.init();

Answer 1

看了一下你的代碼后，我發現了問題：

if (_this._opt.autoPlay) {
  Gallery.start(); // This method doesn't exist on a non instantiated object
}

您正在嘗試在函數本身上調用方法 ，而不是在該函數的實例原型上調用方法 （使用new實例化）。

但是，由於原型方法將無法使用非實例化項目 this一個功能被默認安裝到全局對象。 當使用new ，那this實際上變成一個新的對象，被隱式地從返回的function ，從而創建一個新的實例。 更多關於使用函數構造函數

通常，這可以通過使用上下文（即this ）而不是函數名來修復：

if (_this._opt.autoPlay) {
  this.start(); // Use the context of the instantiated Gallery
}

但是，在您的特定情況下，您正在重新定義原型：

Gallery.prototype = { ... } // You are overwriting the prototype object

而不是使用JS引擎已經為您准備的那個並在其上添加方法：

// add methods directly on the prototype
Gallery.prototype.method1 = // ...
Gallery.prototype.method2 = // ...

這種方式可能不安全，因為在重新定義之前，它最終將刪除原型中存在的任何其他內容。

如果您不關心這一點，並希望保持代碼的原樣，您也可以通過顯式調用原型來訪問您的方法：

if (_this._opt.autoPlay) {
  Gallery.prototype.start(); // Explicitly call the prototype, since you already defined it
}

Answer 2

你沒有正確地稱呼它。 如果你需要調用原型函數，它必須是這樣的

Gallery.prototype.start（）

如果你想實現一個特殊的行為（基於對象），那么它應該像這樣 。 這里還有一些在JavaScript中模擬類的不同方法