search

xenialugovaya / Hotel-booking-service-template

MIT License
0 stars 0 forks source link

JS #4

Closed iatsdotfatr closed 4 years ago

iatsdotfatr commented 4 years ago

Установить eslint с правилами Airbnb, сделать исправления в соответствии с правилами, и ознакомиться с правилами непосредственно https://github.com/airbnb/javascript.

Безымянный

Плагины устанавливать через npm и подключать из node_modules.

iatsdotfatr commented 4 years ago

Или для каждого блока с js-файлом делай инициализацию в отдельном js-файле (добавив постфикс Init camelcase'ом), или инициализируй в том же файле (donut-chart исключение). Безымянный

iatsdotfatr commented 4 years ago

import Импорты из node_modules как они делаются по умолчанию, eslint плохо работает с jquery, в таких случаях делай, например комментарий / eslint-disable-next-line no-unused-vars /. export Функция куда-то экспортируется? Если нет, то и объявление экспорта не нужно. Проверить по всему проекту.

iatsdotfatr commented 4 years ago

rules

handlers naming

iatsdotfatr commented 4 years ago

конс конс 2 Если в конструкторе вызывается несколько методов, объедини их в один.

iatsdotfatr commented 4 years ago

Дополнительный класс только для выбора модуля, поиск внутри по имеющимся классам, проверить все файлы. noneed

iatsdotfatr commented 4 years ago
  1. Кэшировать, не только здесь. cache
  2. Объединить. merge

Код в календарях, естественно, повторяется, имеет смысл объединить в один класс.