Чужой код

Автор: 1way2pray

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

Решил привести небольшую аналогию:

Вот представь, что тебе доверили достроить за другим прорабом лабораторию на острове. Ты приходишь на объект, а там кроме недостроенного здания: огромный вентилятор (размером со здание), большой воздушный шар и комната набитая швабрами.
Почесав голову, ты разбираешь этот хлам и доделываешь лабораторию. Сдаешь объект ученным, но через 5 минут они выбегают с криком: "УТЕЧКА ЯДОВИТОГО ГАЗА!!!"
— Как так–то, блять! Должно же работать! — в отчаянии кричишь ты и звонишь прошлому прорабу:
— Вася, у нас ядовитый газ потёк! В чем проблема?
— Не знаю, должно было все работать. Что–то в проекте менял?

— Немного, швабры вынес...
— Швабры потолок держали!
— Что??? Что, блять, извините???
— Говорю, швабры потолок держали. Над ними цистерны с газом были. Очень тяжелые, пришлось в комнату снизу швабры напихать.

— Ты хотя бы записку на двери повесил бы, что швабры для держания потолка! У нас тут ядовитый газ течет! Что нам делать?
— Включай вентилятор. Он сдует газ с острова.
— Я его, блять, демонтировал сразу же!
— Зачем?
— Зачем ты построил 120 тонный вентилятор? Ты не мог положить ящик блядских ПРОТИВОГАЗОВ?
— Ящик противогазов искать нужно, а вентилятор у меня с прошлого заказа оставался.

— Вася, я убрал твой вентилятор! Мы тут задыхаемся!
— Хули вы тогда там делаете? Садитесь на воздушный шар и уебывайте на хуй!

в аналогиях ты брат не очень силен )))

ДА-А! Как жаль, что этот текст не попался раньше!
Иногда смотришь чужой код и думаешь - это в принципе работать не должно, но он работает.

Картинку в тему!

Это сообщение отредактировал KapitanSH - 9.03.2018 - 11:48

Он то силён, но писал то не он, он сам копипастер, с ним бесполезно говорить.

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

Некоторые конструкции и приемы отличались столь изысканным изяществом, что даже сидя с листом бумаги и ручкой, так и не смог понять, "нафига все это надо и как оно вообще работает ?" .

P.S. В ВУЗе на 3-4м курсе программки делал (шифралгоритмы и т п), решил как-то снова посмотреть -улучшить. Бросил через 20 мин, проще снова написать.

Это сообщение отредактировал stealthman - 9.03.2018 - 11:57

очень часто проще написать все с нуля, чем разбиратся в чужем говнокоде

Я так думаю тебе подойдёт:
"Вовочка, а почему у тебя такие большие ресницы?"- "Это потому что я в детстве много плакал". -" Лучше, если б ты в детстве много сикал".

В чужих макетах копаться та же срань. Почему нельзя сделать по слоям, называть их, группировать? Когда находишь "Слой 33 копия копия 78" проклинаешь все на свете. И это только в фотошопе, а уж что в Индизайне вытворяют...

Это сообщение отредактировал Yubikiri - 9.03.2018 - 12:29

Если бы "копипастер". Не всё так однозначно.

В оригинале у автора концовка такая: "- Херли вы тогда там делаете? Садитесь на воздушный шар и уебывайте!"

А у ТСа такая: "— Хули вы тогда там делаете? Садитесь на воздушный шар и уебывайте на хуй!"

Как можно заметить, ТС отважно дополняет авторский текст, становясь, таким образом, соавтором!

"Дорогая Якобина, ты же меня знаешь: когда меня режут, я терплю, но когда дополняют, становится нестерпимо". © Барон Мюнхгаузен.

Таже хрень у меня на работе (ремонт квартир). Прораб как принесет свои "записульки" с объекта и сидишь гадаешь - что за хрень тут расчерчена и что к чему относится. Но он художник - он так видит.

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

