Хороший, плохой js

CodeStyle

useCamelCasePlease=))

Файл - Параметры - Параметры - Строка поиска

VAR

НИ НУЖЕН - устарело, переменные имеют глобальную область видимости, всплывают, можно где-то переопределить случайно, фиг потом найдешь где

LET, CONST

НАШЕ ВСЕ - современно, имеют блочную область видимости, как-то случайно переопределить сложнее. Использовать const для неизменяемых значений, let для изменяемых

Хорошо

Плохо

Объявление переменных

Переменные объявляем явно;
Переменные объявляем там где они используются, а не в начале функции;
Имена переменных должны быть читаемыми, иметь смысл;

Значения аргументов по умолчанию

https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Functions/Default_parameters

Шаблонизированные строки

https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/template_strings

Использование тернарных операторов

Смысл оператора ? – вернуть то или иное значение, в зависимости от условия. Пожалуйста, используйте его именно для этого. Когда вам нужно выполнить разные ветви кода – используйте if.