metaclass: (Default)
metaclass ([personal profile] metaclass) wrote2012-10-19 10:23 pm
Entry tags:

Бизнес-логика: Структура данных vs код

[livejournal.com profile] plumqqz доставил нам очередную ссылку на ужасы обсуждения БД на хабре. Сама статья еще куда ни шло, за исключением представления constraints как мега-фичи (вроде это основа основ любой БД). В комментариях, само собой, дикий ад, трэш, угар, содомия, коровники, заборы, катаформизм хабракармы и голые бизнес-аналитекши.

Самое ужасное там - это когда структуре данных отказывают в праве считаться частью бизнес-логики, упирая на то, что "надо же абстрагироваться от хранилища".
При этом, даже если добится этого абстрагирования - то все равно структура данных будет в памяти, в виде графа объектов, будет нужен дополнительный код по интеграции этой структуры с тем, что живет в БД, и никуда мы от структуры данных не денемся. А уж про то, можно ли считать констрейнты (в т.ч. и сложные, выражаемые только в виде триггеров) частью типов данных - можно дискутировать вечно. По крайней мере, check и foreign key constraints это точно часть типа данных, в случае FK - это еще и зависимые типы, реализованные в понятном виде. В случае unique constraints - уже сложнее, но по идее, тоже зависимый тип (зависит от данных таблицы, в которой он используется).

Так вот, я считаю, что структура данных для бизнес-логики гораздо более важна, чем собственно код. Если вам скажут, что теперь операция с кодом "777029" не облагается НДС - вам всего лишь нужно добавить веточку в паттерн-матчинг. Структура программы от этого не изменится и коллегам вы билд нахрен не сломаете.
А если окажется, что для того, чтобы узнать, как обрабатывается операция, вам нужно заглянуть в настройки в СУБД - то внезапно, в гламурно-причесанной функционально-иммутабельной функции проверки условий появляется грязная монада IO, провайдеры коннектов к БД, автоматическое управление транзакциями, пул коннектов, кэши и прочая черняга и структура данных (к коей я отношу так же и сигнатуры функций) меняется весьма заметно. Альтернативный вариант - оставить функцию чистой, но добавить к ней еще параметров, передаваемых извне, проверить код коллег, записать в трекер и рассказать, что теперь без передачи в нее оккультного набора параметров функция больше не заработает.

[identity profile] theiced.livejournal.com 2012-10-20 09:35 pm (UTC)(link)
Foo.where(:bar => 'bee').update_all(:huj => 'хуй')

[identity profile] vp.livejournal.com 2012-10-20 09:38 pm (UTC)(link)
Я не про апдейт, я про апдейт схемы базы.
У вас была версия 1.1, вы апдейтите изделие на 1.8
Как ваш чудо орм автоматически обновит ту базу, на которой работает?
в 1.8 у вас удалили половину полей, который были в 1.1 и создали другую половину.

[identity profile] theiced.livejournal.com 2012-10-20 09:41 pm (UTC)(link)
а мне вот интересно - как вы базу обновляете? я вообще принципиально апдейт схемы делаю только миграциями. а ормам похуй какая там схема - они метаданные вытягивают прямо оттуда.

[identity profile] metaclass.livejournal.com 2012-10-20 09:49 pm (UTC)(link)
Я делаю sql cкриптами.
А схема и прочее для ормов генерируется из модели, т.к. полное описание предметки в виде модели слишком сложное, чтобы его запихивать в любой частный язык программирования (за исключением разного рода лиспов, разве что)

[identity profile] vp.livejournal.com 2012-10-21 07:38 am (UTC)(link)
Ну дык для того, чтобы сделать апдейт схемы, необходимо эту схему представлять. И необходимо понимать, на каком состоянии у тебя находится обновляемая система к твоей текущей версии, до которой ты собираешься ее догнать.
Как ты это делаешь без понимания того, как устроена база? (мы только что постулировали, что только так и нужно и нам пофиг что там)

