В период моей работы в агрохолдинге нам поставили задачу внедрить 1С:УПП. Масштаб огромный, экспертизы внутри не хватало, поэтому руководство решило пригласить внедренцев со стороны, да и переход сам по себе непростой — перенос старой системы, обучение сотрудников и всё такое.
Нашли крупного интегратора, нам они понравились и был заключен договор. Со стороны подрядчика организовали две рабочих группы (все имена вымышлены и совпадение чисто случайное, ну почти все).
- Руководитель-аналитик Надежда, опытный разработчик и ещё один аналитик, Александра. Эта группа занималась бухгалтерией.
- Руководитель-аналитик Тимур и два разработчика.
Проект стартовал и потихоньку начал двигаться. Со скрипом, с нажимом, но пошёл. Подрядчик что-то писал, учил наших бухов и всё не торопясь двигалось.
Мне довелось поработать с первой группой, во главе с Надеждой. Чувствовалось, что она специалист опытный. Умеет рулить и отстаивать перед заказчиком свою точку зрения. В принципе, так и должен действовать руководитель. Где-то сломать процесс, который был, и сделать его более правильным с точки зрения учёта, где-то оставить как было и доработать — но всё должно увязываться в единую систему. С ней работал опытный разработчик, с которым я быстро нашёл общий язык, и девушка Александра. Александра была отличным аналитиком-методистом, у нее был хорошо подвешен язык, и на этом проекте она только начинала пробовать себя в разработке.
Мне поставили задачу контролировать код разрабатываемых модулей на 1С. Тогда в 1С не существовало тестов и devops. Всё ручками. Ну а я же максималист / правдоруб 🙂
Сказали проверяй код какого-то готового блока. Да без проблем! Открываю конфигуратор и погнали. И что-то с первых строчек как-то стало не очень… Я открыл Word и начал добавлять замечания. 1, 2, 3, 4, а я всё добавляю и добавляю. В какой-то момент пункты перевалили за 20. Причём некоторые ошибки прям, ну совсем от джуна. Запросы в цикле для высоконагруженной системы, какие-то нелепости, мёртвый код, который никогда не выполнится, обилие закомментированных кусков и т.п.
Провёл ревью и в конце написал что-то в духе:
Вы крупный интегратор, вы занимаетесь автоматизацией и обслуживаете крупных клиентов. Непозволительно использовать в продакшн такой код. Это непрофессионально.
Сейчас бы я так никогда не сделал. Я бы встретился с руководителем, обсудил, поговорил. Попытался бы оценить насколько там вменяемый человек и после этого принял бы решение как быть дальше. Идти к руководству, или мы бы поговорили и они бы исправили все замечания — и дело с концом.
Но тогда я не видел проблемы. Искренне думал: раз наша компания платит (немалые деньги), то я, как представитель заказчика, имею полное право требовать от исполнителей качества.
Эх, что после моих последних строк началось! Надежда была в ярости:
Виталий! Александра написала заявление на увольнение, это она делала этот код! Зачем ты так написал?! Что теперь делать с внедрением? Человек плачет, хочет уволиться. Она преподавала, но решила пойти в разработку, а ты так унизительно отозвался о её профессиональных качествах.
Я тоже не остался в долгу и спросил зачем они поставили писать код человека, который совсем нулевой? Почему опытный разработчик, который был с ними в команде, не провёл своё ревью? Мы разругались тогда с ней очень серьёзно. Вплоть до того, что потом не разговаривали.
А дальше опытный разработчик с их стороны всё поправил и проект поехал дальше. Но шёл с огромным скрипом. А через время я уволился.
Всё было бы просто, если бы это был конец истории. Но нет.
Моя супруга была учителем математики и информатики и хорошо общалась со своей бывшей учительницей Ириной Александровной, или просто Ириной, а теперь уже коллегой. А как известно, муж и жена — одна сатана. Мы поехали на учительский сбор, там я познакомился с ней и с тех пор как-то завязалось наше общение. Она была лет на 10 старше, но молода душой. Любила турпоходы и готовила прекрасные торты. То в гости пришли, то в кафе встретились, как-то так поддерживали связь.
Через время Ира родила дочь и в годик решила её покрестить. Позвала меня крёстным. Я между прочим спросил, кто будет крёстной? Но внятного ответа не получил, да и не обратил тогда на это особого внимания. Чувствуете куда катится история? )))
Да. В день крещения, та-да-ам! Надежда будет крёстной! Я прям запомнил этот момент. Мы оба стоим и смотрим друг на друга. Немая пауза и не озвученный вопрос с обеих сторон: ты что здесь делаешь? 🙂
Но всё обошлось. Мы позже поговорили. Время и неловкость момента сбили весь тот негатив, который между нами был. Всё стало нормально.
К чему эта история? ИТ-мир тесный. А Земля и вовсе круглая. Сегодня ты разносишь чей-то код, а через пару лет стоишь с этим человеком у купели.
После той ситуации я стал по-другому подходить к ревью. Не то чтобы стал добрее, просто понял, что за каждым куском кода стоит живой человек. Можно написать «запрос в цикле, исправить» и получить тот же результат. А можно написать «непрофессионально» и получить заявление на увольнение и испорченные отношения на годы.
Если вижу слабый код от подрядчика, стараюсь сначала поговорить с руководителем. Обсудить, понять контекст. Замечания — к коду, не к человеку. Звучит банально, но мне потребовался скандал и случайная встреча на крестинах, чтобы это до меня дошло.