Итак, названия функций должны описывать реальное действие - есть.

Содержимое функции должно соответствовать действию - есть.

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

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

Функции как таковые должны иметь видимость исключительно на своём программном уровне - вы знаете что такое хаос, вот это оно и есть.

Программные под_уровни должны разделять функции одного уровня в случае жёсткой зависимости от последовательности их применения, в простейшем варианте это просто последовательность объявления функции. Дословно 1-2-3-4-... как объявлены - так и применяются, нельзя перемешивать.
К этому моменту пиздец приравнивается к вселенскому масштабу, потому как буквально все функции хала глобальные.

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

Недавно обнаруженная "фича" в чужой системе, куда я пришёл консультантом, и мой коммент

Это сообщение отредактировал Нанонимно - 9.03.2018 - 12:42

класс!!!
точнее не придумаешь)))

давайте будем честны

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

Применив тождественное преобразование, получим -> "любой код есть говнокод"

Это сообщение отредактировал stealthman - 9.03.2018 - 13:33

Ломыч, ты не устал всякую хуйню постить?
Работаю с 3D моделями. Иногда проще модель с нуля создать чем чужое недоразумение кривить.

Круто! С такими?

Это сообщение отредактировал ofox - 9.03.2018 - 13:38

Это типичная 3D модель. Пизда, жопа, рот.

Нужна стандартизация и единый системный подход.
Когда фигачат напрополую лишь бы работало, без единого форматирования и т.д.
Потом это все обрастает заплатками, часто именно из-за отсутствия системы как таковой. И уже через год возвращаться вспоминать даже автору порой бывает непросто, не говоря уже о других.
При этом заметил, что если в конторе, есть хотя бы общий стандарт по форматированию кода, то уже легче разбираться, когда же все пишут кто как хочет - это реально трэш: примерно можно сравнить так.
Ты собираешь грибы зимой подосиновики, красные шляпки на белом снегу видно издалека - это твой код, привычно отформатированный для тебя. А теперь снег растаял и ты резко оказался в непроходимых джунглях и тебе все так же надо искать грибы )))
Разница существенная. Между тем только примерно в одной из 10 контор есть строгие требования к единому форматированию кода.
А само по себе то, что у нас разрабы пишут код кто как хочет, даже банально с т.з. форматирования по-моему говорит, что мы не далеко ушли от тех же индусов.

Интересно то, что есть ребята, которые форматируют код так, что он разрастается на многие тысячи строк. Отформатируешь и он сокращается на треть и становится более читабельным, разбираться уже легче. Но это время. А если переписать с нуля, то получается еще в два раза короче )

Это сообщение отредактировал dnaquest - 9.03.2018 - 19:02

старое:


если бы программисты строили дома

1.03. Ура! Нам предложили крупный контракт на постройку 12-этажного жилого дома. У всех бурный энтузиазм. Выпили на радостях 2 ящика пива.

2.03. Заказчику не нравится выражение «как только, так сразу». Требует назвать конкретные сроки. Темный, ничего не смыслит в высоких технологиях.

3.03. Обсуждали сроки. Выпили 3 ящика пива. Петрович говорит, что тут всей работы на 4 месяца. Значит, на самом деле 8. В итоге в контракте записали 12, хотя раньше, чем за 16, вряд ли управимся.

6.03. Петрович пошел сдавать бутылки.

8.03. Отмечаем 8 Марта. Женщин у нас в фирме нет, так что праздник никто не портит.

2.04. Петрович говорит, что пора начинать работать. Сговорились они, что ли? Обнесли площадку забором и повесили красивые плакаты «Строительство ведет компания «Аllstrоу» (www.аllstrоу.ru)». С чувством выполненного долга играем в Quаkе.

20.04. Пришел заказчик, интересовался, как дела. Посадили его за Quаkе и позволили нас всех обыграть.

