355 подписчиков
#советы Частые ошибки при код ревью
Мы в пятницу в клубе делали публичное код ревью нескольких проектов. Потому на основании его и код ревью, которое проводят наставники в школе, хочу обратить внимание на частые ошибки, с которыми мы сталкивались:
- Использование const вместо let. Когда вы пишете let, то кажется, что данная переменная может изменяться. Поэтому по умолчанию всегда используйте const, и только если вы действительно хотите задать переменной новое значение - let.
- Линтер и стиль кода. В любом проекте должен быть linter, а так же заранее заданный стиль кода и консистентность наименования функций и переменных. Даже если вы пока работаете над проектом одни.
- Уменьшение вложенности кода. Все негативные случаи в коде обрабатывайте в первую очередь, чтобы сократить вложенность if друг в друга, для упрощения чтения.
- Разделение ответственности. Весь ваш код должен быть максимально модульным, чтобы вы могли удобно его тестировать и каждая его часть имела свою одну определённую зону ответственности.
- Создание объектов. Если вам нужно создать большой объект и его наполнить, оптимально использовать паттерн Builder (разобран в моём курсе TypeScript). Он позволит упростить код и легко переиспользовать сборщик.
1 минута
16 июля 2023