kaladin13 / node_template Goto Github PK

Не используй console.log(/**/). Используй какое-либо решение для логов(winston, log4js ...).

Enum to log response status.

Блок дублируется, можно было бы его вынести.

Единый интерфейс для возвращения данных из контроллеров.

Директория resources обычно находится в src.

Хотелось бы видеть README и описание контрактов для работы с сервисом.

Не очень ясна важность данной функции и ее обозначение как static. Возможно ли вынесение в отдельный модуль?

Не используй require и import вместе, выбери что-то одно.

Продумай внедрение зависимостей.

Стандартная сигнатура в Express - (req, res, next). Было бы неплохо сохранять имена передаваемых параметров теми же, для удобочитаемости.

Данные для валидации необходимо выносить в property.

Надо определиться с тем, как конкретно ты именуешь файлы(заглавные и строчные буквы, добавление имени и задач директории к имени файла и др).

Также это относится к переменным, потому что параметры с использованием _ поодиночке приводят в уныние.

Постарайся использовать const везде, где это возможно.

Express code style.

Для обозначения большого кол-ва маршрутов нужно создавать отдельный Router и выносить логику в него.

Стоит или типизировать все параметры, или не типизировать вообще.(Само собой первое лучше)

Не используй магические константы в коде, выноси их в property файлы, и используй оттуда.

Избыточные объявления локальных переменных.

Не до конца понятно, что данная функция возвращает