Зачем нужен вежливый тон на Code Review?
Часто разработчики пишут прямолинейно: «This is wrong, fix it». В интернациональной команде такой тон может быть воспринят как агрессия или микроменеджмент. Вежливый код-ревью показывает твой профессионализм и уважение к коллегам.
1. Как попросить изменить архитектуру или подход
Вместо директивного "You must rewrite this" лучше задать вопрос или предложить гипотезу:
- ✅ Have you considered using a factory pattern here? It might scale better if we add more types.
- ✅ I wonder if extracting this utility function into a helper file would make the code cleaner.
- ✅ Would it make sense to reuse the existing helper instead of writing a new one?
2. Указание на ошибку в логике или потенциальный баг
Даже если ты уверен в ошибке, мягкая формулировка помогает избежать защитной реакции:
- ✅ It looks like this database connection isn't closed in case of an error. Should we wrap it in a try-finally block?
- ✅ Just a heads up: this endpoint might return a null response if the user is unauthorized. Do we handle that on the frontend?
- ✅ I might be missing something, but won't this trigger an infinite loop if the array is empty?
3. Одобрение и похвала (Appreciation)
Не забывай хвалить хорошие решения. Это мотивирует команду:
- ✅ Nice catch! I didn't think of this edge case.
- ✅ Clean implementation, thanks for refactoring this class.
- ✅ LGTM (Looks Good To Me) — classic approval phrase.
Помните, наша общая цель — сделать продукт лучше. Используй эти фразы, чтобы сделать коммуникацию в команде эффективной и приятной.