Дз5

[Yunnii]

No description provided.

[azproduction]

Лучше сделать вот так:

var Collection = function (items) {

};

// на то, что имя повторяется - пофиг
exports.Collection = Collection;

// и дальше уже работать с локальной переменной...
Collection.prototype.constructor = exports.Collection;

Дело в том, что Collection на строке 16 и дальше будет получен из глобальнй области видимости (exports===window), а обернув в IEFE мы максимально хотим тем самым использовать локальные переменные. Но все это работать будет.

[azproduction]

💯 Молодец! Большая работа поделана :)

Посмотрел твой код мне показалось, что экспортируется много лишнего. Немного расскажу про смысл все этих IEFE и exports.

(function (exports) {
    exports.ListOfEvents = new Events();
}(window));

В JavaScript вообще нет такого понятия как модуль, но все очень желают их использовать потому как они ограждают ту область, которую должны видеть все от приватных переменных "IEFE - это эмуляция модуля". Если экспортировать все подряд, то смысл модуль теряет (разве что меньше писать "use strict").

Нужно выделть те объекты, которые должны быть видны всем (и они реально кем-то используются) и объекты и переменные, которые используются только этим модулем.

Это позволит отделить функции-интерфейсы модуля от левых переменных и уменьшить число глобальных переменных.

В идеале нужно стараться сделать так, чтобы в глобальную область видимости попало 0 твоих переменных. Те твои модули могут работать с глобальными объектами(подписываться на события например или использовать document.*), а сторонние скрипты не могут трогать и использовать твои переменные.

[Yunnii]

Поняла с IEFE. у меня были два модуля, и чтобы взаимодействовать между ними я выкидывала нужные мне переменные в глобал через exports т_Т . только сейчас дошло, что по логике это один модуль дб.

[azproduction]

Насчет TravisCI. Обнаружил новый токен в настройках TravisCI, вбил его в настройки репозитория Github, нажал сохранить, отправить - заработало.