Как критиковать чужой код или дизайн и не стать врагом народа (Модель SBI)

Чтобы давать честную критику и при этом сохранять отношения, используется золотой стандарт менеджмента — модель SBI (Situation — Behavior — Impact).

Анатомия модели SBI

Этот метод убирает из критики эмоции и оценочные суждения («ты ленивый», «ты плохой разработчик»), оставляя только факты.

S — Situation (Ситуация): Где и когда это произошло? (Контекст).

1. Цель: Напомнить контекст, чтобы собеседник вспомнил момент.
2. Пример: «During yesterday’s stand-up regarding the payment module...» / «Во время вчерашнего стендапа по платежному модулю...»

B — Behavior (Поведение): Что именно человек сделал или сказал? Только факты, без интерпретаций.

1. Ошибка: «You were lazy...» (Вы были ленивы). Это оценка личности.
2. Правильно: «...you committed code to production without running unit tests.» / «...вы закоммитили код в продакшн, не прогнав юнит-тесты».
3. Важно: Описывайте наблюдаемое действие, а не свои догадки о его мотивах.

I — Impact (Влияние): К чему привело это поведение? Как оно повлияло на вас, команду или проект?

1. Цель: Показать цену ошибки.
2. Пример: «As a result, our staging environment crashed for 2 hours, and the QA team couldn’t test the release.» / «В результате наш тестовый сервер упал на 2 часа, и команда QA не смогла протестировать релиз».

Финальная формула: «When [Situation], you [Behavior]. The impact was [Impact].»

Примеры применения (Примеры + Перевод)

Кейс 1: Критика коллеги-разработчика (Peer Review) Вам нужно сказать коллеге, что его Pull Request сложно читать.

1. Агрессивно (по-русски привычно): «Ты написал спагетти-код, ничего не понятно, переделывай». (Звучит как атака).
2. По матрице SBI:"Hey Alex, regarding the PR #402 that you submitted this morning (Situation). I noticed there are no comments in the calculateTax function and variable names like n and m (Behavior). Because of that, it took me almost 30 minutes to understand the logic instead of 5, which might slow down the whole team’s review process today (Impact). Could we rename them to taxRate and add a docstring?" (Перевод смысла: Твои переменные непонятны, я потратил полчаса вместо пяти, это тормозит всю команду. Давай переименуем их).

Кейс 2: Design Review (Дизайн) Критика макета от дизайнера.

1. SBI Example:"On the checkout page mockup (Situation), the ’Confirm Payment’ button is grey and looks disabled (Behavior). This might confuse users into thinking they can’t pay, leading to lost sales at the final step (Impact). Can we try making it green?" (Последствие: Потеря продаж. Бизнес-повод для изменений найден).

Как принимать критику (Receiving Feedback)

Для русскоязычного человека первая реакция на критику — защищаться («Да это не баг, это фича!»). Золотое правило: Слушайте до конца. Не говорите «Yes, but...».

1. Listen fully. Не перебивайте, даже если они неправы.
2. Say «Thank you». Это сбивает агрессию дающего.Phrase: «Thanks for flagging this. I didn’t see that perspective.»
3. Phrase: «Thanks for flagging this. I didn’t see that perspective.»
4. Clarify (Уточняйте). Если сказали туманно: «Can you give me an example of when that happened?»
5. Iterate. Согласитесь с фактом, даже если не согласны с выводом.Пример: «You’re right, I did miss that deadline. I’ll set up reminders next time.»
6. Пример: «You’re right, I did miss that deadline. I’ll set up reminders next time.»

Методика REAL-SPEECH Екатерины Шитовой: Тон голоса (Prosody)

В нашей школе английского мы учим, что How you say it важнее, чем What you say. В русском языке мы часто используем нисходящую, жесткую интонацию для важных вещей. В английском деловом общении критика должна звучать спокойно и немного вверх (вопросительно-вовлекающе), а не как приговор судьи.

1. Ошибка: Сказать «Your code has a bug» с падающей, тяжелой интонацией (Sounds like a death sentence).
2. Правило: Используйте Collaborative tone. Вместо «You need to fix this», скажите «We should probably refactor this part to avoid future tech debt». Использование We (Мы) вместо You (Ты) снимает персональную атаку.
3. Вместо «You need to fix this», скажите «We should probably refactor this part to avoid future tech debt». Использование We (Мы) вместо You (Ты) снимает персональную атаку.

Критика — это данные, а не нападение. Используя SBI, вы переводите разговор из плоскости «Ты против Меня» в плоскость «Проблема против Нас». Вы описываете факт, а не ставите диагноз личности.