Исправление чужого кода

16 July 2020
яндекс.картинки
яндекс.картинки

Автор: @eantonov
Телеграм:
Тимлид Очевидность
Обсудить:
Чат канала в телеграм

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

Вред состоит в том, что:

- Ты мог ошибиться, и на самом деле всё с кодом было хорошо. Теперь придется пускаться в объяснения с разгневанным программистом, чей код исправили, и откатывать "исправления".

- Это неуважительно по отношению к другому человеку и его труду. Человек вложил определенное количество труда в написание кода, а ты молча идешь и меняешь его без какого-либо предупреждения. Не надо так

- Даже если ты прав, ты исправил код, ты сделал проект лучше - это непродуктивно. Если программист один раз написал плохой код и за ним молча исправили, то скорее всего он и второй раз так может сделать, и третий.

Как, на мой взгляд, надо сделать:

Прислать коллеге ссылку на сомнительный, по твоему мнению, код. Объяснить что здесь тебе или непонятно или понятно, но ты считаешь, что лучше сделать иначе потому и посему. В конструктивной критике должны быть объективные аргументы, а не просто "потому что мне не нравится". Вы обсуждаете, находите консенсус как же правильно должно это выглядеть. В идеале если все необходимые исправления сделаете не вы, а тот, кто написал код. Так в этом человеке лучше закрепится понимание того как надо, и почему прошлый вариант был ошибочный.

Уважайте чужой труд, разговаривайте с коллегами, делайте лучше и проект, и коллег.