ReSQL.ru
     
powered by simpleCommunicator - 2.0.49     © 2025 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Microsoft SQL Server [игнор отключен] [закрыт для гостей] / Шаблон SP для определения переданных параметров
9 сообщений из 34, страница 2 из 2
  2  все
Шаблон SP для определения переданных параметров
    #40112088
felix_ff
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
felix_ff
Участник
Tketano,

вы же сами указывали на проблему "пропихнуть" NULL как реальное значение"

в вашем решение в случае
Код: sql
1.
2.
3.
4.
5.
exec dbo.usp_update
		@id	= @id,
		@val1	= @val1,
		@val2	= null,
		@val3	= @val3



вы значение @val2 потеряете


aleks222Универсальные лисапеды всегда громоздки...


в процедуре 1000 параметров, добавите еще 1000?
...
Рейтинг: 0 / 0
15.11.2021, 14:39
| Ответить | Цитировать | Написать  
Шаблон SP для определения переданных параметров
    #40112095
Tketano
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Tketano
Гость
felix_ff,

Да, в базовой процедуре update 100+ параметров, значит придется и в обертке объявить 100+ параметров и поставить 100+ case. Сейчас без обертки то не лучше - начитай недостающие 100 параметров c case из таблицы и потом сделай EXEC базовой процедуры. Тут же не стоит вопрос уменьшения кода (хотя этот вариант в любом случае короче), вопрос скорее в понятности и дальнейшей поддержке. Например, в базовую процедуру достаточно часто вводятся новые параметры.

Код: sql
1.
2.
3.
4.
5.
6.
7.
8.
9.
в вашем решение в случае
exec dbo.usp_update
		@id	= @id,
		@val1	= @val1,
		@val2	= null,
		@val3	= @val3


вы значение @val2 потеряете



При таком вызове да. Но я же буду вызывать обертку и значение сохранится:
Код: sql
1.
2.
3.
4.
5.
exec dbo.usp_update_some_rekv
		@id	= @id,
		@val1	= @val1,
		@val2	= null,
		@val3	= @val3



А при передачи @val2 = '' значение будет сброшено в null.
...
Рейтинг: 0 / 0
15.11.2021, 14:59
| Ответить | Цитировать | Написать  
Шаблон SP для определения переданных параметров
    #40112145
Ролг Хупин
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Ролг Хупин
Участник
Tketano
Всем спасибо за мнения! Динамический SQL явно мимо; финальные контроли осуществляются на сервере. Может изначально плохо пояснил задачу. Смысл в том, что есть много входных точек для изменения данных (api), реализующих разные форматы и разный набор полей. Соответственно вопрос был в обновлении информации в базе данных (после приема входящего запроса).
Пока остановились на следующем варианте:
Код: sql
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
create or alter procedure [usp_update_some_rekv]
	@id	int,
	@val1	int		= null,	-- -1 = сбросить в null (значение -1 не допускается бизнес-логикой)
	@val2	varchar(255)	= null,	-- '' = сбросить в null (пустая строка не допускается бизнес-логикой)
	@val3	date		= null	-- '' = сбросить в null (1900 год не допускается бизнес-логикой)
as

select	@val1 = case when @val1 = -1 then null else isnull(@val1, val1) end,
	@val2 = case when @val2 = '' then null else isnull(@val2, val2) end,
	@val3 = case when @val3 = '' then null else isnull(@val3, val3) end
from	[dbo].[tbl]
where	[id] = @id;

-- базовая процедура для записи параметров "как есть"
exec dbo.usp_update
		@id	= @id,
		@val1	= @val1,
		@val2	= @val2,
		@val3	= @val3
go



Хотя итоговое решение не понравилось (вызов такой ХП выглядит "неестественным"). Пока будет использоваться исключительно локально (для решения конкретной задачи). Возможно, часть контролей будет перенесена + прямой UPDATE таблицы + прочая логика процедуры usp_update.


итоговое решение выглядит "нетрадиционным" в понятном смысле этого слова
...
Рейтинг: 0 / 0
15.11.2021, 17:50
| Ответить | Цитировать | Написать  
Шаблон SP для определения переданных параметров
    #40112180
Фотография alexeyvg
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
alexeyvg
Участник
Tketano
Пока остановились на следующем варианте:

Ой, а сначала сказали, что так точно не подходит :-)
Tketano
Смысл в том, что есть много входных точек для изменения данных (api), реализующих разные форматы и разный набор полей.
Так и делайте несколько входных точек (api).
Вы хотите сделать одну входную точку для всего приложения? Поверьте, нехорошо получится
Впрочем большинство учится только на своих ошибках, вы же (и никто из ваших коллег) ещё не сделали проект, который люди сейчас называют легаси? :-) Судя по стремлению к "универсальному коду"
...
Рейтинг: 0 / 0
15.11.2021, 19:03
| Ответить | Цитировать | Написать  
Шаблон SP для определения переданных параметров
    #40112182
Фотография alexeyvg
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
alexeyvg
Участник
Ролг Хупин
итоговое решение выглядит "нетрадиционным" в понятном смысле этого слова
Да, понятность кода поражает.

