Давайте сегодня поговорим о проблемах code review
Есть две вещи, которые могут превратить любой код ревью в поле ненависти: комментарии по поводу код-стайла и фразы «а почему не …?»
Первое злит, потому что люди не видят сути самого кода. Решается добавлением линтера в билд на CI или подключением hound’a
Фразы же «а почему не …?» автоматически вызывают у людей защитную реакцию, и вместо трезвого обсуждения вариантов, все начинают огрызаться
Решается, кстати, тоже ботом, который проверяет комменты на стоп-фразы. Иногда выходит глупость, но чаще работает как надо
А какие у вас есть болевые моменты, связанные с просмотром чужого кода? И что нервирует вас, когда ваш код проверяют?
Это всегда проблема кода, а не программиста. Но отвечать «я ничего не понял, хрень какая-то» тоже не надо, как мне кажется @naorunaoru
Можно попробовать понять, что же вообще хотел сделать наш герой и предложить путь проще. Обычно соглашаются @naorunaoru
из моей практики и первая агрессию вызывала.Небольшое изменение на «а ты не думал насчет…» обычно кардинально меняет картину @from_anywhere
Потому что «а почему не» === «это же очевидно, дебил»
Не понимают, потому что не хотят понять или просто не видят объективных преимуществ?@_yukkat