Осенило

[gornal]

Любимое разработчиками слово "костыль" означает ровно необходимость применения оператора if. "Избавление от костылей" - если не упрощение функционала, то добавление уровня абстракции, избавляющего от использования if через, например, виртуальные функции.

Comments

[http://users.livejournal.com/_winnie/]

Мой опыт - каждый if это полбага в трекере (и виртуальной функции это тоже касается).

Костыль - это не совсем if. Костыль - это причина потери времени разработчика на одновременно неинтересную и ненужную работу.

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

[gornal.livejournal.com]

Я бы поправил так:
... и ненужную _с точки зрения разработчика_ ...

[vanderdecken-lj.livejournal.com]

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

[alll]

> добавление уровня абстракции, избавляющего от использования if через, например, виртуальные функции

Использование виртуальных функций вовсе не избавляет от "if", оно просто уменьшает их количество/дублирование. Потому как один раз тип объекта всё-равно придётся назначить "вручную", пусть даже и упрятав этот выбор где-нибудь в xml-клоаке. :)

[gornal.livejournal.com]

В xml-клоаке if'ов не будет ;-)

[alll]

Там обычно наихудший из всех возможных ифов - тот, который существует исключительно в голове одного конкретного разработчика. :) Недокументированый и немотивированый.

[kouzdra.livejournal.com]

Оно позволяет if реализовать :)))

Это как и в функциональном программировании быстро оказывается, что спагетти-код с кучей goto напрямую транслируется в группу взаимно-рекурсивных функций с хвостовыми вызовами (естественно от такой трансляции не становясь ни на йоту читабельнее - скорее наоборот - потому что goto по крайней мере точно goto - а там еще понять надо что это был goto)

[kouzdra.livejournal.com]

Напомню только что if элементарно реализуется через виртуальные функции: в Smalltalk он именно так и реализован:

Там есть класс bool с методами ifTrue и ifFalse, которые вычисляют значение своего параметра. И наследники этого класса с инстансами true и false с соответствующей реализацией этих методов :)))

[oboguev.livejournal.com]

"Но под одеждой-то они все равно голые".
То есть в ассмблере там все равно появляется condition branch.
Не уверен, в чем состоит смысл его тщательно прятать.
(Я-то конечно вообще нос чайником: например не испытываю терзаний по поводу оператора goto и употребляю его когда уместно.)

[beldmit.livejournal.com]

Не совсем. Костыль - это оператор if с одним и тем же условием во многих местах.

[gornal.livejournal.com]

Показываем 5 индикаторов за выбранное число.
Если выбранное число сегодня, а часть индикаторов не успели подсчитаться, показываем эту часть за вчера.

Это костыль? Многие скажут "да", хотя условие уникально.

[beldmit.livejournal.com]

Это не костыль, это бизнес-логика. Зверь отдельный, хотя повадки похожи.

[gornal.livejournal.com]

Формулировки "не хочу это делать, это бизнес-логика" я пока не слышал :-)

[beldmit.livejournal.com]

А "Есть логика, есть женская логика, есть бизнес-логика".

[gornal.livejournal.com]

Тоже нет, но мне понравилось.

[beldmit.livejournal.com]

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

1. Было только A. Потом добавилось Б. Пример - алгоритмы ЭЦП и код openssl до версии 1.0, где у тебя куча мест, которые имеют вид case RSA: case DSA: case EC:

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

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

[dmih.livejournal.com]

Я про второй случай что-то не понял. Выкидыванием чего?

[beldmit.livejournal.com]

"А эта хрень у нас с тех пор, когда маркетоид Вася (как, ты не помнишь? А, он до тебя уволился) протолкнул акцию. 5 лет назад. Да, наверное, лучше выкинуть тихо, пока начальство не видит. Акции уже три года нет, а ключевые блоки переписал Саша, с которым ты пересёкся на испытательном."

[dmih.livejournal.com]

Мне кажется, не стоит путать написание OpenSSL-я и бизнес-приложения всё-таки :)

[beldmit.livejournal.com]

Оно конечно. Но кейсы первого типа бывают и в бизнес-приложениях, и в системных. А кейсы второго - только в бизнесе.

В OpenSSL есть свои артефакты - типа совместимости с компилятором gcc 2.95 на платформе, которую живьём никто уже не видел, но бОльшая часть их должна в скором времени нас покинуть.

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

[green-shaman.livejournal.com]

...выливающееся впоследствии в переписывание всего и вся если с абстракцией не угадали.

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