|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Коллеги, добрый день! В хранимой процедуре необходимо понимать, какие параметры не были переданы, а каким параметрам передано значение NULL (т.е. определить смысл значения NULL). Это необходимо для реализации логики обновления полей (если не передано значение - не трогаем, передано - обновляем). У кого-нибудь есть пример шаблона хранимой процедуры для решения данной задачи? Вижу 2 варианта. 1) Ввести входной параметр маски реально переданных параметров (т.е. вызывающая сторона уточняет перечень передаваемых параметров); 2) Определить специальные значения входных параметров, которые будут указывать, что передано пустое значение (т.е. default значение null означает отсутствие передачи параметра, а пустая строка, например для varchar, означает необходимость сбросить значение поля в БД в NULL). Спасибо) ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 13:38 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano, Вариант 3. Не выворачивать мозги. Передавать всегда и всё и обновлять тем, что передано. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 13:54 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano, Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9.
? ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 14:23 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Guf, авторВариант 3. Не выворачивать мозги. Передавать всегда и всё и обновлять тем, что передано. Есть сущности, содержащие более 100 столбцов. Объявлять овер 100 переменных и начитывать их перед EXEC - ну такое себе. И, часто, надо обновить лишь небольшую часть столбцов. Понятно, что бизнес-логика должна быть разбита по разным ХП, но не всегда и не везде это получается. Прямые UPDATE на таблицу не рассматриваются, т.к. пролетают мимо важных контролей внутри ХП. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 14:43 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
felix_ff Tketano, Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9.
? Это работает до тех пор, пока вам не надо "пропихнуть" NULL как реальное значение. Т.е. вы никогда не сможете обnullить значение в БД. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 14:45 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano, делайте тогда процедуру, у которой на входе будет названия поля и значение (или несколько в зависимости от типов данных, или с преобразованием в зависимости от типа поля целевой таблицы) и внутри процедуры пилите динамический sql на изменение конкретного поля или портянку на 100500 строк с условиями/case when ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 16:36 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
А есть еще вариант реализации, поражающий своим безграничным ужасом - формировать update нужного поля (или полей) на клиенте. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 16:42 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano, это кручу-верчу бред, сделайте все параметры обязательными для заполнения. За угадыванием надо к гадалке ходить, а не программы писать. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 17:52 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
andy st Tketano, делайте тогда процедуру, у которой на входе будет названия поля и значение (или несколько в зависимости от типов данных, или с преобразованием в зависимости от типа поля целевой таблицы) и внутри процедуры пилите динамический sql на изменение конкретного поля или портянку на 100500 строк с условиями/case when ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 18:03 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
PaulYoung andy st Tketano, делайте тогда процедуру, у которой на входе будет названия поля и значение (или несколько в зависимости от типов данных, или с преобразованием в зависимости от типа поля целевой таблицы) и внутри процедуры пилите динамический sql на изменение конкретного поля или портянку на 100500 строк с условиями/case when Человек хочет боль, человек получит боль ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 18:05 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Соглашаясь с тем что это боль поддерживаю вариант 1 с маской полей (или несколькими масками если полей много) ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 18:20 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Вы еще определитесь с тем, кто же по планам будет больше страдать в процессе разработки взаимодействия клиентского ПО с базой - разработчик клиентского софта или разработчик субд. Ну и к этому прикрутите их квалификацию. Если разраб клиента "эпичен", а разработчик субд из начинающих - пусть колбасят ad-hoc на update на клиенте, не такая сложная задача. Это если контроли в хп можно на триггеры заменить Если разраб клиента умеет подставлять параметры в процедуру, а разраб СУБД крут - любой вариант с сложной и поддерживаемой с болью логикой в процедуре, в том числе и вариант с параметрами "название поля" и "значение поля" - не факт что такой разработчик клиента осилит запилить маску полей для заполнения Если оба достаточно компетентны - развлекайтесь как вам угодно, благо возможностей для реализации такого глума тьма ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 19:06 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano felix_ff Tketano, Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9.
? Это работает до тех пор, пока вам не надо "пропихнуть" NULL как реальное значение. Т.е. вы никогда не сможете обnullить значение в БД. ну окей, объявите у параметров явные дефолты и сверяйтесь с ними Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9.
нужно правда придумать такие дефолты которые в принципе никогда не встретятся в качестве реальных значений таблицы. но имхо: по большей части вы занимаетесь ерундой, контроль правильности значений должен быть на стороне сервера приложений/клиентской части. сейчас в меня полетят ссаные тряпки адептов реализации бизнес логики на стороне СУБД ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 21:10 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
felix_ff нужно правда придумать такие дефолты которые в принципе никогда не встретятся в качестве реальных значений таблицы. гуиды вполне проканают, для параноиков - пара гуидов felix_ff но имхо: по большей части вы занимаетесь ерундой, контроль правильности значений должен быть на стороне сервера приложений/клиентской части. сейчас в меня полетят ссаные тряпки адептов реализации бизнес логики на стороне СУБД лови первый :) ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 21:20 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Принимать на входе XML/Json и формировать по нему динамический Update предлагали? Если плевать на производительность, то может прокатить. ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 21:36 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
А потом тредстартер прочитает что ему тут насоветовали и скажет: "да ну нахрен, пойду в дворники" ... |
|||
:
Нравится:
Не нравится:
|
|||
12.11.2021, 21:52 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
.Евгений Принимать на входе XML/Json и формировать по нему динамический Update предлагали? Если плевать на производительность, то может прокатить. хотите пострадать фигней? :) CLR процедуры сохраняют свои default value параметров в sys.parameters. если прямо сильно заморочиться можно нарисовать CLR обертку где дефолты будут явно тянуться при явном игноре входного параметра. другой вопрос: что скажут остальные разрабы когда создатель любой "мегофичи" уволится, а им потребуется насладится поддержкой его "высокого полета мысли" :) ... |
|||
:
Нравится:
Не нравится:
|
|||
13.11.2021, 01:43 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
felix_ff, трансляция XML в Update в общем не сложна, логика наглядна. Думаю, что поддерживать такое было бы менее противно, чем описанное вами. ... |
|||
:
Нравится:
Не нравится:
|
|||
13.11.2021, 16:17 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano Guf, авторВариант 3. Не выворачивать мозги. Передавать всегда и всё и обновлять тем, что передано. Есть сущности, содержащие более 100 столбцов. Объявлять овер 100 переменных и начитывать их перед EXEC - ну такое себе. И, часто, надо обновить лишь небольшую часть столбцов. Понятно, что бизнес-логика должна быть разбита по разным ХП, но не всегда и не везде это получается. Прямые UPDATE на таблицу не рассматриваются, т.к. пролетают мимо важных контролей внутри ХП. Люди-то в основном злые, не стоит расчитывать на понимание ... |
|||
:
Нравится:
Не нравится:
|
|||
13.11.2021, 20:05 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
felix_ff Tketano пропущено... Это работает до тех пор, пока вам не надо "пропихнуть" NULL как реальное значение. Т.е. вы никогда не сможете обnullить значение в БД. ну окей, объявите у параметров явные дефолты и сверяйтесь с ними Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9.
нужно правда придумать такие дефолты которые в принципе никогда не встретятся в качестве реальных значений таблицы. но имхо: по большей части вы занимаетесь ерундой, контроль правильности значений должен быть на стороне сервера приложений/клиентской части. сейчас в меня полетят ссаные тряпки адептов реализации бизнес логики на стороне СУБД Код: sql 1. 2. 3. 4. 5.
Универсальные лисапеды всегда громоздки... ... |
|||
:
Нравится:
Не нравится:
|
|||
14.11.2021, 11:28 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano И, часто, надо обновить лишь небольшую часть столбцов. Понятно, что бизнес-логика должна быть разбита по разным ХП, но не всегда и не везде это получается. Прямые UPDATE на таблицу не рассматриваются, т.к. пролетают мимо важных контролей внутри ХП. Или можно делать контроль в триггерах. Про всё это уже написали, я уверен, что эти решения будут лучше, чем изощрённая логика со 100 флагами. Проще для разработки, для расширения функционала, для сопровождения, и будут содержать меньше ошибок. ... |
|||
:
Нравится:
Не нравится:
|
|||
14.11.2021, 14:03 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
alexeyvg Tketano И, часто, надо обновить лишь небольшую часть столбцов. Понятно, что бизнес-логика должна быть разбита по разным ХП, но не всегда и не везде это получается. Прямые UPDATE на таблицу не рассматриваются, т.к. пролетают мимо важных контролей внутри ХП. Или можно делать контроль в триггерах. Про всё это уже написали, я уверен, что эти решения будут лучше, чем изощрённая логика со 100 флагами. Проще для разработки, для расширения функционала, для сопровождения, и будут содержать меньше ошибок. Подход "сделать процедуры для изменения любого набора полей" противоречит вашему же подходу "обрабатывать данные в СУБД, предоставляя API клиентам (серверам приложений и др.)" Вы хотите дать возможность делать ad-hoc запросы к базе, только через процедуры, что абсолютно лишено смысла как бизнес, так и технологического. Процедуры ради процедур. Вырождение идеи "СУБД предоставляет API для работы с данными". ... |
|||
:
Нравится:
Не нравится:
|
|||
14.11.2021, 14:11 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Код: sql 1.
это не может получаться только при отсутствии желания это делать. ... |
|||
:
Нравится:
Не нравится:
|
|||
14.11.2021, 21:46 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Всем спасибо за мнения! Динамический SQL явно мимо; финальные контроли осуществляются на сервере. Может изначально плохо пояснил задачу. Смысл в том, что есть много входных точек для изменения данных (api), реализующих разные форматы и разный набор полей. Соответственно вопрос был в обновлении информации в базе данных (после приема входящего запроса). Пока остановились на следующем варианте: Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20.
Хотя итоговое решение не понравилось (вызов такой ХП выглядит "неестественным"). Пока будет использоваться исключительно локально (для решения конкретной задачи). Возможно, часть контролей будет перенесена + прямой UPDATE таблицы + прочая логика процедуры usp_update. ... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 12:17 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Код: sql 1. 2. 3.
... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 12:49 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano, вы же сами указывали на проблему "пропихнуть" NULL как реальное значение" в вашем решение в случае Код: sql 1. 2. 3. 4. 5.
вы значение @val2 потеряете aleks222Универсальные лисапеды всегда громоздки... в процедуре 1000 параметров, добавите еще 1000? ... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 14:39 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
felix_ff, Да, в базовой процедуре update 100+ параметров, значит придется и в обертке объявить 100+ параметров и поставить 100+ case. Сейчас без обертки то не лучше - начитай недостающие 100 параметров c case из таблицы и потом сделай EXEC базовой процедуры. Тут же не стоит вопрос уменьшения кода (хотя этот вариант в любом случае короче), вопрос скорее в понятности и дальнейшей поддержке. Например, в базовую процедуру достаточно часто вводятся новые параметры. Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9.
При таком вызове да. Но я же буду вызывать обертку и значение сохранится: Код: sql 1. 2. 3. 4. 5.
А при передачи @val2 = '' значение будет сброшено в null. ... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 14:59 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano Всем спасибо за мнения! Динамический SQL явно мимо; финальные контроли осуществляются на сервере. Может изначально плохо пояснил задачу. Смысл в том, что есть много входных точек для изменения данных (api), реализующих разные форматы и разный набор полей. Соответственно вопрос был в обновлении информации в базе данных (после приема входящего запроса). Пока остановились на следующем варианте: Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20.
Хотя итоговое решение не понравилось (вызов такой ХП выглядит "неестественным"). Пока будет использоваться исключительно локально (для решения конкретной задачи). Возможно, часть контролей будет перенесена + прямой UPDATE таблицы + прочая логика процедуры usp_update. итоговое решение выглядит "нетрадиционным" в понятном смысле этого слова ... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 17:50 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano Пока остановились на следующем варианте: Ой, а сначала сказали, что так точно не подходит :-) Tketano Смысл в том, что есть много входных точек для изменения данных (api), реализующих разные форматы и разный набор полей. Вы хотите сделать одну входную точку для всего приложения? Поверьте, нехорошо получится Впрочем большинство учится только на своих ошибках, вы же (и никто из ваших коллег) ещё не сделали проект, который люди сейчас называют легаси? :-) Судя по стремлению к "универсальному коду" ... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 19:03 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Ролг Хупин итоговое решение выглядит "нетрадиционным" в понятном смысле этого слова Доктор, понимаете, мой муж работает программистом, и он последний год стал часто смотреть индусский код. А иногда, по ночам, когда думает, что я его не вижу, он его себе вставляет. И это ему нравится! ... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 19:09 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
alexeyvg Ролг Хупин итоговое решение выглядит "нетрадиционным" в понятном смысле этого слова Доктор, понимаете, мой муж работает программистом, и он последний год стал часто смотреть индусский код. А иногда, по ночам, когда думает, что я его не вижу, он его себе вставляет. И это ему нравится! (Y) ... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 20:33 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
alexeyvg, авторОй, а сначала сказали, что так точно не подходит :-)Нет, я такое не писал, Вы меня с кем-то перепутали =) Я сразу озвучил эту идею под вторым вариантом. И в серверной части итак хватает старого легаси-кода, в основном связанного с динамическим sql))) Хламить не хочется, даже для решения небольшой задачи. авторТак и делайте несколько входных точек (api). Вы хотите сделать одну входную точку для всего приложения? Поверьте, нехорошо получитсяНет. Немного love story. Есть базовая процедура для изменения сущности БД (usp_update в примере выше). На вход принимается полный перечень параметров и записывается в БД "как есть". При этом в процедуре реализуется прочая логика по проверкам, записи в логи и пр. Основная система в основном и дергает данную процедуру, т.к. почти всегда обладает информацией о полном состоянии объекта. Однако есть и другие точки входа (различные api) для коррекции информации (включая xml/json форматы). Там перечень параметров гораздо уже. Общий подход следующий: - если параметр не определен форматом, то оставляем текущее значение БД; - если значение параметра/тега указано, то записываем в БД (включая пустое значение => NULL); - если параметр/тег отсутствует, то оставляем текущее значение БД. И проблема как раз в том, что код в этих точках входа (api) уже весьма сложный и неочевидный, с большими свистоплясками, чтобы вызвать базовую процедуру или выполнить прямой UPDATE. В первом случае это начитка большого числа параметров (текущее состояние в БД), а во втором это вынесенные контроли + запись в логи и пр. логика (хотя сам UPDATE таблицы смотрится конечно максимально читабельно). Делать свой вариант usp_update под каждый API выглядит трудозатратным (тем более там небольшая разница от api к api). Какая-нибудь обертка поверх базовой usp_update, которую можно было бы дергать из каждой точки входа (api), значительно бы упростила логику. Вопрос открыт :-) ... |
|||
:
Нравится:
Не нравится:
|
|||
15.11.2021, 22:48 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
Tketano, В качестве идеи Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27.
... |
|||
:
Нравится:
Не нравится:
|
|||
16.11.2021, 09:50 |
|
Шаблон SP для определения переданных параметров
|
|||
---|---|---|---|
#18+
invm, Спасибо за мысль. XML собирать в некоторых случаях может быть чуть труднее, но безусловно логика процедуры становится более прозрачной. На самом деле в некоторые точки (api) на вход и прилетают такие xml :) ... |
|||
:
Нравится:
Не нравится:
|
|||
19.11.2021, 16:34 |
|
|
start [/forum/topic.php?all=1&fid=46&tid=1684094]: |
0ms |
get settings: |
9ms |
get forum list: |
13ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
54ms |
get topic data: |
12ms |
get forum data: |
3ms |
get page messages: |
61ms |
get tp. blocked users: |
2ms |
others: | 11ms |
total: | 173ms |
0 / 0 |