Доктор, понимаете, мой муж работает программистом, и он последний год стал часто смотреть индусский код.
А иногда, по ночам, когда думает, что я его не вижу, он его себе вставляет. И это ему нравится!
...
Рейтинг: 0 / 0
15.11.2021, 19:09
| Ответить | Цитировать | Написать  
Шаблон SP для определения переданных параметров
    #40112201
Ролг Хупин
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Ролг Хупин
Участник
alexeyvg
Ролг Хупин
итоговое решение выглядит "нетрадиционным" в понятном смысле этого слова
Да, понятность кода поражает.

Доктор, понимаете, мой муж работает программистом, и он последний год стал часто смотреть индусский код.
А иногда, по ночам, когда думает, что я его не вижу, он его себе вставляет. И это ему нравится!


(Y)
...
Рейтинг: 0 / 0
15.11.2021, 20:33
| Ответить | Цитировать | Написать  
Шаблон SP для определения переданных параметров
    #40112215
Tketano
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Tketano
Гость
alexeyvg,

авторОй, а сначала сказали, что так точно не подходит :-)Нет, я такое не писал, Вы меня с кем-то перепутали =) Я сразу озвучил эту идею под вторым вариантом.
И в серверной части итак хватает старого легаси-кода, в основном связанного с динамическим sql))) Хламить не хочется, даже для решения небольшой задачи.

авторТак и делайте несколько входных точек (api).
Вы хотите сделать одну входную точку для всего приложения? Поверьте, нехорошо получитсяНет. Немного love story.
Есть базовая процедура для изменения сущности БД (usp_update в примере выше). На вход принимается полный перечень параметров и записывается в БД "как есть". При этом в процедуре реализуется прочая логика по проверкам, записи в логи и пр. Основная система в основном и дергает данную процедуру, т.к. почти всегда обладает информацией о полном состоянии объекта.
Однако есть и другие точки входа (различные api) для коррекции информации (включая xml/json форматы). Там перечень параметров гораздо уже. Общий подход следующий:
- если параметр не определен форматом, то оставляем текущее значение БД;
- если значение параметра/тега указано, то записываем в БД (включая пустое значение => NULL);
- если параметр/тег отсутствует, то оставляем текущее значение БД.
И проблема как раз в том, что код в этих точках входа (api) уже весьма сложный и неочевидный, с большими свистоплясками, чтобы вызвать базовую процедуру или выполнить прямой UPDATE. В первом случае это начитка большого числа параметров (текущее состояние в БД), а во втором это вынесенные контроли + запись в логи и пр. логика (хотя сам UPDATE таблицы смотрится конечно максимально читабельно). Делать свой вариант usp_update под каждый API выглядит трудозатратным (тем более там небольшая разница от api к api). Какая-нибудь обертка поверх базовой usp_update, которую можно было бы дергать из каждой точки входа (api), значительно бы упростила логику. Вопрос открыт :-)
...
Рейтинг: 0 / 0
15.11.2021, 22:48
| Ответить | Цитировать | Написать  
Шаблон SP для определения переданных параметров
    #40112245
invm
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
invm
Участник
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.
declare @t table (id int primary key, f1 int, f2 int, f3 int, f4 int);
insert into @t
values
 (1, 1, 2, 3, 4);

select * from @t;

declare @id int = 1, @f1 int, @f2 int, @f3 int, @f4 int;
declare @params xml = '<params f1 = "10" f4 = ""/>';

select
 @f1 = case when @params.exist('/params[1][@f1]') = 1 then @params.value('xs:integer(/params[1]/@f1)', 'int') else f1 end,
 @f2 = case when @params.exist('/params[1][@f2]') = 1 then @params.value('xs:integer(/params[1]/@f2)', 'int') else f2 end,
 @f3 = case when @params.exist('/params[1][@f3]') = 1 then @params.value('xs:integer(/params[1]/@f3)', 'int') else f3 end,
 @f4 = case when @params.exist('/params[1][@f4]') = 1 then @params.value('xs:integer(/params[1]/@f4)', 'int') else f4 end
from
 @t
where
 id = @id;

update @t
 set
  f1 = @f1, f2 = @f2, f3 = @f3, f4 = @f4
where
 id = @id;

select * from @t;
...
Рейтинг: 0 / 0
16.11.2021, 09:50
| Ответить | Цитировать | Написать  
Шаблон SP для определения переданных параметров
    #40113168
Tketano
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Tketano
Гость
invm,

Спасибо за мысль. XML собирать в некоторых случаях может быть чуть труднее, но безусловно логика процедуры становится более прозрачной. На самом деле в некоторые точки (api) на вход и прилетают такие xml :)
...
Рейтинг: 0 / 0
19.11.2021, 16:34
| Ответить | Цитировать | Написать  
9 сообщений из 34, страница 2 из 2
  2  все
Форумы / Microsoft SQL Server [игнор отключен] [закрыт для гостей] / Шаблон SP для определения переданных параметров
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
NoSQL.ru       Новости, Чат       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=46&gotonew=1&tid=1684094]:
 0ms
get settings:
 9ms
get forum list:
 15ms
check forum access:
 4ms
check topic access:
 4ms
track hit:
 169ms
get topic data:
 14ms
get first new msg:
 8ms
get forum data:
 3ms
get page messages:
 60ms
get tp. blocked users:
 2ms
others: 237ms
total:  525ms
созд. / загр.: 525ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]