Conseil sur une PS qui doit interroger X bases [Résolu]

[Raphael1980]

Bonne rencontre

Voici la situation, j’ai X bases de données de la même structure. Via une procédure stockée, j’aimerais interroger n’importe quelle base de données de mon choix.

Pour le moment, j’ai fait ceci qui fonctionne mais je me demande si c’est la bonne solution et surtout s’il n’y a pas plus propre. C'est juste histoire de ne pas me faire frapper par un DBA s'il me croise dans la rue.

J’ai crée une PS dans la master (ce n'est probablement pas propre... Dans mon cas, j'ai accès à la master mais dans une autre boite ça ne serait pas le cas) avec le code suivant :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
USE [master]
GO
/****** Object:  StoredProcedure [dbo].[test]    Script Date: 07/07/2011 17:46:32 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
ALTER PROCEDURE [dbo].[test] 
	-- Add the parameters for the stored procedure here
	@dbname varchar(30)
 
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;
	DECLARE @monSql char(100);
	SET @monSql = 'SELECT * FROM ' + @dbname + '.dbo.Clients'
	exec (@monSql)
 
    -- Insert statements for procedure here
 
END

D’avance, merci pour vos conseils.
Raphaël.

[elsuket]

Bonjour,

Est-ce que les bases de données sont hébergées par la même instance de SQL Server ?

Vous pouvez éventuellement passer aussi le nom de la table, mais ce qui est gênant dans votre procédure c'est le "étoile".
Mais c'est probablement pour l'exemple.

Vous pouvez coder un peu plus défensivement, et utiliser sp_executesql :

Code :

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
ALTER PROCEDURE dbo.test
	@dbname sysname
AS
BEGIN
	SET NOCOUNT ON
 
	IF @dbname IS NULL
	BEGIN
		RAISERROR('A database name has to be passed', 16, 1)
		RETURN
	END
 
	IF NOT EXISTS
	(
		SELECT	*
		FROM	sys.DATABASES
		WHERE	name = @dbname
		AND	state_desc <> 'ONLINE'
	)
	BEGIN
		RAISERROR('The ''%s'' database does not exist or is not available', 16, 1)
		RETURN
	END
 
	DECLARE @sql nvarchar(max)
	SET	@sql = 'SELECT * FROM ' + @dbname + '.dbo.Clients'
	EXEC	sp_executesql @sql
END

Une autre possibilité c'est de créer une procédure stockée, et de la marquer comme objet système.
De cette façon vous pouvez l'appeler dans n'importe quel contexte de bases de données (voyez le billet que j'ai écrit à ce sujet).
Vous êtes pour cela obligé de la créer dans la base de données master.

Je pense qu'en parler avec votre DBA ne mange pas de pain

@++

[Raphael1980]

Est-ce que les bases de données sont hébergées par la même instance de SQL Server ?

Oui, tout à fait.

Vous pouvez éventuellement passer aussi le nom de la table, mais ce qui est gênant dans votre procédure c'est le "étoile".
Mais c'est probablement pour l'exemple.

Oui, c'était simplement pour l'exemple et pour tester. En production, je n'utiliserai que quelques champs qui seront spécifiés.

Merci pour le code, je vais adapter le mien pour qu'il soit plus robuste.

Merci pour vos conseils et je vais lire votre billet qui me servira sans aucun doute dans un autre cas.