Code review: тестовое задание по Node.js

Видео создано благодаря подписчикам проекта на нашем Patreon.

[alexveremeenko]

Тестовое задание сделано избыточно.
1. QueryBuilder - не нужен, достаточно патерна репозиторий с чистым sql + защита от sql injection
2. Кэширования достаточно простого
3. Настойку и тестирование можно упростить


Вывод: парень перестарался :), можно было так не напрягаться :)

[StepanZubashev]

А неплохой кандидат, как ни крути. Таких тяжело найти.

[ЯнГус-хд]

Я тут случайно оказался, специализируюсь на c#, но довольно интересно смотреть code review на другом языке, автор не токсичный и нормально относится к нелепым ошибкам (писавшего код), приятно было смотреть) query builder считаю само больное, что есть в коде.

[zaverden]

Про опечатки (2:53) - у того же VS Code есть расширение Code Spell Checker, для которого есть словари для разных языков. Для webstorm тоже есть spell check, думаю, можно русский словарь и там найти. Так что опечаток довольно легко избежать, если использовать инструменты более продвинуто

[lyflfflyflff]

А это тестовое задание на позицию Junior?

[bender]

это задание на какой уровень, дж мид син?

[webzen]

А как борешься с index.{js|ts} в реакте? Сам дико не люблю вообще дефолтные экспорты и названия вроде index.*, люблю когда все явно.
В реакте к сожалению так и оставил index, а во всем остальном поборол это.

[crutchmaster]

Как насчёт ревью кода имиджборды на ноде? Взялся бы?

[ivanvano]

Круто як Ілля розбирається у цьому коді, може хтось навчить мене, як таке можна розуміти, з чого нереально круто.

[devschacht]

better-npm-run давно не поддерживается и автор не рекомендует её использовать

[skipper]

Привет! Подскажи что по пунктам надо знать хорошо чтобы считать себя хотя бы мидлом в js желательно конечно уровнем по выше. Хочу с PHP пересесть на JS. Но все равно уровень владения JS считаю для себя достаточно низким. Спасибо

[sergeynakhankov]

Привет, это интеграционные тесты, e2e это уже UI тестирование

[StupidLova]

Было интересно, спасибо. Пару вопросов:


- Чем плох bodyParser на уровне app?
- Почему не используешь breadcamp меню для vscode?
- Как пользуешься minimap vscod-а?

[OnlineGamePlaying]

Шрифт в редакторе Iosevka? похож на этот сильно

[alexsh.]

Почему bodyparser на глобальном уровне это моветон?

[PutlerXLO]

Без QueryBuilder велосипеда, получишь дублирование кода, а если тебе запрещают готовый юзать, то у тебя по сути выхода просто нет...

[AlexanderShelestov]

2:30 "Я бы сделал много ко многим"

А я бы нет. И не взял бы на работу того, кто бы сел это делать, даже если это не требуется в задаче. Потому что переделать связь дело пяти минут и одной миграции (если понадобится в будущем), а желание кандидата пилить все впрок — плохой знак.