Замечания к чужому коду: как не поссориться

Замечания к чужому коду: как не поссориться

Проверка кода ссорит команды чаще, чем споры о технологиях, и обычно из-за формы, а не содержания.

Разделяйте уровни: «сломается на пустом списке» — блокирующее замечание, «можно короче» — вкусовое, и его стоит помечать как необязательное. Когда всё написано одним тоном, придирка к именам читается как обвинение.

Пишите про код, а не про человека. Разница между «функция делает три вещи сразу» и «ты опять всё смешал» кажется мелочью ровно до момента, когда это прочитают двадцать человек.

Объясняйте причину: замечание без объяснения — приказ, с объяснением — обучение.

Больше десяти правок — не пишите, зовите на пятнадцать минут разговора.

И отмечайте хорошее: проверка, из которой приходят только претензии, вызывает глухое сопротивление у всех участников.

IT Киров — t.me/it_kirov_news