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

[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): многие чистые сишники ненавидят си++, а в си++ такую гадость не пришлось бы писать, можно было с помощью деструктора разрулить.

[shigin.livejournal.com]

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

[itman.livejournal.com]

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

[beldmit.livejournal.com]

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

[trurle.livejournal.com]

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

[mopexod.livejournal.com]

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

[vitus_wagner]

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

[itman.livejournal.com]

я имею в виду логику auto_ptr

[vitus_wagner]

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

[vitus_wagner]

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

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

[trurle.livejournal.com]

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

[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 в рабочую рассылку. ГотичноГалахичноФеерично!

[psilogic.livejournal.com]

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

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

[ygam.livejournal.com]

Это вы еще не видели Майкрософт Оффис!

[esspe.livejournal.com]

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

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

[psilogic.livejournal.com]

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

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

[cmm.livejournal.com]

в си++ такую гадость не пришлось бы писать

что именно Вы находите гадостью: паттерн "goto Cleanup;" как таковой или if(0)?

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

[marknn.livejournal.com]

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

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

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

[vitus_wagner]

Ну, лично я не видел. А Петя Квитек - видел. И много и эмоционально об этом рассказывал.

[itman.livejournal.com]

goto куда бы то ни было.

[vitus_wagner]

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

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

[itman.livejournal.com]

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

[cmm.livejournal.com]

ну-ну.