Замечания к чужой работе: как их формулировать
Ревью — место, где рабочий разговор чаще всего превращается в личный. Обычно из-за формы, а не из-за содержания.
Разделяйте обязательное и вкусовое явно: «здесь потеряется ошибка» и «я бы назвал иначе» — замечания разного веса. Вперемешку они превращаются в шум.
Объясняйте причину: «так не делают» не учит ничему, «при пустом списке упадёт» — учит.
Спрашивайте, когда не понимаете, вместо того чтобы предполагать худшее: у странного решения часто есть причина, которой вы не знаете.
Не вываливайте тридцать замечаний разом: после десятого включается защита. Много правок — пятнадцать минут голосом.
И держите в голове цель: чтобы код стал лучше и чтобы автор в следующий раз сделал так сам.
IT Пенза — t.me/it_penza
IT Пенза
@it_penza