Оптимизация запроса

[nactyx]

Всем доброго времени суток, коллеги!

В ходе будничной рутины я наткнулся на один "костыль", который у нас много где используется. И, смотря на него, я почему-то уверен, что его можно написать короче, оптимизировать что ли. Но не могу додумать как. Может кто-то сможет помочь/подсказать, буду очень признателен.

SELECT 
	TOP 1 
	UPPER(SUBSTRING(card.NAME_I, 1, 1)) + '.' + UPPER(SUBSTRING(card.NAME_O, 1, 1)) + '.' + UPPER(SUBSTRING(card.NAME, 1, 1)) + LOWER(SUBSTRING(card.NAME, 2, LEN(card.NAME))),
	(CASE WHEN id_chief_type in (1, 2, 9, 22, 30, 38, 39, 40, 44, 83, 110, 123, 126, 142, 163, 164, 177, 189, 207, 224) 
		THEN 
			(CASE WHEN ISNULL(prtb_chief.dop_inf, '') <> '' THEN (SELECT '(по '+ ISNULL(prtb_chief.dop_inf, '') + ')') END) 
		END)
FROM 
	prtb_chief  
	LEFT JOIN Card ON Card.auto_card = prtb_chief.auto_card
WHERE 
	card.auto_card = 476492
	and id_firm = 17
	AND id_chief_type in (1, 2, 9, 22, 30, 38, 39, 40, 44, 83, 110, 123, 126, 142, 163, 164, 177, 189, 207, 224)
	and dop_inf != '';

"Смущает" секция CASE. Ну может еще что можно реализовать получше.

Заранее благодарю всех и желаю отличной пятницы и хороших выходных!

[Аватар]

А зачем в 1-м case полностью дублируется условие из where? Во 2-ом зачем-то select

SELECT 
	TOP 1 
	UPPER(SUBSTRING(card.NAME_I, 1, 1)) + '.' + UPPER(SUBSTRING(card.NAME_O, 1, 1)) + '.' + UPPER(SUBSTRING(card.NAME, 1, 1)) + LOWER(SUBSTRING(card.NAME, 2, LEN(card.NAME))),
	CASE WHEN ISNULL(prtb_chief.dop_inf, '') <> '' THEN '(по '+ prtb_chief.dop_inf + ')' ELSE NULL END 
FROM 
	prtb_chief  
	LEFT JOIN Card ON Card.auto_card = prtb_chief.auto_card
WHERE 
	card.auto_card = 476492
	and id_firm = 17
	AND id_chief_type in (1, 2, 9, 22, 30, 38, 39, 40, 44, 83, 110, 123, 126, 142, 163, 164, 177, 189, 207, 224)
	and dop_inf != '';

Можно вообще dop_inf передать клиенту и пусть он вычисляемое поле сделает. Самое узкое место:

AND id_chief_type in (1, 2, 9, 22, 30, 38, 39, 40, 44, 83, 110, 123, 126, 142, 163, 164, 177, 189, 207, 224)

Считай что это равносильно тому, что запрос выполнится столько раз, сколько там констант. Ну почти) Занести их в отдельную табличку, прицепить её к запросу - на порядок эффективнее

Ну и, скорее всего, исходя из card.auto_card = 476492 там не left join, а inner join лучше будет

[nactyx]

Аватар, спасибо! Насчёт констант - подобные вещи следствие больших проблем) За которые никто не хочет браться и на которые закрывают глаза. Структурных проблем, если можно так выразиться.

А вот с иннер джоин хотелось бы подробнее, если возможно, именно на этом примере. Всегда была проблема с различием, как не научили нормально - так что-то только «лэфтом» и пользуюсь.

[nactyx]

Да и по доп_инф будто бы case не нужен. Там ведь в секции where то же неравенство. Или я не прав? Извиняюсь за корявость, с телефона пишу.

Хотя нет, там же изнулл.

[nactyx]

Можно закрыть, спасибо Аватару.