Code Review на английском: от «LGTM» до критики
В IT английский — это инструмент выживания. Можно идеально знать грамматику, но если в Pull Request написать сухое "Your code is wrong", вас запомнят как токсичного специалиста. Проблема: Русский стиль слишком прямолинеен для Запада. Там критика кода воспринимается лично. Если вы написали комментарий без смягчающих конструкций, американец решит, что вы считаете его плохим разработчиком. Анатомия фидбека (Feedback Sandwich) Критикуйте код через «бутерброд», чтобы не обидеть коллегу: Positive: The overall approach looks solid. Constructive: However, we have a potential race condition here... Encouragement: Mind taking a look at X? Let me know if you need help. Лексикон Code Review: Nitpick. Мелкие придирки к стилю (пробелы, нейминг). Пример: "Just a nitpick, but could we rename this variable?" Edge Case. Пограничный случай (нулевой файл, отрицательное число), ломающий логику. Пример: "Let's add an edge case test for zero-byte files." Tech Debt. Грязный код ради скорости сейчас, за который придется платить позже. Пример: "We can ship it now, but we're accumulating tech debt." Scope Creep. Задача растет сама собой (просили кнопку — прислали переделку БД). Пример: "This looks like scope creep. Let's merge the bugfix first." Как принимать критику своего PR Не защищайтесь фразой "But my way was faster!". Это звучит агрессивно. Правильная реакция: "That's a fair point. I'll benchmark both approaches." или классическое "Good catch!". Главное правило Знать слова ≠\neq= Уметь говорить. Чтобы уверенно объясняться и не испытывать пауз при защите архитектуры, нужна практика живой речи. Профессиональную лексику можно выучить по списку, но навык мгновенного ответа под давлением тренируется только говорением. В нашей школе мы делаем упор именно на разговорную практику (80% времени занятия), чтобы убрать языковой барьер навсегда.