Как принимать критику своего кода
Получать правки к своему коду неприятно по умолчанию. Учимся реагировать так, чтобы расти, а не защищаться.
Критика кода — не критика вас. Замечание к коду означает, что код можно улучшить, а не что вы плохой инженер.
Отделяйте себя от кода
Главная ошибка — воспринимать правки как личную атаку. Код, который вы написали, — не вы. Чем раньше вы это усвоите, тем спокойнее будете на ревью. Сеньоры получают десятки замечаний и не расстраиваются — для них это рабочий процесс, а не оценка личности.
Базовая реакция: благодарность и вопрос
| Замечание | Защитная реакция (плохо) | Рабочая реакция (хорошо) |
| «Тут можно проще» | «Мне и так норм» | «Спасибо, поправлю» / «А чем проще лучше — производительностью или читаемостью?» |
| «Здесь баг» | «У меня работает» | «Точно, не учёл пустой ввод. Чиню.» |
| Спорное [nit] | молча игнорировать | «Я бы оставил так, потому что …. Но если важно — поменяю.» |
Когда вы не согласны
Принимать критику не значит соглашаться со всем. Если замечание ошибочно — спокойно объясните контекст, которого ревьюер мог не знать. Спорьте аргументами, а не эмоциями.
Шаблон несогласия:
«Спасибо за замечание. Я сделал так специально, потому что [причина].
Если я упускаю [риск], который ты видишь — давай обсудим.
Готов поменять, если так понятнее команде.»Как работает под капотом
Защитная реакция — это древний механизм: мозг трактует критику как угрозу статусу и включает «бей или беги». Поэтому первый порыв — оправдаться или огрызнуться. Приём прост: сделайте паузу перед ответом. Несколько секунд хватает, чтобы префронтальная кора перехватила управление у эмоции. На письме это проще, чем в голосе: можно перечитать ответ перед отправкой и убрать колкости.
Превратите ревью в обучение
Каждое замечание — бесплатный урок от более опытного. Заведите привычку: на повторяющуюся правку («ты часто забываешь обрабатывать ошибки») заведите личный чек-лист и проверяйте себя до отправки PR. Так вы перестанете получать одно и то же замечание.
Частые ошибки
- «У меня работает». Классическая отговорка; работает у вас ≠ работает везде.
- Спор ради спора. Если замечание про вкусовщину — часто дешевле согласиться.
- Молчаливое сопротивление. Проигнорировать комментарий хуже, чем возразить открыто.
- Принимать всё подряд без понимания. Если не поняли «почему» — спросите, иначе повторите ошибку.
Итог
- Код — не вы; критика кода не равна критике личности.
- База: поблагодарить и при необходимости уточнить «почему».
- Не согласны — возражайте аргументами и контекстом, спокойно.
- Пауза перед ответом гасит защитную реакцию; повторяющиеся правки — в личный чек-лист.