Nicolette 13 июл 2010 в 17:50

Код, который невозможно поддерживать

7 мин

11K

GTD*

Перевод

+95

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

Maximuzzz 13 июл 2010 в 18:10

Это кажется называется «обсуфикация кода», не? )))

Счас поначитаются, и начнут развлекаться… А потом сами-же в своём «чистописании» не разберутся )))

Nicolette 13 июл 2010 в 18:12

habrahabr.ru/blogs/arbeit/99016/#comment_3053049
Совершенно разучилась попадать по правильной ссылке для ответа :-(

amlet 14 июл 2010 в 16:22

Счас поначитаются, и начнут развлекаться… А потом сами-же в своём «чистописании» не разберутся )))

Для того чтобы потом не огрести есть всякие SVN'ы, git'ы, mercurial'ы и прочие приблуды, облегчающие последствия экспериментов.

Maximuzzz 14 июл 2010 в 16:33

Те кто начнут эксперементировать ради эксперимента скорее всего с SVNами не знакомы, я полагаю.

amlet 14 июл 2010 в 23:36

Тогда сами себе злые Буратины :)

Nicolette 13 июл 2010 в 18:12

Обфускация. Обсуфикация — это, надо полагать, что-то вроде добавления суффикса _l локальным переменным, _g — глобальным и _p — параметрам? :-)

Надо было добавить дисклеймер — ни переводчик, ни автор не несут ответственности за возможные применения текста? :-)

Maximuzzz 13 июл 2010 в 18:44

Обфускация — она самая, попутался малость.

pomkaster 13 июл 2010 в 19:00

Я под обфускацией всегда понимал немного друг

Nicolette 13 июл 2010 в 19:52

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

seregagl 14 июл 2010 в 09:24

Обфускация — это не то, в начале же написано «ваш код не должен выглядеть неподдерживаемым, а только быть таким»

Ringtail 13 июл 2010 в 18:16

Вы уверены, что хотели написать именно «Хампти Дампти», а не «Шалтай-Болтай»?)

Nicolette 13 июл 2010 в 18:18

Ой. Исправила, спасибо.

spmbt 13 июл 2010 в 18:29

Произвольно смешивайте несколько языков (человеческих или компьютерных).

У нас есть возможность сложных комбинаций: транслит, неграмотный английский, точно такие же имена на грамотном английском, неправильный перевод (good — well) и комбинации всех методов именований в одной переменной.

maxshopen 13 июл 2010 в 18:36

Хороший пример слово code, cod, kod, kode, cd (сокращение) — мне попадались все эти варианты ;)

sgzmd 13 июл 2010 в 19:56

GoodPlusPlus; NotGood != Bad и так далее. Оруэлл наше все. Имена переменных и функций на ньюспике обеспечат всех работой на долгие годы.

Setti 13 июл 2010 в 18:45

The first 90% of the code accounts for the first 90% of the development time. The remaining 10% of the code accounts for the other 90% of the development time.
Tom Cargill

thevery 13 июл 2010 в 20:00

а слышал такой примерный русский аналог: 90% сделано, 90% осталось

volodymyr 13 июл 2010 в 18:52

А.К.Р.О.Н.И.М. / псть скрщнми грзд интрсн

На ум сразу приходит Стив Макконнелл и:

mov AX, 732h   ; R. I. P. L. V. B.

Жаль, что в университете редко объясняют студентам, о том, что чистота и хорошая документированность кода важна не менее, чем его работоспособность — намного проще работать, когда чистый код с внятными комментариями входит в привычку. Хотя все же есть преподаватели, которые это делают — огромное им спасибо.

dmbreaker 13 июл 2010 в 19:57

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

nekt 14 июл 2010 в 18:18

Угу. При этом не использует рекурсию, в каждой строчке не больше 80 символов и всего не более 30 строк кода.

naryl 15 июл 2010 в 02:15

Чего вы нашли неподдерживаемого в рекурсии?

nekt 15 июл 2010 в 02:27

