о поверхностной красоте кода (программ.)

[avva]

Отличная блог-запись, с которой я полностью согласен (англ.). Процитирую ключевой абзац:

So, while Marco's problem started with the project's shoddy documentation and API, his actual code criticism focuses on issues that are apparently superficial. He hasn't discovered a substantive bug or architectural weakness in the snippet above. Instead, what matters to him are simple virtues like consistency, style, and readability. Marco is saying, in fact, that the OpenSSL code sucks because it lacks superficial beauty. I couldn't agree with this position more.

Именно так. А за goto в кусок кода, закрытый if(0), я бы дополнительно еще бил линейкой по рукам.

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

Comments

[mopexod.livejournal.com]

Черт, goto в кусок кода, закрытый if(0) - красиво!
Ну не хватало чуваку локальной функции, выкрутился как мог :)

[itman.livejournal.com]

Что говорит о том, что красота кода - понятие субъективное. По поводу if(0): многие чистые сишники ненавидят си++, а в си++ такую гадость не пришлось бы писать, можно было с помощью деструктора разрулить.

[vitus_wagner]

Вообще-то обычно там делают перед меткой err: return.

[psilogic.livejournal.com]

по-моему тоже красиво :)

но юзать не стану, ибо лоси с линейками запарят...

[esspe.livejournal.com]

if(0) тоже понравился, красивая обработка исключений :)

В комментах там предлагают его превратить в маркос, чтобы глаз не "резало".

[dmpogo.livejournal.com]

Мне тоже понравилось, такой Фортран IV :)

[secondary-tea.livejournal.com]

Можно было if(0) в макрос спрятать.

[shigin.livejournal.com]

Я открыл procmail.c. Кто--нибудь может объяснить зачем там ставят точку с запятой перед блоком? Что бы получился смайлик?

[psilogic.livejournal.com]

предположу, что для читабельности - чтобы отметить тот факт, что это блок не относится к какому-нибудь if/else/for строкой выше

хотя заботиться о читабельности и делать такую длинную функцУ - это я даже не знаю, с чем сравнить...

[itman.livejournal.com]

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

Если верить классикам...

[marknn.livejournal.com]

...Premature optimization is the root of all evil. (DK)

На самом деле, надежность, скорость и пр. гораздо лучше улучшать когда код можно прочитать понормальному. Очень трудно оптимизировать код в котором есть десяток вложенных if и примерно из половины выход через goto.

Форматированием, понятное дело должен заниматься тектовый редактор. А вот без внятности тут никуда. Не думаю что тут уж такая тонкая грань. Только не надо путать комментарии по типа " теперь увеличиваем счетчик", и внятность кода, разумеется. ;-)

[beldmit.livejournal.com]

Не, ну часть IFDEF-ов из OpenSSL теоретически можно изжить, сдвинув все алгоритмы в engine. Практически - после этого, например, не получится реализовать SSL/TLS. Мы с коллегами пробовали, утверждаю со знанием дела.

[vitus_wagner]

Ну ты ж знаешь, что самый цимес там не в ifdef-ах, а в реализованном на препроцессоре парсере/сериализаторе ASN.1.

[dimrub.livejournal.com]

А на плюсы переписать пробовали?

[trurle.livejournal.com]

OpenSSL source code sucks in a big way.
Однажды я добавлял поддержку Authority Information Access в OpenSSL и впечатлился в полной мере.
Вообще, не то что бы качественный, но минимально приемлемый код составляет пренебрежимо малую часть общего объема кода; о причинах такого обстоятельства я боюсь думать.

[vitus_wagner]

Там засада в том, что НИКТО в данный момент не представляет логику работы этого хозяйства в полном объеме. Следствием этого была недавняя история с багом в Debian, когда мейнтейнер решил убрать чтение неинициализированной памяти, и в результате оторвал вообще все источники случайности кроме PID.

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

[amarao-san.livejournal.com]

goto nomore_rc;
goto nomore_rc;
goto nomore_rc;
goto nomore_rc;
goto nomore_rc;
goto nomore_rc;

[ygam.livejournal.com]

Послал ссылку на procmail.c в рабочую рассылку. ГотичноГалахичноФеерично!

[igorlord.livejournal.com]

But the comments there were SO carefully laid out! Someone spent some real efforts on that.

Reminds me of a lipstick on a pig...

[spamsink.livejournal.com]

После знакомства с синтаксисом procmail примерно понятно, что можно увидеть в коде.

[vodianoj.livejournal.com]

Конкретно Гото в кусок кода закрытый иф(0), для "с" не такое плохое решение - оно очень читабельно - сразу ясно как туда попасть и что делает. Некий вид обработки эксепшенов.
Т.е. по стилю выглядит очень современно:
try {
Do something....
} catch(error e) {
Do some error related staff...
} finally {
Perform cleanup...
}
return ret;

С другими вещами и с главным тезисом полностью согласен.

У Кнута, кстати есть хорошая статья по теме goto - "Structured programming with goto statements" - http://pplab.snu.ac.kr/courses/adv_pl05/papers/p261-knuth.pdf

[salas.livejournal.com]

#define try
#define catch(x) if (0)
#define throw goto err;

оффтопик )

[papa-vova.livejournal.com]

Просто сложновато, если честно, вот так взять и начать разбираться в FreeBSD. Что править — то ли настройки SMTP, то ли настройки Exim, то ли конфигурационные файлы sendmail (открыл, посмотрел, закрыл ибо действительно такое ощущение, что это не файл конфигурации, а просто кто-то долго бился головой об клавиатуру) (http://unix-forum.ru/post3660.html#p3660)

Фразу про sendmail.cf, клавиатуру и голову встречал лет десять-двенадцать назад, кажется, в эхе ру_линукс :-)

Re: оффтопик )

[egorfine.livejournal.com]

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

[egorfine.livejournal.com]

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

Знаете, Анатолий, я никогда в жизни исходник procmail не видел, но слова моего учителя про procmail помнил всю жизнь. И сегодня наступил тот день, когда я прочитал этот ваш постинг, статью, приготовил чаек и в предвкушении райского эстетического наслаждения впервые в жизни открыл procmail.c, ожидая проникнуться настоящим искусством, ожидая получить нереальную порцию настоящего системного вдохновения...

Анатолий, расскажите пожалуйста, восстановите мой помутненный разум.... только вы можете мне вернуть покой!... я верю, в вашем опыте найдется ответ на этот вопрос смысла жизни...

Скажите... как можно так писать?????

[stas]

Ну а че, у qmail код покраше прокмейла, по-моему.

[trurle.livejournal.com]

Ы!

[yba.livejournal.com]

Нормальный код, твердая четверка за читабельность, при том что на C я не пишу. Даже 4+.
По поводу goto внутрь if-а - а какие альтернативы? Я не уверен что второй goto вместо if(0) был бы читабельнее. Претензии к форматированию - просто детский сад.

[cmm.livejournal.com]

а какие альтернативы?

return перед меткой, или проверка флага типа "всё ОК или нет".

[emdin.livejournal.com]

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

[emdin.livejournal.com]

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