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

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

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

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

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

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

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

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

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

Итог

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

{{ message }}

{{ 'Comments are closed.' | trans }}