Легкоподдерживаемый код это простой код. Рекурсия не является простым кодом. И именно к простому коду не нужны комментарии.

dimka_kravchuk 15 июл 2010 в 06:04

> Рекурсия не является простым кодом.
С каких пор?

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

nekt 15 июл 2010 в 15:30

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

Может мне табличку «сарказм» вместо аватарки поставить?

naryl 15 июл 2010 в 15:10

Ладно, тогда чего вы нашли сложного в рекурсии?

nekt 15 июл 2010 в 15:29

Циклы тоже лучше не использовать. Они тоже усложняют код.

naryl 15 июл 2010 в 15:53

map-reduce-filter — наше всё. ;)

Почитайте SICP: newstar.rinet.ru/~goga/sicp/sicp.pdf

JapaniseDoll 13 июл 2010 в 20:29

а мне пришли в голову мысли о сокращениях directX, то еще чудо:)

DeusModus 13 июл 2010 в 18:57

Как это знакомо, боже мой. Радует лишь то, что мы уже закончили тащить тот проект.

_getSomethingGet()
getSomethingGet()
__getSomethingGet()

три разные функции, комментарии есть только к одной, в зависимости от входящих данных либо вызывается callback функция(как оно там называлось) и делает что угодно.
Комментарии: 70% пустышки-шаблоны, 20% ломаный английский, 10% русский.

Setti 13 июл 2010 в 19:09

а шаблоны случайно не такие?

DeusModus 13 июл 2010 в 21:25

// comment this позднее

Долго еще буду вспоминать.

VenomBlood 13 июл 2010 в 19:29

Как же я Вас понимаю, недавно пришлось видеть проект от которого ночь не спать можно, вот выдержка:

private void w_hlink_w_r_w_rPr(XmlNode rPr, ref Paragraph p)

        {

            Style stl = new Style();

            foreach(XmlNode node in rPr.ChildNodes)

                switch (node.Name)

                {

                    case "w:rStyle":

                        string seed = node.Attributes["w:val"].Value;

                        stl = styles.GetStyle(seed).FullCopy();

                        SetStyleToP(stl, ref p);

                        break;

                }

            tip_w_rPr(rPr, ref stl);

            SetStyleToP(stl, ref p);

        }

Это ещё достаточно безобидный кусок кода.
Остальные функции и переменные в таком же стиле…

sgzmd 13 июл 2010 в 19:57

какой замечательный код! Мне кажется, он нарушает все существующие coding guidelines и замечателен уже этим фактом! ;-)

Colwin 14 июл 2010 в 09:15

Ему самое место на govnokod.ru )

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

StShadow 13 июл 2010 в 19:17

6 из 20 я наблюдаю прямо сейчас :(

Masterkey 13 июл 2010 в 19:57

найдите разработчика и напомните ему легенду про маньяка-кодера

StShadow 13 июл 2010 в 19:59

хреново, что они в США/Франции/Индии/Китае, а так бы джихад точно получили бы…

sgzmd 13 июл 2010 в 20:00

Вот поэтому практика «no review — no submit», несмотря на вызываемые тонны раздражения и немотивированную агрессию, когда твой код попал к человеку, который докапывается до каждой буквы, зачастую если и не гарантирует, то очень сильно повышает вероятность того, что потом это можно будет поддерживать и ре-использовать.

StShadow 13 июл 2010 в 20:02

Да! Я вырос на Стиве Макконнелле, но, к сожалению, ревью тул прикрутили только после 9 лет разработки проекта…

Masterkey 13 июл 2010 в 19:22

автор забыл про фундаментальные истины

-1. Блоки кода, классы функции, которые есть в коде, но они нигде не используются.
0. Полотнища с закомментированным кодом непонятного назначения, иногда, после удаления таких полотнищ все перестает работать.

ПРЕЛЕСНО )))
а нате вам виртуальный + в репу

sgzmd 13 июл 2010 в 20:01

