Иногда встречаются такие ситуации, когда один программист видит, что другой программист написал код не очень корректно и молча исправляет его. Он думает что совершил хорошее дело, исправил ошибку, сделал проект лучше. Однако на самом деле он принес намного больше вреда, чем пользы.
Вред состоит в том, что:
– Ты мог ошибиться, и на самом деле всё с кодом было хорошо. Теперь придется пускаться в объяснения с разгневанным программистом, чей код исправили, и откатывать “исправления”.
– Это неуважительно по отношению к другому человеку и его труду. Человек вложил определенное количество труда в написание кода, а ты молча идешь и меняешь его без какого-либо предупреждения. Не надо так
– Даже если ты прав, ты исправил код, ты сделал проект лучше – это непродуктивно. Если программист один раз написал плохой код и за ним молча исправили, то скорее всего он и второй раз так может сделать, и третий.
Как, на мой взгляд, надо сделать:
Прислать коллеге ссылку на сомнительный, по твоему мнению, код. Объяснить что здесь тебе или непонятно или понятно, но ты считаешь, что лучше сделать иначе потому и посему. В конструктивной критике должны быть объективные аргументы, а не просто “потому что мне не нравится”. Вы обсуждаете, находите консенсус как же правильно должно это выглядеть. В идеале если все необходимые исправления сделаете не вы, а тот, кто написал код. Так в этом человеке лучше закрепится понимание того как надо, и почему прошлый вариант был ошибочный.
Уважайте чужой труд, разговаривайте с коллегами, делайте лучше и проект, и коллег.
Автор: @eantonov
Телеграм: Тимлид Очевидность
Обсудить: Чат канала в телеграм
