Как работаешь над негативным фидбэком?

Работаешь над негативным фидбэком профессионально: внимательно слушаешь, разбираешь конкретные проблемы, не защищаешься, находишь решение, просишь дополнительные рецензии и учишься на ошибках.

1. Процесс обработки критики

// Кто-то дал критику на code review:
// "Это нечитаемый код, очень сложный и медленный"

// Шаг 1: Не защищаться, слушать внимательно
const myReaction = {
  // ❌ НЕПРАВИЛЬНО
  defensive: 'Но это работает же! Вы просто не понимаете',
  aggressive: 'Напишите сами, если умеете',
  dismissive: 'Мне не важно',

  // ✅ ПРАВИЛЬНО
  curious: 'Спасибо за замечание. Можешь указать конкретные примеры?',
  humble: 'Я согласен. Давайте разберёмся вместе.',
  constructive: 'Какой подход ты бы предложил?',
};

// Шаг 2: Уточнить критику
const questions = [
  'Какая именно часть нечитаемая?',
  'Какие конкретно операции медленные?',
  'Какой результат ты от меня ждёшь?',
  'Есть ли примеры лучшей реализации?',
];

2. Анализ конструктивной критики

// Code review: "Слишком много nolint комментариев"

// Правильное отношение к критике:
const analysis = {
  // Шаг 1: Понять точку зрения рецензента
  understand: 'Почему это проблема? Nolint снижает надёжность кода.',

  // Шаг 2: Согласиться с основанием
  acknowledgement: 'Ты прав, это запах кода',

  // Шаг 3: Найти решение вместе
  solution: 'Давайте рефакторим нарушаемое правило вместо nolint',

  // Шаг 4: Действовать
  action: 'Я переписываю этот участок без nolint',

  // Шаг 5: Попросить повторно
  feedback: 'Проверь пожалуйста, исправил ли я правильно?',
};

3. Разделение между полезной и бесполезной критикой

// Полезная критика (constructive)
const constructiveFeedback = [
  {
    feedback: 'Циклическая сложность > 20, можно разбить на функции',
    action: 'Извлечь логику в отдельные функции',
    impact: 'Код станет понятнее',
  },
  {
    feedback: 'Здесь нарушена цветовая палитра (используется красный вместо оранжевого)',
    action: 'Заменить на `text-accent-orange` из палитры',
    impact: 'Консистентность дизайна',
  },
  {
    feedback: 'Можно использовать useMemo для оптимизации',
    action: 'Обернуть вычисления в useMemo',
    impact: 'Производительность улучшится',
  },
];

// Бесполезная критика (destructive)
const destructiveFeedback = [
  {
    feedback: 'Мне не нравится это имя переменной',
    problem: 'Субъективная критика без ясной причины',
    action: 'Спросить: почему не нравится? Непонятна ли переменная?',
  },
  {
    feedback: 'Я бы написал это совсем по-другому',
    problem: 'Нет конкретной проблемы, стиль может отличаться',
    action: 'Спросить: есть ли конкретная проблема в текущем подходе?',
  },
];

4. Как отвечать на критику в code review

// Сценарий: критика на code review

// ❌ Плохые ответы
const badResponses = [
  'Нет, это правильно', // Игнорирование
  'Так принято в нашем проекте', // Отговорка
  'Это микрооптимизация', // Защита
  'Не важно для конечного пользователя', // Dismiss
];

// ✅ Хорошие ответы
const goodResponses = {
  gratitude: 'Спасибо за точку зрения',
  understanding: 'Я понял, это важно для читаемости',
  commitment: 'Согласен. Сейчас же переделаю',
  question: 'Предложишь ли ты лучший подход?',
  update: 'Обновил код согласно замечанию, посмотри пожалуйста',
};

// Пример идеального ответа:
const perfectReview = `
Спасибо за замечание! Ты прав, код действительно сложноват.
Я переделал функцию validateUser, разбив на отдельные:
- validateEmail (проверка email)
- validatePassword (проверка пароля)
- validateAge (проверка возраста)

Циклическая сложность упала с 18 до 7.
Посмотри пожалуйста на обновлённый PR.
`;

5. Работа с критикой на собеседовании

// Интервьюер критикует твой код:
// "Этот алгоритм O(n²), можно O(n log n)"