-1 — это легко отслеживается, по крайней мере в Java в современных IDE. В плюсах сложнее, но опять же можно отследить.

Если же используются плюсы, то там такой простор для экспериментов и мелкого вредительства, что просто дух захватывает ;-) Знамя Александреску мы несем гордо!

Masterkey 13 июл 2010 в 21:10

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

ну и финалом — это крякнутая патченная делфа 3,1
фаталити — полное отсутствие видения проекта у заказчика

это был мой кошмар-на-яву

ЗЫ да и не такой я матерый делфист тогда был, мягко говоря новичек, возможно для спецов все там выглядело по-другому

naryl 13 июл 2010 в 21:50

Александреску раскаялся и написал TDPL. ;)

shaman007 13 июл 2010 в 20:23

Я думал, что слово «пролог» относится к Прологу. Вот где все, что нужно сделано уже на уровне языка!

Maximuzzz 14 июл 2010 в 10:32

Да уж, ничего придумывать не надо, умудрилсыя «вляпаться» в поддержку Tivoli TEC, вот уж где всё заранее нечитаемо и неподдерживаемо.

VasilioRuzanni 13 июл 2010 в 21:51

Гениально :) И, к сожалению, встречается в живом коде.

WheelReinventor 13 июл 2010 в 21:52

ааа, теперь-то я понял, наконец, почему в проекте, который мне был передан все было так ужасно. автор имел чувство юмора и оставил мне прекрасный код, написанный, видимо, по этой статье. пример:
pastebin.com/N3XCBCcS

akzhan 13 июл 2010 в 22:40

Меня от него сплющило…

Aidsoid 13 июл 2010 в 23:10

А теперь представьте это ещё и без подсветки синтаксиса.

VaiMR 13 июл 2010 в 23:25

Я видал людей, которые пишут программы в мэйне (иногда в паре-другой функций), на переписывание с нуля которых даже с ООП и «правильными подходами» у опытных программистов уходило несколько месяцев. Размеры программ потрясали — 10000-20000 тысяч строк. И все это в одной-двух функциях. И что самое удивительное, эти программы даже иногда работали, с утечками памяти, заваливанием через 20 минут работы, случайным удалением всех файлов в каталоге куда ты сохраняешь другой файл, сообщениями «произошла какая-то ошибка» и падением. Но они работали. Программистам, которым удается запустить такой код необходимо вручать «программистскую премию Дарвина».

Nicolette 13 июл 2010 в 23:27

А получится? Премия Дарвина требует изъятия своих генов из генофонда человечества. По аналогии, такая же премия для программистов должна требовать полного изъятия своего кода из рабочего кода человечества — не уверена, что такое реализуюется.

VaiMR 14 июл 2010 в 10:04

Получится! Если код, попадающий под хотя бы один из вышеприведенных пунктов, не будет становиться рабочим и будет изыматься сразу при рождении. Хотя, я конечно немного идеалистичен. Всегда будут такие программисты, всегда будут компании, в которых работают такие программисты… Но премию выдавать уж точно можно!

kolen 14 июл 2010 в 10:05

> 8. веРхНий реГисТр
> Произвольно переводите в верхний регистр первую букву слога, а также другие буквы.

ЭтО жЕ лАфФныЙ эМО-кОд))))))))))))))))))))) чМаФФФФ))))))))
headrush.typepad.com/photos/uncategorized/emobookcover.jpg

asolntsev 15 июл 2010 в 01:06

Гуд.
Спасибо! Такие вещи обязательно надо переводить на русский.

Nicolette 15 июл 2010 в 09:36

… распечатывать большими буквами и развешивать на стенах офиса? :-)

torbasow 15 июл 2010 в 11:40

Отлично подходят Эсперанто, Клингонский язык и хоббитон.

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

Nicolette 15 июл 2010 в 13:06

Да, имелся в виду хоббичий язык, *смущенно* я просто Толкина последнее время читала только на английском. Исправила, спасибо :-)

Зарегистрируйтесь на Хабре, чтобы оставить комментарий