21.04. Обсуждали проект. Сидоров предлагает крупноблочную архитектуру. Петрович настаивает, что все надо строить по старинке, из кирпича, не по-ламерски. Самый радикальный проект предложил Алекс: построить несколько десятков деревянных коттеджей и потом соединить их подземными туннелями. На Западе сейчас так модно. Напомнили ему, что заказчик требует именно 12-этажный дом. Пытались решить вопрос дуэлью в Quаkе. Алекса с его коттеджами завалили сразу, но между Петровичем и Сидоровым вышла ничья. В итоге каждый будет строить по своему плану, а потом попытаемся все это соединить, чтоб не рухнуло.

30.04. Первый этаж готов! Показали его заказчику. Он интересовался, почему в разных комнатах разная высота потолков, почему из стен вываливаются кирпичи и почему в доме нет подъезда, а влезать приходится через окно. Объяснили ему, что это специальные ограничения демо-версии. Уходим на праздники, гордые собой.

10.05. Петрович протрезвел первым и долго ругался. Мы думали, что Алекс выпил все пиво. Оказалось хуже: мы забыли про фундамент. В проекте он, конечно, описан, но ведь документацию читают только ламеры.

11.05. Ломали первый этаж. Обидно, блин.

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

17.07. Алекс строит чердак и крышу. На земле. Потом поднимем краном.

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

14.08. Разломали несколько панелей, чтобы Петрович мог изучить внутреннюю архитектуру. Петрович ругается, кричит, что проектировщики панелей — полные ламеры.

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

1.09. Стройкомбинат выпустил новую версию панелей, улучшенной прочности и утепленности, со встроенными стенными шкафами. Правда, ни по форме, ни по размеру они не совместимы с предыдущими и в три раза тяжелее. На какую архитектуру они вообще рассчитывают, эти комбинатские?

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

2.10. Петрович добрался до пятого этажа. Горд собой. Обратили его внимание на тот факт, что его стена наклонена под углом 40 градусов. Он ругался, кричал, что мы ламеры и ничего не понимаем. Потом обещал подумать.

3.10. Приходил заказчик. Спросил, почему стена наклонена под углом 40 градусов. Объясняли ему про силу Кориолиса. Он все выслушал, потом сказал, что он, конечно, в строительном деле ничего не смыслит, но у него по соседству точно такой же дом, и там стена прямая. Блин. Потом этот идиот Алекс ляпнул при нем про свои изменяемые окна. Заказчик, естественно, загорелся и настаивает, чтоб делали именно так. Дважды блин.

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

5.10. Петрович признал, что со стеной действительно имеется проблема. Говорит, что неправильно положил какой-то кирпич. Но чтобы понять, какой именно, надо перебрать их все. Проще все снести и построить заново.

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

8.10. Ломали кирпичную часть. Попутно повредили панельную. Вся постройка скрипит и угрожающе шатается. Укрепили деревянными подпорками и пошли играть в Quаkе.

17.10. Петрович вышел из запоя. Работаем.

7.11. Празднуем 7 Ноября — или как оно там теперь называется? Коммунистов у нас в фирме нет, так что праздник никто не портит.

15.11. Вспомнили, что у нас кран достает только до 8 этажа. Послали Сидорова доставать новый кран. Играем в Quаkе. Алекс замочил Петровича. Растет смена!

24.11. Вернулся Сидоров. Кран не достал, зато достал крутой экскаватор. Предлагает вырыть глубокую шахту и построить дом не в высоту, а в глубину. Говорит, что нигде в контракте не сказано, что 12 этажей должны быть над поверхностью. Еле отговорили.

25.11. Устроили мозговой штурм по проблеме крана. На последней бутылке пива нашли решение. Бросили основное строительство. Строим рядом 4-этажный дом. Потом втащим наш кран ему на крышу.

25.12. Празднуем католическое Рождество. Католиков у нас в фирме нет, так что праздник никто не портит.

14.01. Ничего не помню. Голова болит. Мужики, какой сейчас год?

