Что лучше, самовызывающаяся функция или объект для плагина?

Question

[AlexandrMa]

Делаю плагин на JS. Вижу абсолютно разную реализацию. Хотелось бы делать по какому-то единому стандарту.

1. (делаю пока что именно так)

let Plugin = {
  param1: 1,
  param2: 1,
  method1(){
    //
  },
  method2(){
    //
  }
};

2. (часто вижу так)

(function() {

  this.Plugin = function(){

    this.param1 = 1;
    this.param2 = 2;
      
  }

  Plugin.prototype.method1 = function(){
    // public
  }

  function method2(){
    //private
  }

}());

Так как нужно то?)

Comments

[Сергей Соколов]

плагин – для чего именно? Вероятно, есть какие-то требования-рекомендации, как, например, для плагинов jQuery.

[AlexandrMa]

Сергей Соколов, требований нет. Сделать самостоятельный блок сайта, который хранит в себе все значения, методы. Ни с кем не взаимодействует. Сам по себе.

[AlexandrMa]

Сергей Соколов, на javascript.

[sinneren]

стандартов нет, в js анархия.
в вашем примере 2 плагин не доступен извне. если только не присвоить чему-то.
Полагаю, правильнее бы было 2 оформить так:

(function (global) {
var instance = this;
var Plugin = function ....
//some code

global.plugin_name = new Plugin();
} (window));

но... как хотите вообще

[AlexandrMa]

sinneren, а первый пример, он хуже разве?

[Вадим]

sinneren, Почему же во втором случае недоступно извне? Там прокидывается
this.Plugin = function....
А this будет window.

[Владимир]

AlexandrMa, я бы так и делал, но просто потому, что мне нравятся объекты. И с ними немного проще понять что же хранится в this :)

Но это вкусовщина и делать надо так, как вам самом будет потом проще читать и дебажить.

[AlexandrMa]

Ребят, и всё же, как делать то?)
Мне надо как раз инкапсулировать всё внутрь моего плагина. Переменные, методы.
По сути, но встраивается в body. Создает нужные дивы и кнопки, вешает на них addEventListener и тд.
Обычная галерея на JS.

[AlexandrMa]

Владимир, да мне то проще по методу 1. Просто не хотелось бы, потом отдать код другим кодерам и слушать негодования)

[Владимир]

AlexandrMa, если есть корпоративные стандарты, то делать по ним. А если нет, то каждого негодующего слушать, слушалка завянет :).

[sinneren]

Вадим, а, ну да

[sinneren]

AlexandrMa, нет

Answer 1

[Сергей Соколов]

У 1-го варианта возможен косяк, если вдруг подключат файл с плагином дважды: ошибка из-за переопределения переменной Plugin. Как вам вариант:

{ // обернуть, чтобы не засорять глобальную область

  // внутри любая анархия, переменные

  // исходный вариант 1
  const Plugin = {
    param1: 1,
    param2: 1,
    method1(){
      //
    },
    method2(){
      //
    }
  };

  window.Plugin = Plugin;
}

Comments

[AlexandrMa]

Спасибо.