Discussion :

[zemblamoh]

Bonjour,
En exemple j'ai les tables: tab1, tab2, tab3,..et je cherche par procédure stockée à ouvrir une au choix et y appliquer une selection. J'ai écris les instructions de cette manière, mais ça ne donne rien:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
28
29
ALTER PROCEDURE Recherche_Data
	(
		@jour int,
		@n int
	)
 
AS
	declare @d datetime
	set @d=getdate()-@jour
SET NOCOUNT ON
	if @n=1
	select tab1.*
	from tab1
	where
	date>@d
             ----
             if @n=2
	select Tab2.*
	from Tab2
	where
	date>@d
             ----
	if @n=3
	select Tab3.*
	from Tab3
	where
	date>@d
 
	RETURN

J'ai cherché des tuto proches de mon exemple mais je n'ai rien trouvé.
Que me manque-t-il dans cette structure?
Merci pour votre aide

[SQLpro]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
ALTER PROCEDURE Recherche_Data
	(
		@jour int,
		@n int
	)
 
AS
	declare @d datetime
	SET @d=getdate()-@jour
 
SET NOCOUNT ON
 
	SELECT tab1.*
	FROM   tab1
	WHERE  date>@d
	  AND  @n='tab1'
	UNION  ALL  
	SELECT Tab2.*
	FROM   Tab2
	WHERE  date>@d
	  AND  @n='tab2'	
	UNION  ALL  	  
	SELECT Tab3.*
	FROM   Tab3
	WHERE  date>@d
	  AND  @n='tab3'

A noter : DATE est un mot réservé de SQL. Vous ne devez jamais nommer une colonne avec un mot réservé de SQL comme DATE ou TYPE. Vous allez au devant de graves ennuis !


A +

[elsuket]

Bonjour,

On peut aussi le faire en SQL dynamique :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
ALTER PROCEDURE Recherche_Data
	@jour INT,
	@n INT 
AS
BEGIN
	SET NOCOUNT ON
 
	DECLARE @D DATETIME
	SELECT @d = GETDATE() - @jour
 
	DECLARE @SQL NVARCHAR(256),
			@ParamsDef NVARCHAR(64)
 
	SELECT @SQL = 'SELECT mesColonnes FROM dbo.Tab' + CAST(@n AS VARCHAR(3))
					+ ' WHERE date > @d_IN',
			@ParamsDef = '@d_IN DATETIME'
 
	EXEC sp_executeSQL
			@SQL,
			@ParamsDef,
			@d_IN = @d
END

RETURN est utilisé dans les procédures stockées :
- soit pour retourner la valeur d'une variable à l'appelant
- soit pour terminer l'exécution sans condition

@++

[zemblamoh]

Bonjour,
Merci pour vos réponses et éclaircissements.
A SQLPro, j'ai essayé votre code mais il me retourne un message d'erreur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Erreur ADO: Touttes les requêtes combinées à l'aide d'un opérateur UNION, INTERSECT, EXCEPT doivent posséder un même nombre d'expression dans leur liste cible

J'ai oublié de mentionner dans message précédent que les structures des tables sont différentes l'une de l'autre. En plus, mon @n est un intègre.
Je n'étais pas clair dans mon message. je ne cherche pas une Union. Je cherche dans cette procédure à afficher une des trois Tables avec les conditions @date (c'est juste pour l'exemple), une fois que je choisi @n sachant que n=1 me guidera vers tab1, n=2 vers tab2 etc...
A Elsuket, je ne vois pas dans votre code la solution à mon problème.
Est ce que je doit répéter ce code pour chaque table? et comment signaler que si @n=1 par exemple c'est la table tab1 qu'il faudra ouvrir.
Merci encore de votre aide et patience et excuser moi si je n'étais pas clair dans mon explication

[elsuket]

Bonjour,

A Elsuket, je ne vois pas dans votre code la solution à mon problème.
Est ce que je doit répéter ce code pour chaque table? et comment signaler que si @n=1 par exemple c'est la table tab1 qu'il faudra ouvrir.

Dans la procédure que je vous ai proposée, nous construisons la chaîne de requête que nous soumettons ensuite au moteur de base de données pour qu'il l'exécute, à l'aide de la procédure stockée système sp_executeSQL.

La concaténation dans la chaîne de requête de votre variable @n vous permet justement de ne pas préciser le numéro de la table que vous souhaitez interroger.

Vous pouvez tester le code seul en exécutant par exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
DECLARE @D DATETIME,
		@n INT,
		@jour INT
 
SELECT @jour = 11,
		@d = GETDATE() - @jour,
		@n = 2
 
DECLARE @SQL NVARCHAR(256),
		@ParamsDef NVARCHAR(64)
 
SELECT @SQL = 'SELECT mesColonnes FROM dbo.Tab' + CAST(@n AS VARCHAR(3))
				+ ' WHERE date > @d_IN',
		@ParamsDef = '@d_IN DATETIME'
 
EXEC sp_executeSQL
		@SQL,
		@ParamsDef,
		@d_IN = @d

Il vous suffit ensuite de tester plusieurs valeurs de @n

@++

[zemblamoh]

re bonjour;
Je vois très bien ce que vous dites mais, je ne comprends pas le code au niveau:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
SELECT @SQL = 'SELECT mesColonnes FROM dbo.Tab' + CAST(@n AS VARCHAR(3))
				+ ' WHERE date > @d_IN',
		@ParamsDef = '@d_IN DATETIME'

