Хороший, плохой js
CodeStyle
useCamelCasePlease=))
Файл - Параметры - Параметры - Строка поиска
VAR
НИ НУЖЕН - устарело, переменные имеют глобальную область видимости, всплывают, можно где-то переопределить случайно, фиг потом найдешь где
LET, CONST
НАШЕ ВСЕ - современно, имеют блочную область видимости, как-то случайно переопределить сложнее. Использовать const для неизменяемых значений, let для изменяемых
Хорошо
Плохо
Объявление переменных
- Переменные объявляем явно;
- Переменные объявляем там где они используются, а не в начале функции;
- Имена переменных должны быть читаемыми, иметь смысл;
Значения аргументов по умолчанию
Шаблонизированные строки
Использование тернарных операторов
Смысл оператора ? – вернуть то или иное значение, в зависимости от условия. Пожалуйста, используйте его именно для этого. Когда вам нужно выполнить разные ветви кода – используйте if.
Декомпозиция и чистая функция
Пример кода
Декомпозируй - если есть одинаковые куски кода, реализующие одинаковую логику - вытаскивай в отдельную функцию и переиспользуй.
Плохо
Хорошо
Пиши чистые функции - функции, выполняющие одну задачу, без выполнения сайд эффектов
Плохо
Хорошо
Чистый код. Создание, анализ и рефакторинг. Мартин Роберт К.