// Правильный ответ:
const interviewResponse = {
  // 1. Не защищаться
  reaction: 'Спасибо за замечание',

  // 2. Показать понимание
  understanding: 'Ты прав, O(n²) для большого датасета непомерно.',

  // 3. Показать желание учиться
  learning: 'Можешь подсказать оптимальный подход?',

  // 4. Слушать и конспектировать
  listening: 'Слушаю внимательно и записываю',

  // 5. Запомнить на будущее
  remember: 'Спасибо, запомню на будущее',

  // 6. Спросить о деталях
  deepDive: 'Почему именно этот алгоритм подходит?',
};

6. Поиск паттернов в критике

// Если слышишь одну и ту же критику от разных людей:
// "Твой код сложный" (от 3 рецензентов)
// -> Это не случайность, а реальная проблема

// Что делать:
const patternAnalysis = {
  // 1. Признать паттерн
  acknowledge: 'Это повторяющееся замечание',

  // 2. Найти корень проблемы
  root: 'Может быть, я недостаточно рефакторю?',

  // 3. Создать план улучшения
  plan: [
    'Проходить курс по Clean Code',
    'Просить рецензии сосредоточиться на читаемости',
    'Использовать инструменты анализа (ESLint, SonarQube)',
    'Переделать старый код как практику',
  ],

  // 4. Отслеживать прогресс
  tracking: 'После 2 недель пересмотреть, улучшилось ли',
};

7. Эмоциональное управление при критике

// Часто критика вызывает эмоции:

const emotionalManagement = {
  // Шаг 1: Взять паузу
  pause: 'Не отвечай сразу, подожди 10 минут',

  // Шаг 2: Осознать эмоции
  recognize: 'Я чувствую обиду / стыд / гнев - это нормально',

  // Шаг 3: Разделить критику и личность
  separate: 'Критика кода != критика меня как человека',

  // Шаг 4: Сфокусироваться на фактах
  facts: 'Что конкретно неправильно? Как исправить?',

  // Шаг 5: Действовать
  action: 'Улучшу код и покажу результат',

  // Шаг 6: Благодарность
  thanks: 'Спасибо, что помог мне стать лучше',
};

8. Документирование критики для себя

// Веди лог критики:

const feedbackLog = [
  {
    date: '2024-01-15',
    reviewer: 'John',
    feedback: 'Используй useCallback вместо функций в пропсах',
    learned: 'Мемоизация хуков важна для performance',
    status: 'applied',
    nextTime: 'Проверять пропсы на колбэки',
  },
  {
    date: '2024-01-20',
    reviewer: 'Sarah',
    feedback: 'Типизирование неполное, пропущены generics',
    learned: 'TypeScript generics помогают переиспользованию',
    status: 'applied',
    nextTime: 'Всегда использовать generics для reusable компонентов',
  },
];

// Через месяц посмотри: исчезла ли эта критика?
// Если нет -> это слабое место, нужно работать

9. Превращение критики в действие

// Критика: "Нет тестов для этого компонента"

// Немедленное действие:
const actionPlan = {
  // 1. Спросить детали
  question: 'Какие тесты нужны? (unit / integration / e2e)',

  // 2. Написать тесты
  testing: [
    'test: должен рендериться с пропсами',
    'test: должен обработать клик правильно',
    'test: должен показать ошибку на некорректные данные',
  ],

  // 3. Достичь 90%+ coverage
  coverage: 'npm run test:coverage',

  // 4. Обновить PR с тестами
  update: 'Tests added, coverage 95%',

  // 5. Попросить переревью
  request: '@reviewer Переделал согласно замечанию',
};

10. Долгосрочное развитие на основе критики

// Не игнорируй критику, учись на ней:

const longTermGrowth = {
  quarter: 'Q1 2024',
  feedbackThemes: [
    'Типизация: слишком много any',
    'Производительность: ненужные ререндеры',
    'Доступность: нет alt и aria-labels',
  ],

  plan: [
    'Прочитать книгу "TypeScript Deep Dive"',
    'Пройти курс по React performance',
    'Изучить WCAG 2.1 requirements',
  ],

  result: 'Q2 2024: критика на эти темы исчезла',
};

Золотое правило: Критика — подарок, который помогает расти. Профессионалы благодарны за замечания и быстро их исправляют. Любишь критику — растёшь быстро.