2.02. Ну, кажется, наконец-то достраиваем 12-й этаж. Завтра будем прилаживать сверху чердак и крышу, что соорудил Алекс.

3.02. Алекс — ламер. Крыша регулярно съезжает. Пока подперли краном. Думаем, что делать дальше.

4.02. Алекс доказывает, что он не виноват. Просто 12 этажей Сидорова на 4 метра выше и на 5 метров шире, чем 12 этажей Петровича. Выяснилось, что они строили из разных панелей. Но Алекс все равно ламер, поскольку его крыша не подходит по размеру ни одному из вариантов. Его шахта лифта, кстати, тоже.

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

7.02. Снег пошел.

10.02. Соорудили крышу из фанеры, покрасили под жесть. Будем надеяться, заказчик не заметит.

11.02. Тестировали лифт. Его остановки приходятся между этажами, но выбраться из кабины можно. На четных этажах ползком, на нечетных — подтягиваясь. Не забыть описать это в документации.

12.02. А вообще-то лифт ездит крайне медленно. Петрович ругает всех ламерами и собирается заняться оптимизацией.

13.02. Петрович оптимизировал лифт. Тот разогнался, пробил крышу и улетел в неизвестном направлении. Хорошо, что крыша фанерная, и чинить будет легко. После этого шахта лифта рухнула. Вспомнили, что так и не заменили деревянные подпорки на что-нибудь более прочное. Ничего. Ходить пешком полезно.

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

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

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

21.02. Приходил заказчик. Спрашивал, нельзя ли внести в проект небольшие изменения. В частности, вместо 12-этажного дома построить поселок из деревянных коттеджей, соединенных туннелями. Он прочитал, что на Западе сейчас так модно. Нейтрализовали Алекса прежде, чем тот успел открыть рот, и вежливо, но твердо объяснили заказчику, что он не прав.

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

23.02. Праздновали 23 февраля. Военных у нас в фирме нет, так что праздник никто не портил. Женщин тоже нет, так что подарков нам никто не дарил. Обидно.

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

27.02. Вспомнили, что так и забыли сделать подъезд. Размышляли, не рухнет ли дом, если прорубить его сейчас. Сидоров сказал, что лучше не рисковать. Петрович обозвал его ламером и согласился. Не забыть описать в документации вход через окно как особенность дизайна.

1.03. К-как первое марта?! Откуда?! Вчера же еще… Блин. Кто ж знал, что в этом ламерском феврале 28 дней! Выходит, сдача объекта — не через неделю, а послезавтра?!

2.03. Аврал. Работаем 24 часа в сутки, не просыпаясь.

3.03. Убедили заказчика, что нам нужен еще день для финального тестирования. М-да, ну мы вчера и наработали… А в общем, не все так страшно. Ну что с того, что некоторые двери находятся в полу или в потолке, либо ведут с десятого этажа прямиком на улицу, в некоторые квартиры в принципе невозможно попасть, санузел кое-где совмещен с кухней, в половине дома нет воды, в другой половине — электричества, канализация обрывается на шестом этаже, а лестницу между восьмым и девятым пришлось сделать веревочной? Главное — провести заказчика правильным маршрутом. И еще — успеть до завтра развесить на месте исчезнувших окон картинки с изображением заоконных пейзажей…

4.03. Yes! Yes! Мы сделали это! Отмечаем сдачу объекта. Я пью мало, мне надо еще успеть уволиться, прежде чем эта хренотень рухнет к чертовой матери…

нихуя себе вы метафорнули... а почему на острове я так и не догнал..

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

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

А мне очень понравилось.
Правда я праздник 9 марта день освобождения от 8 марта праздную. может в этом дело?

Видел как-то код в одной большой компании. Идёт класс, всё как полагается, author: ****, далее импорты и код, строк на несколько тыщ, всё это закомментировано, далее комментарий "**** - мудак" и новый класс с новым кодом.

Классика наследия.
#define TRUE (random()>=0.5)