search

redKOTik / fsd-frontend-second-task

MIT License
0 stars 0 forks source link

JS #6

Open redKOTik opened 3 years ago

redKOTik commented 3 years ago

Весь JS для блоков должен быть модульным. И наилучшее решение для такого подхода - классы. Тут ссылка на наши стандарты, хоть там написано про jQuery, но относится это и к обычному JS коду, т.к. он тоже не навязывывает какой-то конкретный паттерн проектирования.

Feinminen commented 3 years ago

Не понимаю зачем для инициализации классов используется следующая логика. image

Ты же просто этой записью делаешь тоже что мог бы сделать как new Rate(element') Зачем там статик метод, который тригерит конструктор?

Feinminen commented 3 years ago

Любой JS код должен влиять только на блок в котором он используется. Т.е. даже инициализации вне блока не должно происходить, инициализация должна происходить либо через миксин (и передавать туда параметры которые будут влиять на JS), либо делать отдельный класс для страницы, на котрой будет инкапсулировано использование других классов и вся работа с классами будет производиться через инстансы.

Feinminen commented 3 years ago

Все файлы, которые экспортируют класс - должны называться с большой буквы rate.js => Rate.js и т.д.