Я в одиночку отрефакторил 15 тысяч строк легаси. Это были худшие две недели в жизни



    Несколько месяцев назад я работал в аутстафе. Это не то место, где нужен энтузиазм и вера в великую цель проекта. Меня вместе с командой просто продавали заказчикам, а на митингах было важно, сколько тикетов я закрыл. Приступы перфекционизма — скорее вредная штука для такого места, но я ничего не мог с ними сделать. За один из них я знатно поплатился, попал в адский кранч и провел худшие две недели в моей жизни.

    Моя компания попросила меня приехать на две недели в Москву, в главный офис, поработать вместе с командой. Просьба разумная, я согласился. За несколько дней до отъезда я взял очередной тикет. Нужно было добавить фичу к SDK для десятка проектов, над которыми мы работаем. Эта SDK — непролазное легаси, которую в здравом уме никто не трогает. Там такое болото, что даже прожженные инженеры суеверно крестятся — лишь бы все работало, как работает. Когда нам нужно что-то пофиксить или добавить фичу, мы осторожненько подпираем код ещё одним костылём, и не дыша отправляем её в руки CI и QA.

    Я планировал поступить так же. Суть задачи была простая. Сама SDK — обёртка над очень умным сишным бинарём, который умеет поднимать VPN. Она же общается с нашим беком, получает всякие конфиги, авторизует пользователя, фетчит список стран для VPN и так далее.

    Мне просто нужно было добавить ещё один запрос на бэкенд, который получает список серверов. Выглядит это крайне просто — добавил к публичному API метод, положил внутрь запрос на нужную урлу, сляпал модель, распарсил, отдал. Потом научил клиентские приложения использовать этот запрос, и всё готово. Я проэстимировал таску в два дня.

    Очень большая ошибка. Не бывает простых задач. Одним глазом я уже перебирал московские бары, где мне надо выпить все пиво, другим — увидел первую проблему. Мы не просто получаем список серверов, мы ещё используем один из них для того, чтобы поднимать VPN-туннель. Раньше мы использовали для этого только страны. Но какой-то гений решил, что будет отличной идеей хранить страны как список строк. Само собой, сущность VPN node — слишком ёмкая штука для того, чтобы уместить её в одну строку.

    В итоге в SDK набралась куча методов, которые принимают строку (код страны) и что-то там делают, а теперь все эти методы должны принимать код страны, или объект сервера — и что-то там делать. Всё, что мы делали в коде SDK и приложений со странами, теперь нужно делать и со странами, и с серверами.

    Когда я это понял, то почему-то обрадовался. Скоуп задачи теперь позволял мне слегка отрефакторить болото, полное говна, в котором я всё время плаваю. Я заявил о своем желании, и теперь точно знаю — мне дали добро с выражением, как будто впарили инвалидку по цене Мазератти. Я согласовал недельный бюджет и принялся за дело.



    До отъезда была пара дней. Неделя же выглядела как целая бесконечность. Я решил, пока не буду писать код, просто утрясу решение в голове, а как приеду, быстренько все раскидаю.

    Когда нормальный человек едет в командировку, он заранее покупает билеты, решает вопрос с жильём, предупреждает жену, и вообще всё делает правильно. Я недальновидный идиот, который поперся в Москву с женой и двумя детьми. Жену пришлось брать, потому что я рассказал ей про поездку за три часа до отъезда. За два часа — только начал искать водителя с минивэном, потому что моя машина сломалась. Квартиру я снял в тот же момент. Шесть часов пути с орущими детьми и недовольной женой закончились так, как и должны были — квартира оказалась ровно в две тысячи раз меньше, чем выглядела на фотках. Хлеще всего был запах — там воняло травой. Горничная, которая нас принимала, сказала, что это они травили тараканов (тараканов!!!). Жена ей поверила, а говорить жене, что я отличаю запах травы от тараканьего яда, не в моих интересах.

    Я пришёл в офис, мне выдали ноут. Пол дня я устанавливал среду, со всеми знакомился. Когда всё скачалось и установилось, начал работать. Злоба от ужасного переезда еще не остыла. Я открыл проект с SDK, в очередной раз крепко выругался про себя, и подумал: «А какого хера? Да тут всё к чертям собачим надо выкидывать, и переделывать».

    Я сказал об этом лиду, а он, конечно, ответил: «Согласен, фигачь». Проект на 15 тысяч строк, каждая из которых — преступление против человечества. Я собрался его исправить за две недели. А будь у меня десять часов дороги и трое детей, что бы я предложил лиду? Изобрести за неделю новый ЯП специально для проекта?

    Но мне показалось, если я приведу в порядок хоть что-то — мне станет легче. Сейчас распинаю этот говнокод, сделаю всё как надо, заживём. Я когда-то читал книги про то, как нужно производить глубокий рефакторинг. Там было много умных мыслей про то, что ты расписываешь шаги, пишешь тесты, продумываешь свои действия так, чтобы на каждом шаге иметь уверенность — проект работает, ничего не сломано. Книги и лучшие практики, это очень круто, но я никогда не учусь на чужих ошибках. В самом начале карьеры я очень много читал про всякие базовые штуки, но не верил в них, пока мне не понадобилось расширить функционал своего первого проекта.

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

    Свой рефакторинг я стал делать не опираясь на практики. Я мысленно разделил проект на большие логические куски. В его структуре файлов это не было отражено, поэтому я снёс эту структуру файлов к чертям собачьим. Насоздавал папочек верхнего уровня: вот тут у нас работа с бэком, тут работа с VPN, тут хелперы, тут исключения. Существующие юнит тесты я тоже забраковал. Тогда мне казалось, что говнокод не имеет права на жизнь, а сейчас я понимаю — я выпрыгнул из самолёта, и не взял парашют, потому что его уложили с нарушением инструкции.

    Я начал переносить говнокод из старых папочек в новые, попутно приводя его в порядок. Ничего особенного, просто следуешь хорошим практикам. Есть класс, описывающий модель пользователя, у него все свойства опциональные, все с сеттерами. Превращаешь его в пацанскую иерархию наследования. Всё ридонли, всё через конструктор. Так, чтобы пользователи этого класса всегда точно знали, что у них в руках. Есть говнометод на четыреста строк — бьешь на сабметоды. Видишь файл с тысячей обязанностей — превращаешь в тысячу файлов с одной обязанностью. Есть класс, у которого метод Initialize, который всегда надо вызывать сразу после создания класса — применяешь паттерн «состояние»

    Понятия хорошего и плохого кода очень размытые. Я всю свою жизнь буду учиться различать их. Но вот откровенно дерьмовый код я могу видеть уже сейчас. Видеть и исправлять. Я старался не сильно перелопачивать архитектуру верхнего уровня и как можно меньше менять внешнее API SDK.



    Задерживаться на работе на пару часов, когда тебя ждёт жена в чужом городе, в самой худшей халупе в мире, а у неё двое маленьких детей, которые от стресса ревут не переставая — плохое решение. Я задержался на семь часов. В первый же день работы я просидел в офисе до часа ночи и переписал от силы процента два.

    «Дома» меня ждал заслуженный скандал, утром — дерьмовый растворимый кофе. Я снял квартиру в пяти минутах пути от офиса, но мне было так странно и страшно в этом ужасном городе, что первые три дня я ездил на работу на такси.

    В офисе все были очень радостные — народ собирался пить вечером. У этих москвичей так каждый день. Меня тоже позвали. Я обещал постараться, хотя сам точно знал — нет опций. Снова принялся за работу.

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

    Быстро решил, что мой вчерашний подход слишком компромиссный. Если уж переделывать, то вообще всё. Я перестал ограничиваться переименованиями и разбиениями. Каждый файл, который я собирался переработать, подвергался тщательному анализу. Я уже довольно долго на этом проекте, я хорошо понимаю, чего тут хочет бизнес, и что должен делать наш код.

    Но здесь я стал постоянно встречать код, который я не понимаю — какую проблему он решает и зачем вообще был написан. И я принял решение — если я не понимаю, зачем нужен код, значит код не нужен. Я с радостью стал отправлять этот цифровой мусор в преисподнюю.

    Такой подход породил новые проблемы. Публичное API проекта было неправильным. Там было много методов c нелепой сигнатурой (например метод принимает строку и объект, который содержит ту же строку). У многих методов были идиотские названия. И в целом, если используешь эту SDK с тем API, которое у неё было — можешь сломать работу приложения десятью тысячами способов.

    К этому моменту я зашёл уже достаточно далеко. Проект не собирался, и не было шанса, что он будет собираться в ближайшие несколько дней. Всё, что я в нём меняю, я смогу проверить только в конце работы. А раз я поменял уже так много, почему бы и не поменять тогда публичное API? Хорошо, мне хватило ума сделать копии репозиториев с SDK и приложений, так я мог дублировать изменения API в этих копиях и находить ошибки. Ведь основную версию SDK я даже не мог скомпилить.

    Проблемы начались сразу. Изменил сигнатуру одного метода? Лови плюс две сотни ошибок времени компиляции в приложениях. Из-за неправильных сигнатур методам передавались неправильные параметры, а приложения работали. При этом ты работаешь в четырёх окнах IDE. После того, как я поменял три метода, и вылечил все ошибки с ними, подумал — раз работа уже проделана, глупо будет поменять только часть дурацкого API — надо переделывать дальше. И полез в процесс еще глубже.



    Мне постоянно приходилось принимать решения, по большей степени этические. Вот я знаю, как должно что-то работать, и вот я вижу код, который должен это делать. Я читаю его и понимаю, что он этого не делает. Но этот код в проде, он прошёл QA. И кто тут дурак? Как я могу брать на себя эту ответственность? Поначалу я долго думал над каждым таким кейсом, консультировался с лидом и командой. Но это занимало слишком много времени.

    Я понял, что я работал медленно, потому что боялся что-то сломать, пытался где-то разобраться, быть аккуратным. Тогда я сказал себе: «Я умнее чем тот, кто писал это говно, я знаю, что должно быть сделано, у меня есть задача и бюджет на рефакторинг и я имею право на всё». Разбираются и переживают слабаки, мужчина знает чего хочет, и делает, не задавая лишних вопросов.

    В какой-то момент я вошёл в странное состояние робота, который методично перерабатывает один файл с кодом за другим. Ничего не проверяя, ни в чём не сомневаясь. Вижу плохой код — меняю. Это по цепочке поднимает новые проблемные места, чиню их. В каждом заменяю говнокод на что-то сносное.

    Это похоже на блиц в шахматах. Ты играешь в игру, где самое главное — продумывать все возможные варианты, но у тебя так мало времени, что ты не можешь даже подумать, какие варианты стоят того, чтобы продумать их. Я всегда проигрываю в блиц. За второй день я сделал очень много. Вот только код, который ты не можешь проверить — это не код. И я об этом знал.



    В один из вечеров я вспомнил, что обещал жене съездить в Икею. Квартира, в которой мы жили, была ужасна, и жена хотела напокупать всякой уютной херни. Я весь день писал код, мой мозг был в край измотан, но ехать пришлось. Поставил каршеринг, ближайшая тачка была в двадцати минутах ходьбы. Я вызвал такси, чтобы доехать до машины. Представьте себе типичный московский тупичок, забитый автомобилями под завязку. В конце такого меня ждал Рено Каптур. Победив лагающий UI делимобиля я смог открыть это ведро. Там меня ждал сюрприз — коробка автомат. Я никогда не ездил на автомате, но подумал, что справлюсь. Если хочешь ехать вперёд, надо передвинуть ручку вперёд, если назад — назад. Нет. Я понял, что машина едет не туда ровно в тот момент, когда ещё можно было спасти ситуацию.

    Вдавил тормоз. Естественно, не той ногой, ведь в дурацком автомате всего две педали, и мой мозг всё понял неправильно. Дикий рывок, тачка встала. Два миллиметра до мерседеса S-класса.

    Кое-как я выполз из тупичка, и понял — вокруг меня все машины мира, а я понятия не имею, куда и как мне ехать. Я доехал до «дома», забрал жену, полночи ездил по самому ужасному городу на земле, пропустил десяток поворотов, оставил целое состояние в икее.

    Мы вернулись часа в четыре утра, мой мозг отключился, как только глаза увидели кровать.



    В восемь утра я снова был на работе. Там снова все счастливые, обсуждают свои московские дела и бары, до которых мне не добраться. У меня снова SDK. С тяжелой головой я бегло просмотрел изменения, которые наделал. Всё очень плохо. Уже завтра митинг по таске, а мне нужно ещё три сотни лет, чтобы всё закончить.

    У меня есть хорошее умение — когда я получаю большой тикет, я всегда декомпозирую его на несколько маленьких. Отсюда моё рабочее флоу. Один тикет, один-два коммита, один пулл реквест. Поэтому я в принципе не умею в процессе кодирования разбивать задачу на несколько коммитов — мне это обычно не нужно. Но стресс меня добивал, и умение подвело. В процессе работы я превратил маленькую задачу в гигантскую, но начал над ней работать так, как работал бы над маленькой. Без системы, тупо где увидел проблему, там и починил. Не стал дробить процесс на логические шаги, а просто как последний джун хватался за всё сразу.

    Сделанного не воротишь. Главная проблема оставалась — я не мог собрать и запустить всё решение, и понятия не имел, как оно заработает. Это очень пугало, в последний день я посмотрел на диффы и понял, что я переписал почти весь проект. Было чертовски страшно.



    Все всегда сползает на последний день. Командировка кончалась, мое растянутое время на рефакторинг — тоже. Жена позвонила и сказала, что нас выселяют в час дня, хотя я договаривался на поздний вечер. Естественно, это означало, что доработать последний день до конца я не мог.

    Вместе с тремя галлонами холодного пота до меня доходило то, что должно было дойти, когда я взялся за этот таск, задумал рефакторинг, согласился на командировку, взял детей, снял притон на две недели — я самый главный идиот на всем белом свете. Разбираться в моих мотивах стоит психиатрам, но я решил идти до конца. То ли чтобы самому себя наказать, то ли на какой-то иррациональной вере, что все делаю правильно.

    С горящей жопой я мёржу всё за час, запускаю CI и сваливаю домой. Всю дорогу я прокручиваю в голове свой код и я абсолютно уверен — ничего не выйдет. Слишком много изменений, будет сотня багов, и мы откажемся от этого рефакторинга.

    Когда я вернулся домой, меня ждала новость — проект работает. Лид поправил пару мелочей, мои изменения прошли автотесты и прямо сейчас успешно проходят ручное тестирование. Я не поверил. Спулил код на домашнюю тачку, собрал и вручную проверил все сценарии, которым не доверял. Оно работало. Работало!!!

    Я был так горд собой. Качество кодовой базы выросло в разы, появилось несколько фич, я выпилил кучу багов, до которых ещё не успели добраться тестеры. Либой можно было комфортно пользоваться, а я доказал себе свою профпригодность.



    Порывы энтузиазма и самопожертвенного труда — удел наивных глупцов. Я был бы рад доказать себе этой историей обратное. Но через несколько дней после возвращения меня ждала еще новость.

    Заказчик решил отказаться от проекта. Почти сотня человек в аутстафе (половина всей компании) оказалась «в ожидании нового проекта». Считай — лимбически безработными. А удаленщиков при таких делах режут в первую очередь. Мне сказали, переезжай в Москву насовсем и переходи на другой проект. Либо мы тебя уволим.

    Я спросил, что будет со старым проектом.

    В такие моменты люди становятся болезненно честными. Лесть, приукрашивания и корпоративная этика отпадают. Мне сказали, этот код больше никому в целом мире не нужен, отправлен на свалку истории. Самими продуктами ещё очень долго будут пользоваться, но поддерживать кодовую базу не станет никто. Всё вот это качество и читаемость кода, масштабируемость, удобство в использовании, однозначность поведения — всё сделано зря.

    Я слушаю и понимаю, что мог просто сделать фичу и ходить по барам. Но я сидел ночами, не спал, терпел скандалы ради хорошей кодовой базы, которая больше никому не нужна. Энтузиазм — лживая штука. Ты говоришь, что готов работать, даже если в этом нет выгоды. Но когда тебе говорят, что твоя высшая безвозмездность действительно не приносит выгоды — это почему-то убивает. Ты-то надеялся изменить своим порывом всю вселенскую неправильность.

    Мне кажется, я все отлично понимал с самого начала. Просто я не хотел быть прагматичным парнем.

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

    Подробнее
    Реклама

    Комментарии 401

      +7

      Приятно читать Вашу статью! Когда пишу костыли стараюсь отрефатчить именно эту часть, чтобы для следующего костыля пришлось тратить меньше сил и не сломать остальное самим рефатчем

        +1
        Вот именно так и надо делать, а не заниматься глупостью, как автор статьи.
      • НЛО прилетело и опубликовало эту надпись здесь
          +4
          Насчёт перезда, и денег жалко было, и переезжать не хотелось. Для детей и так был огромный стресс засыпать на новом месте.

          Тег поставил.

          Легаси было на шарпах.

          Вот самый дикий кейс, что я там встретил выглядел примерно так
          Есть методы. которые забирают что-то с апишки. Внутри них зачем-то вызывается универсальный механизм запроса на дженериках, который возвращает нужный инстанс модели ответа.
          Типа такого:
          class UserResponse 
          {
            string Name {get; set;}
            ...
          }
          

          Так вот, во время запроса что-то могло пойти не так, и в этом случае этот универсальный механизм создавал инстанс модели ответа через РЕФЛЕКСИЮ. Причём не всегда, несколько видов ошибок он умел обрабатывать, и плеваться исключениями.

          public async Task<T> FetchData<T>(string url)
          {
            try
            {
              return await SomeHelper.Fetch<T>(url);
            }
            catch(SomeExeption e)
            {
              throw new SomeAnotherException(e);
            }
            catch(Exception)
            {
              return (T)Activator.CreateInstance(typeof(T));
            }
          }
          


          Это, конечно, создавало массу проблем. В клиентском коде часто невозможно было отличить валидный ответ от невалидного, классы респонсов не могли иметь конструкторы с параметрами, и я не мог использовать ООП, что бы моделировать возможные типы ответов (есть же не просто валидный и не валидный, валидных типов ответов в нескольких запросах могло быть по пять штук).

          Вообще лучший способ понять качество кода, это посмотреть, насколько хорошо в этом коде организована работа с исключительными случаями.
          • НЛО прилетело и опубликовало эту надпись здесь
              0
              Клиентов всех мы сами разрабатывали, я поменял контракт и во всех клиентах переделал вызовы методов либы.
              +2

              Мы с детьми всю жизнь ездим по миру, не очень понимаю от чего стресс. Вот стресс от 6 часов в автомобиле — это я бы понял, я стараюсь сделать так, чтобы было комфортно. Но квартира? Мы тут недавно прокатились по 5 городам Италии за 12 дней с детьми 10, 3 и 0 лет. Не понимаю откуда стресс.

                +3
                И в каждом вы работали с 8 утра до 24, а жена искала где Икея?
                  0

                  Всякое бывало. Но не надо путать "переживают из-за того что папы нет" и "переживают из-за непривычного места". В первое я верю, во второе не особо. Хотя в целом я считаю что это художественное произведение, не имеющее отношение к релаьности.

                    +1

                    У жены, слава богу, есть такси и яндекс карты. Искать икею не надо — она есть на карте.

                      –1
                      Я однажды поехал в Икею на прокатной машине в новом для меня городе. По дороге обратно в телефоне с картами села батарейка, и мы пару часов ночью пытались выехать и найти дорогу домой.
                        +1

                        В автомобилях обычно есть 12В розетки (они же бывшие прикуриватели). С их помощью можно заряжать смартфоны. Не благодарите. :)

                          0
                          Спасибо, теперь буду знать!
                        0
                        Дети слишком мелкие — с ними в одиночку очень тяжело куда-то ездить
                          0
                          asm0dey писал, что у него из мелких годовасик и трехлетка. Куда уж мельче. Если трехлетку еще можно оставить с няней, то грудничка очень сомнительно… так что вероятно у него жена сама перемещается куда нужно. Хотя возможно в Италии это делать проще…
                            +1

                            Уточним: я летал с детьми практически в любом возрасте: от 1 месяца до 10 лет. Но никогда не летал дальше чем на 4 часа. Я живу в мск, мне удобно, в 4 часах от меня вся Европа кроме Испании и Португалии. Летаем только без пересадок чтобы не усложнять. Никогда не перемещаемся быстро между городами. Поживём 2-3 дня, потом на поезд в следующий поезд. Иногда подолгу живём в одну месте — 5-8 дней.
                            Не живём в отелях, живём к вартирах, снятых через AirBnb — там есть стиралка, кондиционер и всё необходимое. Едим в ресторанах/кафе, но иногда покупаем что-то домой в продуктовых. Особенно если продуктовые поблизости есть.
                            С няней/бабушками никого не оставляем. Дети ничем не заслужили такого бросания с моей точки зрения. Он итакие же полноценные члены семьи как я и имеют права на развлечения и на то, чтобы смотреть новое не меньше чем я.

                              0
                              А это режим отдыха или работа? Т.е. в этих поездках работать приходится?
                                0

                                Не, это отдых, конечно. Ну не то чтобы я в отпуске не работаю, но это не является доминирующим видом деятельности. Я ж не враг своей семье — возить их куда-то для работы. Когда я работаю мы и дома можем посидеть или там на даче или в арендованном доме в подмосковье.

                      +2
                      Мы с детьми всю жизнь ездим по миру, не очень понимаю от чего стресс.

                      Стресса у вас и детей нет, так как для вас это норма жизни. Если кто-то никогда никуда не ездит, то для него и выезд на дачу бывает со стрессом
                        0

                        Тогда о чём думали родители детей, которые так жестоко поступили?

                          0

                          О том, что пора бы, наконец, куда-нибудь съездить, а тут и возможность подвернулась, например?

                            0

                            Ну тогда не на что жаловаться, м?

                        0
                        Люто плюсую. Мы переехали с одного побережья США на другое (дважды) с ребенком и ничего. И это не турпоездка, а именно переезд. И с работой во время.
                          +1

                          Хвастаетесь?

                        0
                        Легаси было на шарпах.

                        Опа, сишарп уже легаси. =8-[ ]
                          +3
                          Сам ЯП то тут причём? Это код на нём легаси, а шарп вполне себе современный
                            0

                            Даже сам шарп может быть легаси почти 20 летней давности:) я не исключаю, что то код, который я писал тогда ещё крутится

                      +80
                      «Жену пришлось брать, потому что я рассказал ей про поездку за три часа до отъезда.»
                      "«Дома» меня ждал заслуженный скандал, утром — дерьмовый растворимый кофе. "
                      " жена хотела напокупать всякой уютной херни."

                      Дружище, сочувствую тебе. Жена должна помогать, а не мешать. Ладно, она не понятно о чем думала, но о чем думал ты? Брать жену с 2 мя детьми в новый город, попутно увеличивая свой стресс. Отрефакторь лучше свои легаси отношения с женой. Она должна быть хранительницей очага и надежным тылом, а не источником дополнительного пиз#еца.

                        +17
                        В моём мире, жена мне ничего не должна.
                          +47
                          стало быть и ты ей «ничего не должен»? тогда зачем вы вообще друг другу?
                          не пойми неправильно — просто люди сходятся, влюбляются, женятся, заводят детей не только ради галочки «у меня семья — я молодец», а потому что вдвоём выживать в этом мире проще, чем поодиночке… «У каждого меча должны быть свои ножны» (с)
                            +38
                            Я немного в замешательстве. Я упомянул про жену в статье, что бы объяснить своё тогдашнее состояние, а не для того что бы узнать, что люди думают про мой брак. Видимо, это не очевидно. Поэтому я вот так прямо и скажу — мне не хочется знать чужое мнение про мою жену, и я не думаю, что это допустимо.
                              +10

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

                                +54
                                Если не хотите, чтобы вам советовали на форумах, не надо жаловаться на форумах.
                                  +5
                                  так автор вроде и не жаловался. И помощи не просил) Помощь сама нашла кому помочь)
                                    +2
                                    А вы помните, что значит «жаловаться»?
                                    Это были худшие две недели в жизни
                                    это не «жаловаться»?
                                +17
                                Тогда не надо было вообще про семью упоминать. Чего теперь удивляться, что люди начали комментировать, то, что написано в статье (про брак)? Статья вообще о чем? О ваших чувствах/эмоциях или о рефакторинге?
                                  +5
                                  Интересно, если бы я и про дочь упомянул, мне бы заодно объяснили бы, что и дочь у меня дурацкая? Статья — история, для истории важно, в каком я был состоянии. Очевидно, если бы всё было как обычно, я бы и не взвалил на себя неподъёмную задачу, а сделал бы что-то минимально необходимое.
                                    +7
                                    Вполне может быть. Ведь не упоминается просто наличие, рассказывается про какие-то взаимодействия, причем с явно негативным окрасом.
                                    Не стоит сначала выносить личное на публику, а потом кручиниться о том, что люди это обсуждают с позиций своего опыта.
                                    Это все равно что человек, сняв штаны, потряс голой задницей на публике, а потом ругается на то, что кто-то это снял и выложил на youtube. Но он сам вынес что-то на публику…
                                    Я не оправдываю тех, кто дает вам советы по поводу как вам жить. Просто странно ожидать иного.
                                      –12
                                      Не знал, что где-то существует правило, по которому я даю согласие на критику всего, что так или иначе выношу на публику. В любом случае, у других людей есть право говорить что угодно, так же как и у меня. И вот я говорю, что они, и их опыт, который они вынесли на публику — говно собачье.
                                        +10
                                        Не знал, что где-то существует правило, по которому я даю согласие на критику всего, что так или иначе выношу на публику.

                                        Ваше согласие здесь не играет никакой роли. ПМСМ, это очевидно — люди обсуждают то, что видят и слышат — у них, как вы сказали, есть право говорить что угодно. Вынося что-то свое в публичное пространство вы делаете так, что люди это что-то видят, и, соответственно, имея право говорить что угодно, обсуждают. Простая причинно-следственная связь: люди не обсуждают того, чего не было (не видели). Если человек не хочет, чтобы какие-то его действия обсуждали, он не выносит их на публику.

                                        Здесь можно долго спорить о терпимости и т.п., но… если человек, выйдя в дождь без зонта просто ворчит на свое растяпство — это одно. Если он не понимает причинно-следственной связи между прогулкой без зонта и мокрой одеждой, кляня при этом небеса — это другое.

                                        И вот я говорю, что они, и их опыт, который они вынесли на публику — говно собачье.

                                        Впору вспомнить социальную рекламу, в которой девчушка, запостившая свою фривольную фотографию, видит насмешки других, а саму фотографию — видит потом на всех стендах. Безуспешно срывает ее раз за разом. В конце надпись: «помни — однажды закинутое в сеть остается там навсегда».
                                          0
                                          О оно играет роль для меня. Я же о своём восприятии говорю.
                                          +5
                                          > Не знал, что где-то существует правило, по которому я даю согласие на критику всего, что так или иначе выношу на публику.

                                          Это правило прописано в конституциях большинства стран. Называется «свобода слова».
                                            0
                                            Свобода слова про то, что тебя не могут наказывать силой закона за слова. Говорить тебе, что ты должен заткнуться могут — ведь свобода слова не только у тебя есть. Вот например внизу чувак оставил комент, в котором наоскорблял меня лично, но у него мало кармы, и я мог отклонить его комент — вот в моём понимании, если бы я отклонил — это было бы нарушение свободы слова. А принять, и сказать, что он мудила — тут всё нормально.
                                    +1
                                    Потому что страна советов, habr хоть и com, но всё же ru. Плюньте.
                                      +7

                                      Не обращайте внимания эти на комментарии. Я например понял зачем Вы упоминали о "семейном" аспекте ситуации. И вопросов или предложений по улучшению качества семейной жизни у меня не возникло.

                                        +14
                                        Мне не нравится переход на семью автора с которого эта ветка началась, но я вот честно вообще не могу понять зачем брать родственников или знакомых в, по факту, рабочую командировку?
                                          +3
                                          Мало ли может быть причин. Может у них любовь все дела? Такое тоже случается… Или вот: не смотрели фильм «Смертельные узы»? На героях там были такие ошейники, что им нельзя дальше километра друг от друга отдаляться иначе взорвутся. В общем какая нам разница: надо было — пришлось взять. Я периодически в командировки езжу, с удовольствием взял бы жену с ребенком в Питер, к примеру, ей нравится город. Но как-то все не до того. Да и командировки у нас коротковаты — дней 5.
                                            +1
                                            Ну как по мне командировка и так обычно стресс, да и предполагается что в командировку едут решать вопросы которые иначе не решаются или решаются плохо/медленно. Будь у меня жена мне бы даже в голову не пришло ей предлагать со мной поехать. Скорее наоборот, попросил бы пореже отвлекать. А даже если бы пришло в голову и не удалось попросить заранее — то сказал бы приезжать когда ей удобно будет, а не собираться теряя тапки.
                                            0
                                            Командировки разные бывают. Конкретно в эту (на целые две недели?) и правда непонятно, дополнительный стресс и для себя, и для жены, плюс и для детей. А в более расслабленные короткие поездки я брал, чтобы город посмотреть, погулять.
                                              +4
                                              Автора никто не спрашивал, судя по тексту.
                                                0
                                                Я не москвич, но не понял также негатива к городу — я бывал в Москве на рок-концерте, и мне город очень понравился. Может не в нем дело, а в самом авторе?
                                            +5

                                            Лол.


                                            Одному проще, потому что ты гарантированно знаешь, что ты не сможешь не оправдать ничьи ожидания. Тьфу ты, много «не», но суть понятна, надеюсь. Если у вас нет собаки, её не отравит сосед.

                                              +18
                                              У вас слишком большая боязнь не оправдать чужих ожиданий в любых отношениях (даже если это просто поболтать). Не надо так.
                                              Как человек, который имеет друзей и жену и при этом уже давно переставший пытаться оправдывать чужие ожидания говорю. Пока вы пытаетесь оправдать чужие ожидания (будь то родители с их списком вещей, что ты должен по их мнению делать; или друзья, которым ты прям обязан чем-то помогать, когда они о тебе вспомнили; девушка с точки зрения которой ты обязан соответствовать её виденью идеального партнера), так вот, пока вы это делаете — успехов достичь не получится, не говоря уже о том, как ваши попытки будут ещё и обесценивать постоянно.

                                              Вы должны быть собой, а остальные либо найдут плюсы в общении с вами, либо…
                                                +5

                                                Есть некая картина мира, которую можно кратко (и неточно) подытожить как «а нафига я всем этим людям, кроме как их ожидания оправдывать?» Рационально понятно, что в реальной жизни это не всегда так, но тут-то речь идёт об эмоциональной сфере, а как её приводить в соответствие, непонятно.


                                                Тем более, что все мои взаимодействия с окружающим миром за последние лет 10 (сколько я более-менее стараюсь себя осмыслять) были в рамках этой парадигмы. У работодателя есть ожидания, и в зависимости от уровня их оправдывания я получаю тот или иной результат. У преподов в вузе есть ожидания, и в зависимости от уровня их оправдывания я получаю ту или иную оценку. У ревьювера в опенсорс-проекте есть определённые ожидания, и в зависимости от их оправдывания мой пуллреквест принимается, отклоняется или дорабатывается. И это представляется разумным (особенно если принимать во внимание, что ты не обязан все ожидания оправдывать).


                                                Так что да, одному проще.

                                                  +6
                                                  У работодателя ожидания связаны с полученным от вас резюме. Он же не ожидает от вас, что вы вдруг станете суперменом и пр?, Преподаватели в вузе имели не ожидания, а требования, разные для каждой отметки и вы либо выполняли оные, либо не соответствовали планке.
                                                  В личном плане по большей части от вас требуется быть адекватным тому, что вы сами о себе говорите. И тому, что вы сами из себя демонстрируете. Все другие ожидания оправдывать вы совершенно не обязаны, это даже вредно во многих случаях. Когда находишь человека, перед которым не надо лезть из шкуры, чтоб оправдать его ожидания — то внезапно это понимаешь. У моей жены ровно одно ожидание от меня — что я буду таким, каким был эти 2 года. Я все эти 2 года был просто сам собой и не требовал от неё быть другой, чем она была со мной всё это время. И за два года у нас не было ни одной серьезной ссоры или размолвки, что для меня лично не характерно, так как я как раз не люблю, когда меня заставляют соответствовать чужим безосновательным ожиданиям.
                                                    0
                                                    У работодателя ожидания связаны с полученным от вас резюме. Он же не ожидает от вас, что вы вдруг станете суперменом и пр?

                                                    Есть много вещей, которые не упомянуты в резюме. От участия в on-call-ротации до уважения принятых в коллективе/компании критериев неявной субординации и готовности войти в положение продолбанного менеджмента и покранчить перед релизом. Или готовность менять стек, например.


                                                    Преподаватели в вузе имели не ожидания, а требования, разные для каждой отметки и вы либо выполняли оные, либо не соответствовали планке.

                                                    Я, если честно, не вижу большой разницы между понятиями «ожидания» и «требования», а так да, согласен.


                                                    Но, опять же это всё даже не особо важно, ведь разговор именно что формулируется в терминах требований (или ожиданий, это не столь важно) и зависимости вашего профита (в широком смысле) от выполнения этих требований.


                                                    Когда находишь человека, перед которым не надо лезть из шкуры, чтоб оправдать его ожидания — то внезапно это понимаешь.

                                                    Если потребность (или привычка) рассуждать в терминах соответствия чему бы то ни было не идёт изнутри.


                                                    я буду таким, каким был эти 2 года

                                                    Гы, интересно, а вот то, что 6 лет назад на шестом курсе я угорал по машинному обучению и был перспективной партией с зарплатой 300k/nanosec, три года назад случайно взял алгебраичный учебник по линалу, и теперь уже всерьёз думаю в следующем году (хотя не, GRE сдать не успею, так что через год) поступать в вуз на PhD в существенно менее перспективную область, с соответствующей сменой стиля жизни, это считается?

                                                      +2
                                                      Неа. Ведь как раз в этом ваша суть. И загоняетесь вы по умным логическим вещам, что с точки зрения некоторых людей — будет плюсом, потому что им как раз этого, например, и не хватает. А доходы… это такая странная тема. Вы ж не будете работать в макдаке за еду? Если кому-то от вас нужны лишь доходы, то возвращаемся к вопросу, а зачем вам соответствовать чужим ожиданиям? Или считаете, что у учителей, преподов в РФ нет пар или они все несчастливы в браках?
                                                      PS. Ладно, я завершу офтопик…
                                                        0
                                                        Вы ж не будете работать в макдаке за еду?

                                                        Ну, судя по быстрому гуглу, зарплата макдональдсе — 22к в год. Аспиранту в UPenn платят стипендию в районе 27 в год. Так что...


                                                        Если кому-то от вас нужны лишь доходы, то возвращаемся к вопросу, а зачем вам соответствовать чужим ожиданиям?

                                                        А тут тот же вопрос с парадигмой, в которой вы (или я) привыкли жить.


                                                        PS. Ладно, я завершу офтопик…

                                                        Да и мне ныть хватит. Спасибо за фидбек.

                                                      0
                                                      У моей жены ровно одно ожидание от меня — что я буду таким, каким был эти 2 года.

                                                      Это странно, поскольку отношения — это совместное развитие, а не стагнация. Получается, что зачем они тогда нужны, если развития нет.

                                                        0
                                                        Это и есть развитие, если только у вас жена — не алгоритм, ожидания у которого будут строго постоянные. Люди меняются все и всегда, а не то, что вы меняетесь, а ваша жена — нет. Или наоборот.
                                                          +2
                                                          Смотрите, во время ухаживаний или общения изначального люди стремятся соответствовать. После свадьбы вдруг оказывается, что человек не такой. Через год — развод.
                                                          Понятно, что мы оба развиваемся и меняемся, но это надстройка базы по сути. А не то, что ты казался успешным, стремился соответствовать мат.ожиданиям, а потом, достигнув, сел на попу ровно.
                                                            +1

                                                            Если заявил "на берегу" что-то вроде "моя карьерная цель стать хорошим тимлидом (со хорошей для рынка оплатой)", то имеешь полное моральное право и перед женой/мужем, и перед работодателем, достигнув этой цели не то чтобы просто сидеть на попе ровно, но отказываться от других карьерных возможностей хоть в сторону CTO/VP, хоть в сторону архитектора, хоть ещё куда-то, развиваясь только чтобы соответствовать меняющимся требованиям к этой конкретной позиции на рынке, чтоб быть уверенным, что не зависишь от конкретного работодателя.

                                                      0
                                                      Предположим, вы будите абсолютно категоричны в этом подходе, — абсолютно никакой подстройки под партнера. Тогда получается, что вашими партнерами могут быть только те, кто старается оправдать ожидания, а не настаивать на своих. Тк всех остальных вам придется отвергнуть. По вашему утверждению, те кто оправдывает ожидания, не достигнут успеха. Получается что ваши отношения смогут быть построены только с человеком который не достиг успеха, а это вроде плохо?

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

                                                        С теми, кто не достиг, но пытается — тоже хорошо, но нелегко. Не всегда понятно, достигнет ли вообще когда-нибудь успеха.

                                                          0
                                                          Предположим, вы будите абсолютно категоричны в этом подходе, — абсолютно никакой подстройки под партнера.
                                                          Скажите, а вы уже соответствуете тому, каким должен быть мужик? И делаете по списку то, что мужик должен?
                                                          А ещё всю жизнь делали то, что вам говорили родители? Ведь вы же им должны.
                                                          Тогда получается, что вашими партнерами могут быть только те, кто старается оправдать ожидания, а не настаивать на своих.
                                                          Мои партнером не может быть тот, кто знает что и как я обязан был делать вне моих желаний и возможностей.
                                                          По вашему утверждению, те кто оправдывает ожидания, не достигнут успеха.
                                                          Где, где вы это вычитали?
                                                          Получается что ваши отношения смогут быть построены только с человеком который не достиг успеха, а это вроде плохо?
                                                          Не получается.
                                                          Веду я к тому, что между отстаиванием своих желаний и подстройкой должен быть баланс у обоих партнеров, тогда они смогут достичь гармонии.
                                                          Вопрос был в боязни не оправдать ожидания, которых человек, во-первых, ещё не знает, а во-вторых вовсе не обязан оправдывать.
                                                          На свете куча людей и у всех свои ожидания. Вы можете автоматически оправдать ожидания одного и никогда — другого, но невозможно всегда оправдывать желания всех вокруг. И это не нужно.
                                                            0
                                                            Тогда получается, что вашими партнерами могут быть только те, кто старается оправдать ожидания, а не настаивать на своих. Тк всех остальных вам придется отвергнуть.
                                                            Это утверждение корректно только в случае если автор, не пытающийся оправдать чьих-либо ожиданий, сам навешивает кучу ожиданий на партнеров.

                                                            Здесь включается второй принцип, который хорошо сочетается с первым: «чем меньше ожиданий – тем меньше разочарований». Если не выдумывать слишком большой список ожиданий к партнеру, то и отвергать всех не прийдется.
                                                              0
                                                              Здесь включается второй принцип, который хорошо сочетается с первым: «чем меньше ожиданий – тем меньше разочарований». Если не выдумывать слишком большой список ожиданий к партнеру, то и отвергать всех не прийдется.
                                                              Бинго! Какая замечательная формулировка)
                                                          0

                                                          Свои ожидания всегда можно не оправдать.

                                                            0

                                                            Но про это-то только я знать буду.

                                                        +12
                                                        Квартира, в которой мы жили, была ужасна, и жена хотела напокупать всякой уютной херни.
                                                        Вы всегда обставляете уютной херней отели в которых побываете во время отпуска, съемные квартиры(в которых планируете жить меньше 2х недель), может быть даже самолет в котором летите на отдых?)

                                                          +3

                                                          Я сначала подумал, что уютную херню предполагалось отвезти в место постоянного проживания, в город, в котором нет икеи…


                                                          Оу. Нет, это действительно для того, чтобы обставить двухнедельную квартиру.

                                                            0
                                                            Ну так мы же все эти вещи потом домой забрали
                                                              +1
                                                              Это безусловно все прекрасно, но мне кажется у вас имеются проблемы с планированием времени, иначе довольно сложно объяснить поездку за покупками посреди рабочей недели да еще и с возвращением домой в 4 утра. В спб кстати икея работает до 11-12, сомневаюсь что в мск она работает круглосуточно, по этому опять же тут что то не сходится. Про доставку из икеи я вообще наверное промолчу.
                                                              Не поймите меня неправильно, я вас не осуждаю и не критикую, но ход ваших мыслей (а особенно то, что вы пишете в комментариях выше) мне кажется странным и как для психолога и как для человека.
                                                                +2
                                                                Я объясняю почему я так сделал. В смысле, я не думаю, что в этой истории я хоть что-то сделал правильно, но я отлично понимаю, почему я поступал так как поступал. В следующий раз я просто бы никуда не поехал. А если бы поехал, то без жены. А если бы поехал с женой, то остановился бы у друзей. И рефакторить бы ничего не взялся, пока не попросили бы. А рассказал я про всё это, потому что из всего надо делать выводы, и не только мне.
                                                                  0
                                                                  А если бы поехал с женой, то остановился бы у друзей.
                                                                  Ехать по работе и останавливаться у друзей?
                                                                    0
                                                                    Почему нет?
                                                                      0
                                                                      1. Вы уверены, что вашим друзьям это не помешает?
                                                                      2. Неужели ваш работодатель не оплачивает постой в рабочих поездках?
                                                                0
                                                                а, фух…
                                                          +11
                                                          но о чем думал ты?

                                                          по-моему эту мысль автор сам повторил раз 10 по ходу статьи.
                                                            +4
                                                            странно что минусуют — чувак-то дело говорит
                                                              +10
                                                              Может и дело, но не по делу.
                                                              Личные отношения любого из авторов с его семьей, мне кажется, должны оставаться за рамками обсуждения на хабре. Это не фэсбук и не вконтакте, мы тут все не близкие родственники и друзья чтобы давать советы личного характера. И как я уже отмечал все свои промахи автор сам обозначил в статье. Но хороший тон не должен позволять комментатору повторять «ты мудак дурак», даже если или, может даже, в особенности если автор себя так назвал.
                                                              А какое такое «дело говорит», кроме как наезд на личные отношения автора (которые никого не касаются), мне не очень понятно.
                                                              Можно было прокомментировать например: «а вот моя жена, когда мы были в командировке мне в офис борщ горячий привозила». Смысл тот же, но от подачи многое меняется.
                                                                +15
                                                                Личные отношения любого из авторов с его семьей, мне кажется, должны оставаться за рамками обсуждения на хабре.

                                                                Включая текст статьи.


                                                                Можно было прокомментировать например: «а вот моя жена, когда мы были в командировке мне в офис борщ горячий привозила».

                                                                Забавно. Мне это кажется более, эм, агрессивной формой.

                                                              0
                                                              Аххахах, расскажите как это сделать, очень нужно.
                                                                0
                                                                Как будто у жены не может быть своих планов?
                                                                  +2

                                                                  А теперь еще дома его будет ждать скандал за эту статью, так как упомянул здесь супругу и многих это зацепило

                                                                    0
                                                                    «Дома» меня ждал заслуженный скандал
                                                                    Не совсем заслуженный, на самом деле. Почитайте, что пишут фрилансеры и те, кто работает из дома. Просто один к одному. До членов семьи очень медленно доходит, как это так ты «не дома, а на работе», когда вот же ты — дома.
                                                                    Возможно, автор не объяснил, что после работы его снова ждёт работа.
                                                                      +3
                                                                      Ага, помню что мать вообще не понимала про то, что если я сижу дума из-за плохого самочувствия, то это не означает, что я свободен, а мог просто отпроситься поработать из дома. В итоге еще и обижалась на то, что я отказываюсь помочь по дому в рабочее время.
                                                                      Но это вообще от многого зависит. Так вот некоторые люди не могут понять, что если человек сова, то будить его в его законный выходной в 8-9 утра с криками «всю жизнь проспишь» — не надо.
                                                                    +8

                                                                    Вы выбрали не тот язык, чтобы писать. Вам нужно выбрать обычный русский :)

                                                                      +3
                                                                      Мои искренние соболезнования.
                                                                      Я знаю что в следующий раз все повторится.
                                                                      Себя не переделать.
                                                                      Вернее переделать себя можно и нужно, но это занимает многие годы.
                                                                      И нет никакого мерила того насколько глубоко надо лезть в рефакторинг, но и работать с говнокодом год за годом очень токсично и вредно. Я тут пару лет назад узнал термин «yak shaving».
                                                                        +9
                                                                        Зачем переделывать себя, если можно изменить окружение?

                                                                        В продуктовой компании, где проекты не выбрасываются так спонтанно, за проделанный труд могли и премию выписать. Просто работать на аутстафе/аутсорсе таким как автор вредно для ментального здоровья.

                                                                        Основная «проблема» автора – острый перфекционизм, как глубоко внутреннее чувство несовершенства окружающей действительности. Задавшись высокоморальной целью хоть немного исправить эту действительность, автор жертвует собственным комфортом и комфортом своей семьи. При этом, с точки зрения окружающих действия автора выглядят как сизифов труд, абсурдный по своей сути, но от которого автор не в состоянии отказаться.

                                                                        Я считаю что попытки давить перфекционизм в себе так же губительны в долгосрочной перспективе и приводят к депрессии, ведь индикатор несовершенства никуда не исчезает, вы просто перестаете предпринимать попытки исправить ситуацию. Как результат, возникает непреодолимое чувство тщетности бытия, как противоположность ловушки в которую угодил автор – сизифов труд, который оказался нужен только ему, и то исключительно с точки зрения удовлетворения внутренних побуждений.

                                                                        Если давить перфекционизм в себе нельзя и слепо следовать ему тоже нельзя, можно попытаться направить его в конструктивное русло. Например, вместо того чтобы быть «идеальным» программистом (работающим только с идеальным кодом), можно стать «идеальным» менеджером (достигающим наилучшего результата за отведенное время). При этом не обязательно менять название должности, достаточно быть внутренним менеджером самого себя. Тогда жизнь начинает потихоньку налаживаться, вроде бы.

                                                                        Этот комментарий отредактирован 7 раз.

                                                                        </диванная психология>
                                                                          0
                                                                          Кстати, а ещё можно заняться теоретической физикой, и использовать свою жизнь на рефакторинг современных знаний. Например, попытаться согласовать четыре фундаментальных взаимодействия или обосновать существование кварков стройной математической системой. И пусть только кто-то попробует вам сказать, что вы ерундой занимаетесь.
                                                                            +3
                                                                            Например, попытаться согласовать четыре фундаментальных взаимодействия или обосновать существование кварков стройной математической системой...
                                                                            когда вам на работе дали простую задачу по расчёту вентиляционной системы)
                                                                              0
                                                                              Не, в случае с расчётом вентиляционной системы, решать уравнение Навье-Стокса в трёхмерном варианте. Правда, оно для жидкостей, но вдруг решат переиспользовать систему вентиляции как водопровод.
                                                                            +1
                                                                            А еще в продуктовой компании, в зависимости от количества средств и финансового выхлопа могли втык сделать за нецелевое расходование «средств»))
                                                                              +1
                                                                              Просто работать на аутстафе/аутсорсе таким как автор вредно для ментального здоровья.

                                                                              Я думаю наоборот. Тема раскрывается в предыдущей статье. Как раз благодаря аутстаффу автору удаётся дать отдушину перфекционизму. В продуктовой разработке сюда бы наложились ещё соображения экономической нецелесообразности подобных рефакторингов. А в аутстаффе что — раз «уплочено», то валяй, переписывай, нам-то что.

                                                                              0
                                                                              У меня на проекте есть команда с таким названием, наконец я узнал, что это значит
                                                                              0
                                                                              Интересная статья.
                                                                              Возможно имело смысл какие-то решения или подходы вытянуть в свой репозиторий, но так чтобы НДА не нарушать там всякие.
                                                                              Чтобы уж совсем труды не вылетали коту под хвост.
                                                                                +13
                                                                                К сожалению, очень часто такой код не имеет особой ценности, будучи вырванным из контекста проекта, даже если нет ограничений в виде  NDA.
                                                                                История именно о том что: ради чего выкладываться как при стойка БАМа, если никто не оценит и трудами своими потом нельзя любоваться и гордиться даже. Не говоря уже о какой-то материальной компенсации «ударного труда»?
                                                                                По факту человек отработал за 2 недели полугодовой объем команды из 3х человек, получил «дежурное спасибо» и оплату 40ч х 2 нед = 80 часов работы, выжат как лимон, ему требуется время на восстановление, к тому же чувство опустошенности, скандалы в семье и прочие последствия. На самом деле статья на Хабре в данном случае — это все что можно записать в плюс по итогу этой истории.
                                                                                  +3
                                                                                  Да, но классная статья получилась. Думаю многие нашли себя в его истории. Постоянно хочешь сделать лучше то, что «никто не хочет» и как тебе кажется «не может» сделать. И внутри у тебя просыпается супер мотивация, ты знаешь, что всех спасаешь. И это вводит тебя в состояние потока — это круто.

                                                                                  Конечно по итогу эти старания не оцениваются как важнейший вклад, и на какое-то время ты теряешь мотивацию сворачивать горы, но потом это прекрасное чувство возвращается и ты снова готов сделать этот мир лучше.

                                                                                  Конечно хотелось бы, чтоб таким поступкам давали подкрепление (премии, похвалы, не важно), но это какая-то утопия, стереотипное мышление, которое прививали нам в школе (решил контрольную правильно — получил 5).
                                                                                  Сам можешь сказать себе что делаешь все классно.
                                                                                    0
                                                                                    Может и к лучшему, что все так закончилось. Потому что после такого рефакторинга, зачастую появляются бомбы замедленного действия, которые однажды срабатывают со всеми вытекающими.
                                                                                      +3

                                                                                      Бомбы может и появляются, но изначальный код обычно является не просто бомбой, а бомбой ядерной. Если рефакторинг был проведён более-менее грамотно, то с последствиями взрывов от появившихся бомб справится куда проще, чем от старых.

                                                                                +5
                                                                                Всего лишь 15к? Искренне рад за вас, вы еще не видели полного трещака за 100к строк, полный монолит и зоопарк из языков )
                                                                                  +15
                                                                                  И кто-то это сделал за 2 недели в одиночку?
                                                                                  Мне кажется что 15 тыс отрефакторенного кода за несколько недель в одиночном плавании, да еще не в своей привычном окружении, а на «гастролях», с успешным прохождением тестирования в конце — это реально подвиг и достижение.
                                                                                    +9

                                                                                    Думаю, все видели полный трешак. Иногда сидишь, рефакторишь, рука тянется написать куда-нибудь статью об этом. Но потом попьешь валерьянкичаю, немного отпускает.


                                                                                    Я могу наверное уже трехтомник написать о том, что в своей жизни пришлось рефакторить. Причем три раза (может больше, три раза точно помню) отказывался от работы увидев код. Два раза на фрилансе, третий раз в офисе. Вышел на работу, и пока еще не успели уладить все формальности, сказал, что ухожу сразу же. Это было 10+ лет назад, проект большой и все еще живет, что для меня удивительно. До сих пор иногда вспоминаю этот момент, решиться было непросто, размышляю, правильно ли поступил. Наверное все же правильно.


                                                                                    Во всех трех случаях ситуации были идиотские до невозможности. Допустим, есть проект на Django. Берут человека его допиливать. Этот человек начинает дописывать функциональность, но при этом в своих кусках использует не шаблоны Django, а Jinja2. То есть, в проекте начинают использоваться два разных (но очень похожих) языка шаблонов. И, допустим, с включением куска одного в другой, возникают проблемы. На вопрос: «зачем?!!» человек сказал, что ему так удобнее, и уволился. Но это были только цветочки. Когда я начал смотреть, что он там написал в Python, то пришлось уже волосы вырывать по всему телу. Кстати, на Хабр он по-моему есть. Я гуглил его из чистого любопытства — просто хотелось понять, откуда взялся такой уникум, и куда ушел… Если он этот комментарий читает, то наверняка вспомнит — ведь он наверное один такой, который пытался сделать проект, в котором используются два разных языка шаблонов. Привет, мозголомщик!


                                                                                    Ну а были случаи, когда брался и рефакторил код. Каждый такой эпизод наверное отнял от старости пару лет, и прибавил седых волос.

                                                                                      +1
                                                                                      проект большой и все еще живет, что для меня удивительно

                                                                                      Как-раз ничего удивительного. Если Вас напугала огромная кодовая база трешняка, то фокусироваться в этом надо не на слове «трешняк», а на слове «огромная» — значит, это пишут давно, этим много кто пользуется и есть все шансы, что и следующие 10 лет так оно с этим проектом и будет. Т.е. в плане стабильной работы и зарплаты — это может быть вполне себе вариант работы. В плане перфекционизма, конечно, это ад. Но тут уж кадому своё.
                                                                                        +2

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


                                                                                        Но вообще, для меня удивительно, что проект до сих пор жив, и не только потому, что код был трешовый. Уже тогда мне казалось, что сама идея бесперспективна. Это, собственно, вторая причина, почему я удивлен, что он существует по сей день. Просто не хочется вдаваться в подробности, чтобы нельзя было догадаться о каком проекте идет речь.

                                                                                        0

                                                                                        На Python не пишу, но объективно два шаблонизатора ни о чем ещё не говорят. Гораздо больше шансов на внедрение нового, если внедрять его плавно, а не "за 2 недели" переписать все шаблоны системы

                                                                                          +1

                                                                                          Немного затрудняюсь объяснить.


                                                                                          Во-первых, эти два шаблонизатора практически одинаковые по синтаксису, по скорости, и всем остальным параметрам, то есть в принципе нет ни единого резона заменять один другим. И уж точно в этом проекте разница шаблонизаторов не влияла ни на что — основные задачи и сложности там были совсем в другом.


                                                                                          Во-вторых, все сторонние приложения, все приложения самого Django сделаны на шаблонах Django. А любой нетривиальный проект на Django использует сотни батареек: для форм, для админки, для автокомплита, для REST API, и т.д. и т.п. Все эти батарейки используют исключительно шаблоны Django. То есть для использования другого шаблонизатора должны быть очень веские причины, т.к. можно потерять совместимость и начать постоянно изобретать велосипед, вместо использования готовых решений.


                                                                                          В-третьих, этот товарищ не заморочился переписыванием всех шаблонов. И даже не планировал. Просто до него было, скажем, 15 шаблонов на Django. А он просто стал писать все новые шаблоны на Jinja. То есть, его не смущала мешанина из тех и других (что особенно «приятно» с учетом почти одинакового синтаксиса, но все же отличающегося, чтобы постоянно пытаться по ошибке писать в Jinja как в Django и наоборот, и получать ошибки).


                                                                                          В четвертых, я уже не стал писать, что там было в Python коде, потому что это однозначно выдало бы его с головой, и там был просто адский кошмар. Например, этот товарищ создал абстрактный класс ORM Address. Затем унаследовал от него CustomerAddress. Затем от CustomerAddress унаследовал BusinessAddress. В итоге, получилась иерархия классов и таблиц в базе данных, которые не имеют никакого смысла, особенно в контексте задачи, где в ее определении вообще не требуется никаких разных типов адресов (не говоря уж о том, что в ORM такое реализуется без наследования).


                                                                                          Уже только по этому стало понятно, что человек до этого никогда не программировал, взял какую-то древнюю книжку-пособие по ООП, и не понимая, что там написано, стал это все применять в жизнь там, где это не нужно. Ну все помнят же эти книги с идиотскими иерархиями наследования типа Shape <- Square <- Circle или Employee <- Manager <- Boss.


                                                                                          Вообще, я о той ситуации мог бы тоже целую статью написать, у меня еще осталось, что сказать, просто не думаю, что это кому-то интересно.

                                                                                            +4

                                                                                            Ну а в целом, почему эта ситуация так мне запала в душу и запомнилась. Это фриланс проект. Меня туда очень долго решались пригласить, около полугода. Все эти полгода этот товарищ (скажем, А) хреначил свой неработающий говнокод. Он что-то там показывал, какие-то потемкинские деревни в виде HTML, за которым не стояло никакого работающего кода.


                                                                                            Заказчик (З) и его менеджер (девушка по имени И.) не обладали достаточными знаниями, чтобы понять, что им морочат голову. В итоге, когда А сказал, что он нашел работу получше, и сваливает немедленно, они в панике решились сделать мне предложение. Я согласился, не видя кода. За полгода мы много раз обсуждали задачу, и она казалась мне интересной.


                                                                                            Как только я получил доступ к репозиторию и начал работать, то сразу сказал сакраментально-банальное: «Внутри неработающий код и все придется переписать». З и И мне вроде как поверили, но А уже успел написать десятки тысяч строк. Я потихоньку начал в этом разбираться и переделывать. Но каждый час(!!!) И меня спрашивала: «Когда будет первый результат, который можно показать З?». После 4-5 вопроса это нешуточно достало, так как переписать десятки тысяч строк за несколько часов нереально — нужны недели, месяцы. Я попытался объяснить и спросил откуда такая настойчивость, почему обязательно надо дергать меня каждый час. И показала скриншот переписки с заказчиком, где он возмущается, что я уже наработал целых два часа, а результата еще нет.


                                                                                            Тут я понял, что кашу видимо не сваришь. Сказал, что возвращаю все заработанные на тот момент деньги и прекращаю контракт. У З и И случилась истерика, они обвинили меня в непрофессионализме и т.п. Хотя я успел проработать всего 1-2 дня, и отказался от денег, заработанных в это время (потерянных для меня, как и потерянного времени, когда они полгода постоянно беседовали со мной, но решались пригласить в проект — постоянные собеседования, обсуждения, вытягивание информации, как бы стал делать этот проект).


                                                                                            Это было на известной бирже фрилансеров. После этого я нашел профиль этого деятеля на бирже. Во-первых, З поставил ему наивысший рейтинг (в отличие от меня, но свою оценку я успешно оспорил). Во-вторых, этот человек поставил себе рейт в два раза выше чем я, хотя ничего не умеет, и за полгода, видимо, заработал астрономическую сумму, не сделав вообще ничего работающего, просто набросав свалку кода, которая ничего не делала и не могла сделать! В-третьих, у него в профиле была куча регалий: закончил серьезный ВУЗ, различные курсы, и швец, и жнец, и на дуде игрец...


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

                                                                                              0
                                                                                              Ну так рынок же.

                                                                                              Предыдущий работник был в тех же условиях постоянного давления, но успел и денег заработать и опыта набраться и заказчик еще доволен — win win же :)
                                                                                                0

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


                                                                                                Правда я вроде бы не занимался такими вещами, как впаривание неработающего кода со словами: «Тут все на 80% готово!», как сделал товарищ А в данном случае. Из-за чего заказчик и полагал, что осталось совсем чуть-чуть поднапрячься и проект будет готов.


                                                                                                Эта история разбередила старую рану, пошел на биржу, порылся в старых проектах и нашел профиль этого товарища. Уже успешно сделал еще несколько проектов и последние пару лет работает на одного и того же заказчика. Значит, набрался опыта и вырос, может уже писать рабочий код.


                                                                                                Потому каждый раз и останавливаю себя, когда рука тянется написать какие-то ужастики из своего опыта, пускай даже так, что ни проект, ни человека угадать нельзя. Думаю, что слишком сильно эмоционирую, да и люди часто на месте не стоят, набираются опыта, растут. К сожалению, не все. Некоторым что в лоб, что по лбу — годами пишут одно и то же, и не могут понять элементарные вещи. В таких случаях, я просто теряюсь. Человек вроде бы говорит, что понимает. Но продолжает делать по старому.


                                                                                                Дело даже не в вещах, которые субъективны. А например в том, что человек не понимает, что, например, эффективнее написать условие в запросе в БД, нежели чем вытаскивать целую таблицу в память, и проверять условие в коде. Вроде разжевываешь все неоднократно до мельчайших подоробностей. Показываешь, почему это медленно и неэффективно. Человек клянется, что прозрел и все понял. На следующий же день (!) в pull request видишь такую же ботву… В такие моменты руки опускаются.

                                                                                                  0
                                                                                                  А например в том, что человек не понимает, что, например, эффективнее написать условие в запросе в БД, нежели чем вытаскивать целую таблицу в память, и проверять условие в коде.

                                                                                                  Да ладно) Это же зависит от задачи, может там 10 записей в таблице и разницы никакой нет где там что фильтровать.
                                                                                                    0
                                                                                                    Да ладно) Это же зависит от задачи, может там 10 записей в таблице и разницы никакой нет где там что фильтровать.

                                                                                                    Иногда да. А конкретный человек, про которого пишу, иногда умудрялся написать код, который 500-600 записей из таблицы по одной таскал. Работало очень медленно.

                                                                                                      +1
                                                                                                      Сделайте таску на улучшение перформанса, а потом отчитайтесь что этот самый перформанс улучшили в 100500 раз. /сарказм
                                                                                                        0

                                                                                                        На самом деле "очень медленно" не значит ничего пока нет конкретных требований к скорости. Если реально "очень медленно", то пользователи/ПО/заказчики на это обратят внимание разработчиков. А таскать записи по одной может экономить память, например :)

                                                                                                          0

                                                                                                          Не может это экономить память, на фоне рантайма самой программы эти 600 записей просто потеряются.


                                                                                                          И вообще, для экономии памяти записи надо не по-одной таскать, а получить одним запросом и обрабатывать построчно, не собирая в массив.


                                                                                                          Если реально "очень медленно", то пользователи/ПО/заказчики на это обратят внимание разработчиков.

                                                                                                          Иногда лучше не дожидаться пока пользователи или заказчик обратит на что-то внимание.

                                                                                                            0

                                                                                                            Так потеряется экономия или не будет экономить? :)


                                                                                                            Если получить одним запросом, то они уже будут все в памяти. Скорее всего в массиве или типа того. Тут какие-то генераторы/курсоры/сессии/страницы на стороне источника данных нужны, чтобы источник данных мог отдавать результаты запроса клиенту именно построчно/постранично. Только тогда получится сэкономить память на клиенте в рамках "одним запросом". Типа пхпешного PDOStatement::fetch, если источник данных и клиентская либа поддерживают построчную передачу результатов одного запроса

                                                                                                              +2

                                                                                                              Конкретно в этой ситуации, человек, например, всегда проверяет, что в таблице есть какая-то строка таким образом:


                                                                                                              def user_has_listing(user):
                                                                                                                  result = query("SELECT listing_id, user_id FROM listings")
                                                                                                                  return user.id in result.fetchall()

                                                                                                              (упрощенный псевдокод)


                                                                                                              Или даже:


                                                                                                              def user_has_listings(user):
                                                                                                                   for user in query("SELECT user_id FROM listings").fetchall():
                                                                                                                         if user == user:
                                                                                                                             return True
                                                                                                                   return False

                                                                                                              Всегда, какие бы ни были проверки и ситуации, он пишет самый неэффективный возможный код.


                                                                                                              Тем, кто говорит, что раз нет строго определенных требований к проивзодительности, то это норм: уже была ситуация, когда этот человек реализовал несколько фич. Каждое изменение содержало несколько подобных неэффективных мест. После того, как все это попало в production, критические места катастрофически замедились, потому что множество мелких неэффективностей сложились и получился колоссальный кумулятивный эффект. А еще в продакшене данных было намного больше, так что там где O(n²) было незаметно при тестировании на тысячах строк, оно стало невыносимо медленно на сотнях тыcяч.


                                                                                                              И сто раз объясняешь ему, что эффективнее сразу в запросе проверять. Говорит: «Ох, нифига же себе! Я все понял». Через день вижу такой же код. И история повторяется по кругу.


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


                                                                                                              Тут я не знаю что делать. Если его уволить, то надо взять нового, а где его сейчас взять? Как-то замотивировать его у меня нет ни рычагов, ни влияния. Вроде пытаюсь постоянно ему показывать, как делать лучше, он со всем соглашается, но продолжает делать по-своему.


                                                                                                              Говорю ему, например: «Ты пишешь неэффективно, смотри вот хорошая статья на тему итераторов, в которой очень интересно написано, как лучше писать в таких ситуациях код». Он соглашается, исчезает, потом я снова вижу точно такой же код, как и раньше. То ли он не читал статью, то ли не понял, хотя она реально простая… Эхъ...

                                                                                                                0

                                                                                                                Ну это даже, уже перебор. Сочувствую, сам в подобной ситуации был, хотя и немного в другом ракурсе.

                                                                                                                  +1
                                                                                                                  а где его сейчас взять?

                                                                                                                  А что, проблема найти человека на обучение? Судя по всему, вчерашний студент будет лучше него, у студента хотя бы перспектива есть и обучаемость.
                                                                                                        +1
                                                                                                        «Тут все на 80% готово!»
                                                                                                        Как правило, это означает 0x80% из 0x100% :)))
                                                                                                      0
                                                                                                      Правильное таргетирование. Предыдущий разраб все их поставил правильно. Заработал деньги, рейтинг и опыт. Скорее работающий проект в его цели не входил.
                                                                                                        0

                                                                                                        Я скорее о том, что эта история благодаря этому товарищу стоила мне нескольких потерянных дней и массы потерянных нервов. Потому что объяснить двум людям: заказчику и его менеджеру, что все, что они получили — это кучу ничего не делающих бессмысленных исходников — очень тяжело. Они видели HTML формы, которые имитировали какую-то работу.


                                                                                                        Сами они ничего в программировании не понимают, хотя менеджер говорила мне, что до того, как уйти в менеджеры, занималась каким-то программированием.


                                                                                                        А так вообще просто хотел сказать, что у меня за жизнь столько примеров ужасного кода скопилось, что если начинать блог типа The Daily WTF, то моего материала на год хватит писательства регулярных статей.


                                                                                                        Чего только я в жизни не видел… Типа «эксперт» CTO нескольких австралийских компаний, который не знает, как нормализовать базы данных, поэтому в его коде, например, есть таблица с тремя (серьезно, не шучу и не преувеличиваю!) полями: active, is_active, user_is_active. Причем в разных местах в коде устанавливались и проверялись разные атрибуты: один модуль, например, устанавливает флаг active, а другой проверяет is_active. Естественно, когда я начал это нормализовать, мне пришлось долго размышлять над этой троичной логикой в попытках свести ее к одному флагу… И если бы это был единичный такой случай в его коде. Там можно было целую коллекцию насобирать. Также как и его высказываний. Например, он предлагал после выхода каждой новой мажорной версии продукта, начинать историю Git с нуля!

                                                                                                        • НЛО прилетело и опубликовало эту надпись здесь
                                                                                                            +3

                                                                                                            Когда увидел сообщение заказчика, пересланное менедежером, где он писал (почти дословно): «Этот чувак уже забиллил в клиенте биржи 2 часа, а результата еще нет. Он дорого нам стоит. Когда будут результаты?» я просто сразу понял, что лучше получить негативный отзыв и какие-то другие негативные вещи (например, менеджер забанила меня в мессенджерах и т.п., внесла в свой черный список), нежели пытаться варить с таким человеком кашу.


                                                                                                            Если он не понимает, что для нового человека в новом неизвестном проекте, два часа — едва-едва достаточно, чтобы развернуть его (установить исходники, зависимости, тестовую базу, понять откуда начать, и т.п.), и даже не всегда этого времени достаточно, а он за два часа хотел уже какой-то результат, то я не собираюсь заниматься воспитанием этого клиента.


                                                                                                            В другой ситуации, возможно, да, попробовал бы объяснить. Что за два часа ни в каком проекте сложнее hello world с нуля не получить никакой результат. Что он получил имитацию исходников. И так далее. Но в конкретно той ситуации, я понял, что все, что можно извлечь из этого проекта — огромную головную боль и отчеты каждый час. Зачем мне это надо, когда вокруг десятки тысяч адекватных заказчиков?..


                                                                                                            Считаю, что отказавшись через пару часов после начала и его не подвел тоже. Он потерял два часа времени. Я потерял намного больше. Другое дело, если бы я пару дней хотя бы его мурыжил (как он меня полгода мариновал не решаясь давать проект).


                                                                                                            Просто я понял, что с этой нервотрепкой с постоянными запросами по Скайпу «ну когда уже я увижу работающий проект», я бы просто через пару дней взорвался. Либо он зажал бы платить «дорогому разработчику».

                                                                                                0
                                                                                                Я так год рефакторил код объёмом примерно в 300к строк. Это было С++ приложение под Windows со своей кривой реализацией STL, многопоточностью, окошками. Оно не использовало ни стандартные С++ либы, ни стандартную виндовую многопоточность. Всё это было сделано с применением максимального числа антипаттернов, с кучей багов, дедлоков, ошибок и т.д. И сделано это было не для ускорения (всё наоборот тормозило), а просто потому, что те, кто всё это начинал писать, не знали об этих вещах. А потом 8 лет туда присобачивали новые костыли и велосипеды. В общем, я уволился в итоге, потому что стало противно на всё это смотреть.
                                                                                                0

                                                                                                Счастья всем, и пусть ни кто не уйдёт обиженным ©

                                                                                                  0

                                                                                                  Шедевральненько.

                                                                                                    +13
                                                                                                    До конца не верил, что проект соберется
                                                                                                      +3
                                                                                                      Да, я тоже. Я вот всё пытаюсь понять, а как люди делают такое же в динамически типизированных проектах?
                                                                                                        +3

                                                                                                        Ну, во-первых, они придумывают свой собственный маленький тайпчекер в виде юнит-тестов.
                                                                                                        Во-вторых, я бы не стал так уж обольщаться, ибо разница с точки зрения гарантий между каким-нибудь питоном или JS и каким-нибудь С++ куда меньше, чем между С++ и хаскелем или ещё более навороченными языками (при всей моей любви к С++).

                                                                                                          +9
                                                                                                          Я не столько про гарантии сколько про процесс. Вот был у меня тип A, я поменял его публичное АПИ, и получил от компилятора все места, где от этого возникли проблемы.

                                                                                                          А насчёт тестов, обычно у говнокода тесты — такое же говно, которое ничего не проверяет.
                                                                                                            0

                                                                                                            Так у них вместо компилятора юнит-тесты. Получат список тестов, которые завалились, прям как вы (или я) сообщения об ошибках от компилятора.

                                                                                                              0
                                                                                                              Допустим, у меня есть тип А, и его метод a. Есть тип В, который этот метод А#а использует. Я меняю сигнатуру A#a, чтобы вместо двух параметров принимать один. Допустим, падает два юнит-теста. Я их чиню и радостно иду дальше рефакторить. Потом ловлю ошибку в runtime, что из B метод A#a вызывается неправильно. Как в таком случае юнит-тесты помогут решить проблему?
                                                                                                                0

                                                                                                                Не знаю, я не фанат этого подхода.

                                                                                                                  0

                                                                                                                  Пишите юнит-тест на тот метод В, который падает в runtime :)


                                                                                                                  Есть еще статические анализаторы, которые такое иногда могут отловить, и выдадут ошибку. Для python есть pylint, который отработает такую ситуацию и выдаст


                                                                                                                  Too many positional arguments for function call (too-many-function-args)
                                                                                                                +2
                                                                                                                Я не столько про гарантии сколько про процесс. Вот был у меня тип A, я поменял его публичное АПИ, и получил от компилятора все места, где от этого возникли проблемы.

                                                                                                                ну так а refactoring разве не начинается с написания тестов? Если нету тестов, которые прикроют твою спину, то либо их нужно написать либо с идеей refactoring'а завязывать
                                                                                                                  +1

                                                                                                                  От языка зависит. На некоторых языках мне тесты для рефакторинга ни разу не пригождались. Юнит-тестов там и не было, а интеграционные сразу были зелёные, как рефакторинг заканчивался (после починки всех нужных матюгов тайпчекера, конечно).

                                                                                                                  0
                                                                                                                  у вас есть тесты, ммм.
                                                                                                                    0

                                                                                                                    Статанализ есть не только в компилируемых языках, да и много других инструментов, начиная со старого доброго grep до синтакичнских анализаторов, генерирующих AST, применяющих патч и генерирующих новый код. Но проблем больше немного, да.

                                                                                                                      0
                                                                                                                      Статанализ проверит сохранение контракта при рефакторинге?
                                                                                                                        +1

                                                                                                                        А в мэйнстрим компилируемых языках проверит? Нет, только соблюдение сигнатур, но не контрактов.

                                                                                                                          0
                                                                                                                          Сигнатуры — тоже контракт. Во вполне мейнстримовых C++ и Scala их сохнанность при рефакторинге помогает. Думаю в Rust тоже, но рефакторить его мне не приходилось.
                                                                                                                            0

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


                                                                                                                            Не все подобные вещи заметит, но большинство, если нет особо упоротой динамики.

                                                                                                                              +1

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


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

                                                                                                                                0
                                                                                                                                Видимо, потому что типы кажутся им сложными.
                                                                                                                                  0

                                                                                                                                  Они бывают сложными, да. Но, главное, неочевидна, скажем так, выгода от их повсеместного введения, даже на уровне перехода на такие мэйнстрим языки как C# или Java.


                                                                                                                                  Ну и не надо путать строготипизированные и компилируемые — это пересекающиеся множества, не более.

                                                                                                                                  0

                                                                                                                                  А кто отказывается от типов? Тайпчекинг — часть статанализа.

                                                                                                                      0
                                                                                                                      Ну, во-первых, они придумывают свой собственный маленький тайпчекер в виде юнит-тестов.

                                                                                                                      Не только. Я сейчас вынужден придумывать тайпчекер который через интроспекцию ходит по классам и выверяет типы по аннотациям. :)

                                                                                                                      +5
                                                                                                                      А как люди раньше писали неделю код, чтобы в ночь выходного запустить его и проверить?
                                                                                                                        0
                                                                                                                        Я вот всё пытаюсь понять, а как люди делают такое же в динамически типизированных проектах?

                                                                                                                        С трудом и на полном автопилоте. Я с JS-говнокодом однажды мучался всего лишь неделю (его не очень много было), но к пятнице навык «прочитать говнокод и переписать на нормальный, работающий точно так же со всеми граничными случаями» был прокачан просто до нечеловеческих величин. Справедливости ради, половину пятницы я всё-таки дебажил — тестов не было вообще, так просто взять и добавить их — заняло бы еще добрую неделю. Но в общем, 4.5 дня лютейшего рефакторинга на полдня дебага и потом всё завелось и прошло QA — это, я считаю, было неплохим достижением. Больше никогда в жизни в подобном цирке участвовать не буду.
                                                                                                                          +2
                                                                                                                          Через много тестов. Да и современные IDE помогают сильно, за всех говорить не могу, но с Python работать можно вполне комфортно, это я как человек пересевший на него с C# говорю.
                                                                                                                            0
                                                                                                                            Это зависит от того, как код спроектирован и написан. Иногда рефакторинг практически невозможен или слишком рискован. Если более-менее структурировать нормально — то можно зарефакторить, написав предварительно тесты на определенные моменты — black box/unit/инеграционные тесты.
                                                                                                                              +11

                                                                                                                              Да никак. Мы сходу хорошо пишем )))

                                                                                                                                +2

                                                                                                                                То же самое что делаете вы, когда компилятор отказывается помогать с очередной задачей, только чаще.

                                                                                                                              0
                                                                                                                              Есть класс, у которого метод Initialize, который всегда надо вызывать сразу после создания класса — применяешь паттерн «состояние»

                                                                                                                              Суть паттерна ясна, а можно подробнее именно про этот рефакторинг? Можно ссылкой.
                                                                                                                              То, что тело метода инкапсулируется внутрь "состояния" — понятно. Пытаюсь понять, кто и когда дернет этот Initialize у "состояния". А если оно почему-то переедет в конструктор "состояния", то тоже самое можно бы было сделать и с изначальным объектом (но не нужно).

                                                                                                                                0
                                                                                                                                Эм, ну тут всё вроде просто.
                                                                                                                                Допустим есть класс, которому для своей работы нужно сначала сделать что-то асинхронное.

                                                                                                                                Асинхронный конструктор мы сделать не можем, вариант с неконтролируемым запуском таска внутри конструктора нас так же не устраивает.

                                                                                                                                Вот то что надо сделать — не факт что удачное.

                                                                                                                                поэтому люди иногда делают так

                                                                                                                                class Some 
                                                                                                                                {
                                                                                                                                  private SomeType data;
                                                                                                                                  public async Task Initialize()
                                                                                                                                  {
                                                                                                                                    this.data = await SomeHeleper.Fetch();
                                                                                                                                  }
                                                                                                                                
                                                                                                                                  public void DoSomeWork()
                                                                                                                                  {
                                                                                                                                    if (this.data != null) 
                                                                                                                                    {
                                                                                                                                       //...
                                                                                                                                    }
                                                                                                                                  }
                                                                                                                                }


                                                                                                                                вот это не очень хороший подход.
                                                                                                                                Мне больше нравится такой
                                                                                                                                
                                                                                                                                class CompleteSome
                                                                                                                                {
                                                                                                                                  private SomeType data;
                                                                                                                                  public CompleteSome(SomeType data)
                                                                                                                                  {
                                                                                                                                    this.data = data;
                                                                                                                                  }
                                                                                                                                
                                                                                                                                  public void DoSomeWork()
                                                                                                                                  {
                                                                                                                                       //...
                                                                                                                                  }
                                                                                                                                }
                                                                                                                                
                                                                                                                                public class Some 
                                                                                                                                {
                                                                                                                                  public async Task<CompleteSome> Initialize()
                                                                                                                                  {
                                                                                                                                     var data = await SomeHeleper.Fetch();
                                                                                                                                     return new CompleteSome(data);
                                                                                                                                  }
                                                                                                                                }
                                                                                                                                


                                                                                                                                А в клиентском коде мы сами решаем, какие модули у нас инициализируют этот класс, а какие уже работают с проинициализированным.
                                                                                                                                  +4

                                                                                                                                  Ага, понятно, т.е. мы просто переносим проблему в другое место. Честно говоря это мало напоминает паттерн "состояние" и место, где его нужно применять. Кажется, это больше похоже на принцип инверсии зависимостей.

                                                                                                                                    0
                                                                                                                                    Такой подход гарантирует, что DoSomeWork() не будет вызван до инициализации. А проблема и так всегда была в другом месте, потому что это тут так или иначе именно клиенту придётся решать, где вызывать Initialaize
                                                                                                                                      +1

                                                                                                                                      Да, подход дельный, но термин неточный подобран, вот сразу и не разобрался о чем речь. Когда появляется желание инвертировать управление зависимостью, я почти всегда долго не думаю. Перебарщивать и инвертировать всё и вся, конечно, нехорошо. Но если желание появилось в процессе, то обычно это знак. Спасибо за диалог :)

                                                                                                                                    +2
                                                                                                                                    В принципе можно использовать factory method.

                                                                                                                                    class Some 
                                                                                                                                    {
                                                                                                                                      private SomeType data;
                                                                                                                                      private Some (SomeType data) { this.data = data; }
                                                                                                                                    
                                                                                                                                      public static async Task<Some> CreateSome()
                                                                                                                                      {
                                                                                                                                         var someData = await SomeHeleper.Fetch();
                                                                                                                                         return new Some(someData);
                                                                                                                                      }
                                                                                                                                    
                                                                                                                                      public void DoSomeWork()
                                                                                                                                      {
                                                                                                                                        ...
                                                                                                                                      }
                                                                                                                                    }
                                                                                                                                    
                                                                                                                                      0
                                                                                                                                      Да, тоже хороший способ. В том коде, который я рефакторил он не подходил, потому что Some до инишиалайза мог что-то делать. Вообще я сейчас задумался. и понял что подсознательно избегаю использование фабричного метода, хрен знает почему. На ум приходит проблема с ограничением наследования
                                                                                                                                        0
                                                                                                                                        В таком случае Builder может быть подойдет больше, по сути ваша реализация это builder и есть, если заменить Initialize на Build.
                                                                                                                                        Оно так и получится:
                                                                                                                                        var some = new Some();
                                                                                                                                        some.DoSomeNecessaryStuff().Initialize();
                                                                                                                                        // или в случе bulder
                                                                                                                                        some.DoSomeNecessaryStuff().Build();
                                                                                                                                        
                                                                                                                                          0
                                                                                                                                          Я не совсем про это. Вроде, вот такой корректный пример стейта
                                                                                                                                          class SomeBase
                                                                                                                                          {
                                                                                                                                            public virtual string GetSomeInfo();
                                                                                                                                          }
                                                                                                                                          
                                                                                                                                          class CompleteSome : SomeBase
                                                                                                                                          {
                                                                                                                                            private SomeType data;
                                                                                                                                            public CompleteSome(SomeType data)
                                                                                                                                            {
                                                                                                                                              this.data = data;
                                                                                                                                            }
                                                                                                                                          
                                                                                                                                            public void DoSomeWork()
                                                                                                                                            {
                                                                                                                                                 //...
                                                                                                                                            }
                                                                                                                                          }
                                                                                                                                          
                                                                                                                                          public class Some : SomeBase
                                                                                                                                          {
                                                                                                                                            public async Task<CompleteSome> Initialize()
                                                                                                                                            {
                                                                                                                                               var data = await SomeHeleper.Fetch();
                                                                                                                                               return new CompleteSome(data);
                                                                                                                                            }
                                                                                                                                          }
                                                                                                                                          


                                                                                                                                          т.е. можно вот так пользоваться:
                                                                                                                                          var s = new Some();
                                                                                                                                          s.GetSomeInfo();
                                                                                                                                          var cs = await s.Initialize();
                                                                                                                                          cs.GetSomeInfo();
                                                                                                                                          cs.DoSomeWork();
                                                                                                                                          
                                                                                                                                      0
                                                                                                                                      Всегда можно выполнить синхронно, предварительно отвязав от контекста, чтоб дедлок не словить.
                                                                                                                                    +5
                                                                                                                                    Я этим летом три с половиной тысячи строк рефакторил — два месяца. В итоге пошёл к неврологу за рецептами на успокоительные и уехал на две недели на море (чего лучше бы не делал: если вдруг вам когда-нибудь захочется поехать в Архипо-Осиповку с июня по август — лучше нажритесь вусмерть и эти две недели проваляйтесь в вытрезвителе: более приятное времяпрепровождение выйдет)
                                                                                                                                    Начинал как и автор: надо было добавить функционал, но то там отрефачу, то вот тут, а потом «понимаю» что переписывать надо всё. В итоге ни одной старой строчки не осталось. И тоже угнетает что всем на этот проект наплевать, я «один в поле воин».
                                                                                                                                    P.S. дело было на Яве
                                                                                                                                      +2
                                                                                                                                      И тоже угнетает что всем на этот проект наплевать, я «один в поле воин».

                                                                                                                                      У меня теперь профессиональная травма: когда указываю в pull request на говнокодплохой код, и мне отвечают: «Какая разница, я проверил — оно работает!» — глаз начинает дергаться, и рука тянется к топору. Потому что видел к чему приводит даже 6 месяцев такого подхода, если не зарубить в самом начале...

                                                                                                                                        0

                                                                                                                                        А в сентябре там было почти пусто и совершенно прекрасно, да-да. Не надо вместе со всеми, факт.

                                                                                                                                          +2
                                                                                                                                          Рефакторинг должен быть оправдан и далеко не всегда стоит им заниматься в таких объемах. А с такими нервишками лучше про рефакторинг дольше пары дней вообще забыть ).
                                                                                                                                            +2
                                                                                                                                            Про программирование.
                                                                                                                                          +5
                                                                                                                                          Мне кажется, автору этот опыт пригодиться, чтобы оценивать возможные последствия таких задач. При этом, оценивая не только профит кодинга, но и бизнес ценность, риски, планирование и пр.

                                                                                                                                          Не обижайтесь, но поиск жилья, машины да и в целом многое чего были скверно организованы. Рекомендую автору тут поработать над скилами.
                                                                                                                                            +1
                                                                                                                                            Как по мне отличное получилось резюме. =)

                                                                                                                                            • НЛО прилетело и опубликовало эту надпись здесь
                                                                                                                                                –1

                                                                                                                                                А надо заканчивать как в том анекдоте — "расстреляли меня, внучек!"?

                                                                                                                                                  +1
                                                                                                                                                  Ознакомьтесь с другими статьями автора. После них вы возможно измените своё мнение об этой =)
                                                                                                                                                  • НЛО прилетело и опубликовало эту надпись здесь
                                                                                                                                                      0
                                                                                                                                                      Да нет, почему, думаю обсуждать стоит. Даже если ситуация произошла не с автором или очень сильно преувеличена — есть вероятность что она происходила/происходит с кем то другим.
                                                                                                                                                +2

                                                                                                                                                Читал и узнавал себя. Сам питаю слабость к основательным рефакторам тогда, когда можно без них в принципе обойтись, и никто особо о них и не просил.


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


                                                                                                                                                И в скоростной рефакторинг можно тоже выкидывать очки опыта, как в отдельную ветку своего дерева навыков. Главное, чтобы не всегда этот навык применялся вхолостую, а был когда-никогда нужен и востребован.

                                                                                                                                                  +1
                                                                                                                                                  Буду знать, к кому обращаться для разгребания нашего легаси кода :)
                                                                                                                                                  • НЛО прилетело и опубликовало эту надпись здесь
                                                                                                                                                      0
                                                                                                                                                      Победитель получает право разгребать легаси за солидное вознаграждение.
                                                                                                                                                      Очень сомнительный приз.
                                                                                                                                                      • НЛО прилетело и опубликовало эту надпись здесь
                                                                                                                                                        0
                                                                                                                                                        Для чистоты эксперимента тесты командам не выдавать.
                                                                                                                                                      +14
                                                                                                                                                      Жену пришлось брать, потому что я рассказал ей про поездку за три часа до отъезда.

                                                                                                                                                      Где связь внутри этого предложения?

                                                                                                                                                      Естественно, не той ногой, ведь в дурацком автомате всего две педали, и мой мозг всё понял неправильно.

                                                                                                                                                      Естественно? Расположение газа и тормоза не поменялись. Если вы хотели затормозить, то права нога жмет на тормоз, левая — на пустоту или тоже на тормоз. Какой ногой и куда вы промахнулись?

                                                                                                                                                      Дикий рывок, тачка встала.

                                                                                                                                                      Я всё меньше понимаю ситуацию с автомобилем… Почему она дернулась? Почему она встала?
                                                                                                                                                      Дернулась — значит нажали на газ… Остановилась потому-что… Почему?
                                                                                                                                                        +2

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

                                                                                                                                                          –5
                                                                                                                                                          Я когда первый раз на на своей новой машине с коробкой-автоматом поехал, то для себя решил, что буду левую под тормоз переучивать… Первое выжимание тормоза как сцепления завесило меня на ремнях (хорошо пристегнулся, а то мог бы мордой отформатировать руль и/или лобовое стекло).

                                                                                                                                                          Долго потом вырабатывал плавность жима в левой, но зато позже пару раз левая на тормозе реально спасала, т.к. если бы я правую переносил с газа, то не успел бы оттормозиться.
                                                                                                                                                            +8
                                                                                                                                                            переучивать левой на тормоз — это чтобы газ с тормозом одновременно жать? и как же вы на ручной коробке тогда раньше успевали тормозить? если вам левая нога помогла успеть затормозить, то это вопрос не к педалям, а к стилю вождения.
                                                                                                                                                              –6
                                                                                                                                                              Зачем одновременно? Первая и основная вещь при обучении левой тормозить на автомате — это БЕЗОПАСНОСТЬ.

                                                                                                                                                              Если левая на полочке валяется, а праву нужно с газа перенести на тормоз, вы теряете доли секунд, которых вам может не хватить.
                                                                                                                                                              У меня левая уже стоит на педали тормоза и я в критической ситуации тормозить начинаю сразу.
                                                                                                                                                              На ручке такой возможности нет. Но если на автомате она есть то почему бы ее не освоить?

                                                                                                                                                              На ручке и на автомате я езжу совершенно нормально. Никаких проблем не возникает. Просто я выдрессировал свои инстинкты оптимально под оба типа автомобилей, а не пытался сократить издержки на дообучении под автомат.
                                                                                                                                                              +9
                                                                                                                                                              НЕ надо так делать.
                                                                                                                                                              На машинах с автоматом специально для левой ноги есть полочка. Левая нога полочку покидать не должна.
                                                                                                                                                                –4
                                                                                                                                                                Во сколько учителей жизни сразу образовалось!!!

                                                                                                                                                                Хотите ставьте на полочку — я вас левую на тормоз переучивать не заставляю, но сам как решил так и сделал и получил с этого приятные бенефиты,
                                                                                                                                                                  +6
                                                                                                                                                                  Просто чтобы понимали. А то возьмут с вас пример, а решение-то не для среднего человека, ибо бенефиты тут примерно как у пистолета без скобы. От бедра стреляет быстрее, но есть нюанс…
                                                                                                                                                                    –4
                                                                                                                                                                    А где вы увидели у меня что я другим советую?
                                                                                                                                                                      +1

                                                                                                                                                                      Вы приводите опыт из жизни с профитом. Что ≈ лайфхак, что в свою очередь для кого-то может быть ≈ совет.
                                                                                                                                                                      Для того, чтоб этого не было, придумали фразу "не повторяйте это дома" или "автор не несёт отвественности, что …".
                                                                                                                                                                      Как-то так.

                                                                                                                                                                        –3
                                                                                                                                                                        Ок. Только никому не признавайтесь что у вас есть член иначе сочтут что вы насильник.
                                                                                                                                                                +2
                                                                                                                                                                Я когда первый раз на на своей новой машине с коробкой-автоматом поехал, то для себя решил, что буду левую под тормоз переучивать…

                                                                                                                                                                Вы же конечно же на тот момент были в курсе, что это грубое нарушение ПДД, техники безопасности эксплуатации автомобиля и т.п.?

                                                                                                                                                                Что-то вдруг ещё страшнее ездить стало после таких высказываний…
                                                                                                                                                                  +2
                                                                                                                                                                  Это не нарушение.

                                                                                                                                                                  Ну и, чтобы два раза не вставать.

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

                                                                                                                                                                  Спортивное вождение автомобиля и выполнение приёмов высшего водительского мастерства подразумевают умение работать с педалью тормоза обеими ногами в зависимости от ситуации. Но это умение надо специально тренировать.