Замечания к чужому коду: форма важнее содержания

Замечания к чужому коду: форма важнее содержания

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

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

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

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

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

Отмечайте хорошее.

И держите темп: код, висящий на проверке три дня, устаревает, а автор всё это время не может двигаться дальше.

IT Ярославль — t.me/yaroslavl_it