Excusez moi, c'est quoi mescolonnes From dbo.Tab? quel Tab justement Tab1, Tab2...
Je ne vois pas comment diriger le code vers l'une de mes tables.
Je pense que la réponse au code sera inchangé quel que soit @n vu que dbo.tab est unique.
J'espère que vous voyez ma difficulté à comprendre votre code!!
Merci encore

[elsuket]

En fait nous contruisons la chaîne de requête qui sera donc :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
SELECT mesColonnes FROM dbo.Tab' + CAST(@n AS VARCHAR(3))
				+ ' WHERE date > @d_IN

La variable @SQL est de type VARCHAR, donc on doit transtyper la variable @n à ce type pour être compatible avec le type de @SQL.
@n est ensuite concaténée à dbo.Tab, ce qui donnera à l'exécution dbo.Ta1 ou dbo.Tab2 ou dbo.Tab3, ...

La procédure stockée système sp_executeSQL permet d'exécuter une chaîne de requête (en l'occurence @SQL) paramétrée : d'où le paramètre @d_IN.
Je définis ce paramètre dans :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

@ParamsDef = '@d_IN DATETIME'

Ce qui me permet de le passer à sp_executeSQL dans cet appel :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
EXEC sp_executeSQL
		@SQL,
		@ParamsDef,
		@d_IN = @d

mesColonnes doit normalement être la liste des colonnes de la table à requêter.
Au départ j'avais supposé que vos tables avaient la même structure, mais ce n'est pas le cas.
Vous devez donc le remplacer par *, mais sachez que cela est contre-performant, sauf combiné avec COUNT() ou EXISTS, qui sont spécialement optimisées pour cela. En effet, cela obligé le moteur de base de données à interroger les tables de métadonnées pour connaître la liste des colonnes à retourner

J'espère que vous voyez ma difficulté à comprendre votre code!!

J'essaie d'expliquer le plus clairement possible, j'espère y parvenir

@++

[zemblamoh]

A Elsuket,
Je vois mieux maintenant. Le @n après concaténation s'ajoutera à tab comme un suffixe pour formet Tab1 et le tour est joué. C'est génial, comme astuce
Merci encore et bon week end

[zemblamoh]

Désolé, c'est toujours moi Elsuket;
j'ai appliqué votre code. J'ai juste changé @n int en @table nvarchar(64) parceque les noms de mes tables sont variables. Voici donc mon code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
ALTER PROCEDURE Recherche_D
		@jour int,
		@Table nvarchar(64)
AS
BEGIN
	SET NOCOUNT ON
	DECLARE  @D DATETIME
	SELECT @D=GETDATE()-@jour
	DECLARE @sql nvarchar(256),
		@ParamsDef nvarchar(64)
select @sql= 'select * from '+@Table+' Where date_sup>@D_IN',
			@ParamsDef='@D_IN datetime'
Exec sp_executeSQL 
		@sql,
		@ParamsDef,
		@D_In=@D
END

Quand j'execute, je réponds à @table par la table que je veux ouvrir, ça fontionne parfaitement mais malheureusement je n'obtiens rien même pas les titres des colonnes .
A la prochaine

[elsuket]

Bizarre : je viens d'exécuter le script de test suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
28
29
30
31
32
33
34
35
36
37
38
39
40
CREATE TABLE TEST
(
	date_sup DATETIME NOT NULL CONSTRAINT DF_TEST_date_sup DEFAULT (GETDATE()),
	uneValeur TINYINT
)
GO
 
INSERT INTO dbo.TEST (uneValeur) VALUES (1)
WAITFOR DELAY '00:00:05'
INSERT INTO dbo.TEST (uneValeur) VALUES (2)
WAITFOR DELAY '00:00:05'
INSERT INTO dbo.TEST (uneValeur) VALUES (3)
WAITFOR DELAY '00:00:05'
INSERT INTO dbo.TEST (uneValeur) VALUES (4)
GO
 
CREATE PROCEDURE Recherche_D
		@jour INT,
		@TABLE NVARCHAR(128)
AS
BEGIN
	SET NOCOUNT ON
 
	DECLARE @D DATETIME
	SELECT @D = GETDATE() - @jour
 
	DECLARE @sql NVARCHAR(256),
				@ParamsDef NVARCHAR(64)
 
	SELECT @sql = 'SELECT * FROM '+ @TABLE + ' WHERE date_sup > @D_IN',
				@ParamsDef = '@D_IN datetime'
 
	EXEC sp_executeSQL 
			@sql,
			@ParamsDef,
			@D_In=@D
END
GO
 
EXEC dbo.Recherche_D 1, N'TEST'

Et j'obtiens :

datesup uneValeur
------------------------------
2009-10-30 18:54:37.010 1
2009-10-30 18:54:42.020 2
2009-10-30 18:54:47.027 3
2009-10-30 18:54:52.033 4

@++

[zemblamoh]

Bonjour,
J'ai essayé ce matin mes tables, mais le problème persistait. Mes tables sur lesquelles je travaille n'ont pas de clef, je suis revenu sur une, j'ai fait la clef et le code a marché. Je suis revenu sur cette même table, j'ai enlevé de nouveau la clef et mystère ça a marché et depuis tout marche.
En tout cas merci, c'était très intéressant.