[identity profile] metaclass.livejournal.com 2012-10-21 08:08 am (UTC)(link)
В рубях ВСЯ схема базы делается в виде классов и миграций, т.е. вообще вся. И в базе есть таблица, аналогичная нашей, для трекинга версий
Т.е. оно банально один раз запускаешь rake db:migrate и апдейты на базу накатывается.

[identity profile] theiced.livejournal.com 2012-10-21 08:54 am (UTC)(link)
оно везде так. даже в жабе ;]

[identity profile] metaclass.livejournal.com 2012-10-21 09:05 am (UTC)(link)
На первой странице гугла по "java orm migrations" выводит только одну ссылку на эту тему, и та - порт activerecord :)

[identity profile] metaclass.livejournal.com 2012-10-20 09:45 pm (UTC)(link)
Что ты несешь, мы же обсуждаем апдейты данных, а не метаданных.

Для метаданных есть миграции в рубевой activerecord, а поскольку в базе логику и вообще хоть что-то осмысленное делать у них не принято, то функций типа "добавить/удалить таблицу" и "добавить/удалить поле" более чем достаточно :)

Т.е. вообще говоря, все эти ORM и прочая чернь - это способ упростить себе работу с базой, не вникая в ее тонкости. При этом приходится выкидывать все, что СУБД умеет делать сама, т.к. другие сервера это делают по другому.

[identity profile] theiced.livejournal.com 2012-10-20 09:50 pm (UTC)(link)
вот, до вас доходит же потиху. именно - ВЫКИНУТЬ К ХУЯМ всю это ненадёжную поебень.

[identity profile] metaclass.livejournal.com 2012-10-20 10:07 pm (UTC)(link)
А зачем ее выкидывать?
Констрейнты в виде FK,Unique и собственно структуры БД я оставлю по любому (по крайней мере, пока базы с ACID и типизированной функциональщиной не появится на горизонте).

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

[identity profile] theiced.livejournal.com 2012-10-20 10:15 pm (UTC)(link)
FK и прочая генерится ормом. триггера и хранимые говнопроцедуры не нужны же! об этом и разговор. обычно эти критические места - это различного рода отчёты и похуй сколько оно будет генериться - минуту или три.

[identity profile] metaclass.livejournal.com 2012-10-20 10:30 pm (UTC)(link)
Не, ты что. Меня ждать отчета даже 10 секунд бесит, а ты предлагаешь минуту :)

[identity profile] blackyblack.livejournal.com 2012-10-22 06:36 am (UTC)(link)
Гы. Ребе Б. ругает констрейнты, ребе А. ругает сторед проки, ребе М. использует всего по-маленьку. И все на меня псят за то, что я сказал, что форейн ключи не нужны. :)

[identity profile] vp.livejournal.com 2012-10-21 07:40 am (UTC)(link)
То есть как обычно есть золотая пуля и на самом деле вопрос выеденого яйца не стоит. Ок.

[identity profile] metaclass.livejournal.com 2012-10-21 08:09 am (UTC)(link)
Я более чем уверен, что если мы возьмем эту "золотую пулю", мы через неделю в накопаем в ней чернейших багов, а через месяц упремся в то, что "ДА ЭТОГО ЖЕ НИКТО ТАК НЕ ДЕЛАЕТ".

[identity profile] vp.livejournal.com 2012-10-21 08:16 am (UTC)(link)
Нет, мы упремся в то, что в нашей реальности этим можно пользоваться, но только допилив 50% вручную.

[identity profile] metaclass.livejournal.com 2012-10-21 09:06 am (UTC)(link)
Серьезно. У меня затыки вида "нужно идти на поклон к автору или чинить самому и потом поддерживать свои патчи поверх апстрима" возникают раз в неделю.

(Anonymous) 2012-10-22 08:35 am (UTC)(link)
Серебряная.