search

hoka-hoka / ML.test

0 stars 0 forks source link

JS #12

Open iatsdotfatr opened 3 years ago

iatsdotfatr commented 3 years ago
  1. Деструктуризируй объект, передаваемый в параметрах, чтобы сразу было видно, какие у него поля. Деструктуризировать можно или в самих параметрах, или в теле функции.
  2. Вместо var используй const, если надо, let, даже если это pug. prefer destr, vars
hoka-hoka commented 3 years ago

Исправил.

iatsdotfatr commented 3 years ago
  1. Еще можно кое-где деструктурировать.
  2. Установить eslint и добавить туда правила airbnb.
  3. JS код переписать на es6 классы.
hoka-hoka commented 3 years ago
  1. Пересмотрел, добавил деструктуризацию ещё в двух pug-миксинах, куда могут передаваться параметры. Мы ведь говорим о деструктуризации именно в pug-миксинах или вообще во всём js?
iatsdotfatr commented 3 years ago

Сейчас про pug, но в JS тоже надо деструктурировать то, что уместно деструктурировать

hoka-hoka commented 3 years ago

Правила к eslint от airbnb добавил.

iatsdotfatr commented 3 years ago

  1. Предполагается, что после добавления правил будут исправлены появившиеся предупреждения. Screenshot from 2021-03-21 15-49-54

  2. Логика некоторых компонентов до сих пор не переписана на классы.

  3. Все классы должны именоваться с заглавной буквы. В одном файле должен находиться один класс, файл также должен именоваться с заглавной буквы.

  4. Не использовать циклы, заменить их на методы массивов.

  5. Проверить правильность импортов.

iatsdotfatr commented 3 years ago

Зачем использовать IIFE вместе с модулями? Или переписать на класс, или, если это сторонняя библиотека, подключить через npm. Screenshot from 2021-06-21 12-39-39

iatsdotfatr commented 3 years ago
  1. Часть элементов выбирается через js-, часть нет. И js- должен быть префиксом, то есть .js-block/element/modifier.block/element/modifier. js-