Как происходило Code Review на последнем месте работы?

Что такое Code Review и его значимость

Code Review (проверка кода) — это процесс, при котором один или несколько разработчиков анализируют код, написанный коллегой, перед его интеграцией в основную ветку проекта. Это критически важный процесс, который улучшает качество кода, распространяет знания в команде и предотвращает попадание багов в production.

Процесс: От создания PR до мержа

git checkout -b feature/add-user-profile
git add .
git commit -m "feat: Add user profile page with avatar upload"
// В описании PR указываю что было сделано, почему, и как тестировать

Требования к PR перед review

npm run test:run      // Все тесты зелёные
npm run lint          // Нет ошибок стилей
npm run build         // Успешная сборка
npm run test:coverage // Coverage >= 90%

// Коммиты логичные и читаемые
// Правильно: "feat: Add user profile"
// Неправильно: "wip", "asdf"

Как проходит сам review

Первичный анализ (5-10 минут):

• Читаю description и список изменённых файлов
• Проверяю что PR не слишком большой (не > 400 строк)
• Смотрю на масштаб изменений

Детальный review (15-30 минут):

// 1. Архитектура и структура
// - Не нарушены ли слои (presentation -> application -> domain)
// - Правильное разделение ответственности
// - Не скопирован ли код (DRY принцип)

// 2. Логика и корректность
// - Обработаны ли edge cases
// - Нет ли race conditions
// - Правильна ли бизнес-логика

// 3. Качество кода
function getUserData(userId) {
  // Неправильно: SQL injection
  const user = db.query(`SELECT * FROM users WHERE id = ${userId}`);
  
  // Правильно: защищено от SQL injection
  const user = db.query("SELECT * FROM users WHERE id = ?", [userId]);
}

// 4. Performance
// - Нет ли N+1 проблем
// - Не забыли ли useMemo/useCallback где нужно

// 5. Тестирование
// - Есть ли тесты
// - Покрывают ли основные сценарии
// - Edge cases обработаны

// 6. TypeScript
// - Нет any типов
// - Правильны ли интерфейсы

Типы комментариев в review

// 1. MUST FIX (Блокирующие)
// "This is a SQL injection vulnerability."

// 2. SHOULD FIX (Рекомендуемые)
// "Consider extracting this into a function for reusability"

// 3. NICE TO HAVE (Предложения)
// "This might be faster if we memoize this"

// 4. QUESTION (Уточнения)
// "Why did you choose this approach?"

// 5. PRAISE (Похвала)
// "Great test coverage!"

Пример комментария

// Код в PR:
async function fetchUserProfile(userId) {
  const response = await fetch(`/api/users/${userId}`);
  const user = await response.json();
  return user;
}

// Мой комментарий:
// "Good start! Несколько предложений:
// 1. Add error handling
// 2. Add loading state
// 3. Consider timeout

const fetchUserProfile = async (userId: string) => {
  const controller = new AbortController();
  const timeout = setTimeout(() => controller.abort(), 5000);
  
  try {
    const response = await fetch(`/api/users/${userId}`, {
      signal: controller.signal,
    });
    
    if (!response.ok) {
      throw new Error(`HTTP error! status: ${response.status}`);
    }
    
    return await response.json();
  } finally {
    clearTimeout(timeout);
  }
};"

Процесс одобрения

// После всех исправлений:
// 1. Разработчик пушит обновления
// 2. Проверяю что все замечания адресованы
// 3. Ставлю "Approve"
// 4. После approval от 2+ человек -> можно мержить
// 5. CI/CD пайплайн прогоняет тесты и деплоит

Культура code review в команде

• Уважение: Комментарии конструктивны
• Обучение: Возможность поделиться знаниями
• Скорость: Review в течение 24 часов
• Перекрёстное опыление: Разные части проекта
• Автоматизация: Линтер, тесты, статический анализ