Как передавать опции в функцию конструктор?

Question

[Андрей Кварцов]

Доброе утро, Тостер!

Всё глубже начинаю изучать JavaScript. И чем дальше, тем больше вопросов.

На codepen сделал небольшую функцию конструктор, а для прототипа конструктора добавил несколько методов, тут возникает несколько вопросов:
1. Правильно ли организован конструктор и его прототип в целом?
2. Правильно ли вызывать метод init() прямо в конструкторе?
3. Как для конструктора задать стандартные опции?

Например, я хочу отдавать опцию: active, по дефолту она false, на при создании нового экземпляра объекта, я хочу отдавать true - следующим образом:

var x = new iRead('.selector', {
    active: true
});

А в методе init проверять значение переменной, и совершать какие-то действия:

...
init: function() {
    var item = this.item;
    this.toggleBtn.on(EVENTS.CLICK, function() {
      $(item).toggleClass(CLASS_LIST.READ_CLASS);
    });
   if(active) {
      $(item).addClass(CLASS_LIST.READ_CLASS);
     }
  }
...
});

И вообще, правильно ли я понимаю, как работаю опции?

Answer 1

[Anton fon Faust]

передавать опции в iRead так же, как и в любые другие функции в js

function iRead(cls, options) {
    defaultOptions: {
        active: false
    }
    this.options = $.extend( defaultOptions, options ); // вот тут сохраняем наши опции в свойства новоиспеченного объекта
}

а init лучше переписать на что-то такое:

init: function() {
    var _this = this;
    var item = this.item;

    this.toggleBtn.on(EVENTS.CLICK, function() {
      _this.toggleClass($(item), CLASS_LIST.READ_CLASS, active);
    });
    _this.toggleClass($(item), CLASS_LIST.READ_CLASS, active);
  }

toggleReadClass: function(el, cls, active) {
    el.toggleClass(cls, active);
}

Comments

[Андрей Кварцов]

Anton fon Faust по поводу опций полностью понял, благодарю за ответ.

А вот по init() мне не совсем понятно, что вы хотели этим показать.. Вероятнее всего, у вас ошибка в коде и вы хотели написать:

_this.toggleReadClass($(item), CLASS_LIST.READ_CLASS, active);

Объясните пожалуйста, чем такой подход к init() лучше, чем мой обычный?
И правильно ли вызывать init() прямо в функции конструкторе?

[Андрей Кварцов]

Anton fon Faust, подскажите еще пожалуйста.

Помимо обычных опций, которые можно будет переопределить при создании экземпляра, мне нужно создать приватные опции, которые нельзя будет изменить извне функции конструктора. Правильно ли я понимаю, что такие опции нужно создавать прямо внутри функции конструктора и после ссылаться на объект с приватными функциями? Объясню что имел ввиду: Joxi

[Андрей Кварцов]

Anton fon Faust, по поводу приватных опций понял - что их нужно ставить в самый конец $.extend, чтобы они не переопределялись в options, который передаётся при создании экземпляра

[Anton fon Faust]

Андрей Кварцов, да, опечаточка)
Мой подход лучше, так как делает код более простым и понятным: вам не нужно выяснять что делают две разные функции.

Вызов init зависит от вашей логики.
В js нет спец метода init, как в некоторых других языках, поэтому нет разницы как вы ее назовёте. Если подходит по логике - вызывайте.
По поводу приватных методов - в js, как правило это делается с помощью замыканий