На что обратить внимание, когда проводишь code review

При проведении code review важно проверять:

1. Читаемость кода

   • Осмысленные названия переменных и функций
   • Отсутствие "магических чисел"
   • Адекватное форматирование

2. Архитектурные решения

   • Соответствие принципам SOLID/DRY/KISS
   • Правильное разделение ответственностей
   • Отсутствие избыточных зависимостей

3. Безопасность

   • Санитизация пользовательского ввода
   • Защита от XSS/CSRF
   • Корректная обработка ошибок

4. Производительность

   • Отсутствие лишних ререндеров
   • Оптимальные алгоритмы
   • Правильное использование мемоизации

5. Тестируемость

   • Чистые функции
   • Достаточное покрытие тестами
   • Мокирование зависимостей

Пример проблемного кода:

// Плохо: магическое число, сложная логика
if (status === 3) {...} 

// Лучше:
const ORDER_STATUS = { COMPLETED: 3 };
if (status === ORDER_STATUS.COMPLETED) {...}