Table paramteree dans un SELECT dans une stored procedure ?

Version imprimable

Bonjour a tous et merci de votre temps et de votre aide.

Je debute en MS SQL Server et j'ai une application avec 2 tables qui ont exactement les memes champs (note : je dois conserver les 2 tables).
Quand j'interroge l'une generalement j'interroge l'autre.
Dans une strored procdure voici que j'ai fait pour un de mes traitements.
Ceci est un exemple mais j'ai le cas tres souvant dans mon appli.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
         SELECT 
		T.Trip#, 
		Cast(T.DatePosted as DATETIME) AS DateCriteria, 
		(CASE WHEN C.[FollowUpNo] IS NULL AND T.[FollowUp_No]=0 THEN 2 -- It is a 0, so at least 2 notices have been sent
		 ELSE CASE WHEN C.[Level] = 1 THEN 1 -- 1st specified follow up
			  ELSE CASE WHEN C.[Level] > 1 THEN 2 -- 2nd or more specified follow up
			       END
			  END
		 END) as NoticeNumber
	FROM MY_DATA AS T 
		LEFT JOIN #TEMP_Config as C 
			ON T.[Region] =  C.[Region]
			   AND T.[Application] = C.[Source]
			   AND T.[FollowUp_No] = C.[FollowUpNo]
	WHERE ISDATE(T.DatePosted)=1
		  AND (CAST(T.DatePosted as DATETIME) BETWEEN @StartDate AND @EndDate)
		  AND T.[FollowUp_No] IS NOT NULL

Et la seule difference est ici :

Code:

FROM MY_DATA AS T

ou je vais changer les noms pour MY_DATA_AutreNom et actuellement ce que je fais c'est recopier le code et le modifier.

Ce que j'aimerai et ce sans me retrouver avec une quirielle d'objets dans ma BD c'est avoir la possibilite de passer le nom de la table en parametre.

Avoir quelque chose comme :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
         SELECT 
		T.Trip#, 
		Cast(T.DatePosted as DATETIME) AS DateCriteria, 
		(CASE WHEN C.[FollowUpNo] IS NULL AND T.[FollowUp_No]=0 THEN 2 -- It is a 0, so at least 2 notices have been sent
		 ELSE CASE WHEN C.[Level] = 1 THEN 1 -- 1st specified follow up
			  ELSE CASE WHEN C.[Level] > 1 THEN 2 -- 2nd or more specified follow up
			       END
			  END
		 END) as NoticeNumber
	FROM {{{Ici_La_Table}}} AS T 
		LEFT JOIN #TEMP_Config as C 
			ON T.[Region] =  C.[Region]
			   AND T.[Application] = C.[Source]
			   AND T.[FollowUp_No] = C.[FollowUpNo]
	WHERE ISDATE(T.DatePosted)=1
		  AND (CAST(T.DatePosted as DATETIME) BETWEEN @StartDate AND @EndDate)
		  AND T.[FollowUp_No] IS NOT NULL

Que recommandez-vous ?

A+

30/07/2018, 23h33
SQLpro

Il faut faire du SQL dynamique

A +
31/07/2018, 13h09
marot_r

Bonjour et merci de ta reponse.

C'est ce que j'avais finalement trouve et j'ai aussi trouve un article deconseillant en termes clairs de faire cela :

The Curse and Blessings of Dynamic SQL
http://www.sommarskog.se/dynamic_sql.html

Qu'en penses-tu ?

J'avoue etre surpris que MS SQL Server repose sur un mecanisme aussi peu fiable, car tellement sujet aux erreurs humaines et non validable par le compilateur, que la composition de SQL par le code.
Je travaillais majoritairement en Access ou ce genre de tour de passse passe est regulier mais tout le monde sait bien que Access n'est une "vraie" BDD :-) donc j'esperais quelque chose de plus sofistique dans MS SQL Server.

A+

Il n'y a aucun danger à utiliser du SQL dynamique si tu en contrôle les valeurs des arguments passés.

Dans ton cas c'est simple. Comme le paramètre est un nom de table (par exemple @TBL_SCH sysname , @TBL_NAME sysname), les premières lignes de ton code doivent être celles de contrôle, par exemple :
Code:

1 2 3 4 5 6 IF NOT EXISTS(SELECT * FROM INFORMATION_SCHEMA.TABLES WHERE TABLE_TYPE = 'BASE TABLE' AND TABLE_SCHEMA = @TBL_SCH AND TABLE_NAME = @TBL_NAME) RETURN;
Ne jamais retourner un code d'erreur précis afin de luter efficacement contre les injection de SQL.

A +

02/08/2018, 13h12
marot_r

Ok, merci.

A+