Discussion :

[JeanYvette]

Bonjour

Alors je sais pas si ce que je vais demander est possible ou même si c'est beau (je n'en suis pas totalement sûr) mais je vais essayer de m'expliquer.
Dans ma base de donnée, il y a une table qui recense l'historique des requêtes insert/update/delete dans cette même base. Cette table contient l'id du type de requete, la date de la requete, la table ciblé par la requete, l'id de l'utilisateur l'ayant effectuée ainsi que l'id cible dans la table. Par exemple, je modifier un centre ayant pour ID 2, j'aurais donc deux dans le champs id_cible de ma requête.
J'ai fait une première fonction me permettant de renvoyer plus précisément ma table historique requête, tout va bien, maintenant, le soucis arrive. J'avais dans l'idée de faire une fonction montrant le détail de la ligne infectée par la requête faite par l'utilisateur. Le soucis est le suivant :
Je n'ai, logiquement, pas le même nombre de colonnes dans ma table centre et dans ma table utilisateurs (par exemple), du coup, je ne peux pas savoir à l'avance combien de colonnes mettre dans ma fonction de type table, ni quel nom mettre à ces colonnes. Je voudrais donc savoir si il existait une méthode dynamique pour faire cela ou non

En espérant avoir été clair...

Merci

Ps : Si cela peut aider, j'avais eu dans l'idée de faire :

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
 
create function DetailHistorique(@IdHistorique int)
returns @DetailHistorique table
(
	idHistorique int identity(1,1)
)
as
	begin
 
		declare @Table varchar(50);
		declare @NomIdTable varchar(50);
		declare @IdCible int;
 
		set @Table = (select TableCible from HistoriqueRequete where Id_Historique = @IdHistorique);
		set @NomIdTable = (select Nom_id_table from Tables_bdd where Nom_table = @Table);
		set @IdCible = (select Id_cible_requete from HistoriqueRequete where id_Historique = @IdHistorique);
 
 
		select * into @DetailHistorique from @Table where @NomIdTable = @IdCible;
 
	return
	end
go

[aieeeuuuuu]

Code :

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

select * into @DetailHistorique from @Table

Ceci ne fonctionnera pas. Il faudrait passer par du SQL dynamique, ce qui n'est pas autorisé dans les fonctions.

quel est le but de tout cela ? comment compter vous utiliser cette fonction par la suite ?

[JeanYvette]

Bonjour,

Le but est le suivant, un administrateur se connecte sur le site (en asp.net) et voit la liste des historiques requêtes, présentant la date, type, table, idcible, et utilisateurs qui a fait la requête. Une fois sur cette page, il peut faire diverses recherches, ect... Tout ça, c'est fait
Maintenant ce que je veux c'est que notre administrateur, une fois qu'il a sélectionné la ligne qu'il veut "inspecter", il puisse en voir les détails, et ce que j'entend par détail, c'est la ligne dans la table sélectionnée. Donc par exemple si on a modifier notre centre 2, l'administrateur verra la ligne de l'enregistrement 2 de notre table centre

[escartefigue]

Et pourquoi n'utilisez vous pas des trigers en ce cas ?

Autre solution (que j'ai déjà rencontrée chez un client, mais qui n'a pas ma préférence) : avoir une table mouchard contenant un longvarchar qui stocke la concaténation des différentes colonnes de votre table d'origine avec une ligne pour la valeur avant et une autre pour la valeur après mise à jour. En cas d'insert, seule la valeur après existe, en cas de delete, seule la valeur avant.

[JeanYvette]

La solution vue chez votre client peut passer je pense, il faut que je vois
De base, je voulais passer par une fonction car c'est ce qu'on me demande, tout les create/update/delete doivent passer par une procédure stockée, et les select (notre cas) par des fonctions

[ypelissier]

Bonjour,
Pourquoi ne pas utiliser une procédure stockée pour un SELECT ? J'en avais fait une pour récupérer les libellé d'une table de telle sort que ce soit paramétrable

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
 
IF OBJECT_ID (N'dbo.MaPS', N'P') IS NOT NULL
	DROP PROCEDURE MaPS
GO
 
CREATE PROCEDURE MaPS
	@nvcTableName SYSNAME
,	@nvcLanguage typSmartCode = NULL
AS
BEGIN
	SET @nvcTableName = ISNULL(@nvcTableName, N'');
	SET @nvcLanguage = IIF(ISNULL(@nvcLanguage, N'') = N'', N'FR', @nvcLanguage);
 
	IF OBJECT_ID (N'dbo.' + @nvcTableName, N'U') IS NOT NULL
		EXEC (N'SELECT		T.nvcLabel
				  FROM	' + @nvcTableName + N' S
							INNER JOIN	tblTranslation T ON T.CdeLabel = S.CdeLabel
							INNER JOIN	tblLanguage L ON T.CdeLanguage = L.LID
				 WHERE	L.chrCodeISO2 = ''' + @nvcLanguage + N'''
				 ORDER
					BY	T.nvcLabel')
END;
GO

Ca marche nickel

Du coup je te proposerais bien ce 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
18
19
20
21
22
23
24
25
26
27
 
IF OBJECT_ID (N'dbo.MaPS', N'P') IS NOT NULL
	DROP PROCEDURE MaPS
GO
 
CREATE PROCEDURE MaPS
	@nvcTableName SYSNAME
,	@nvcLanguage typSmartCode = NULL
AS
BEGIN
	SET @nvcTableName = ISNULL(@nvcTableName, N'');
	SET @nvcLanguage = IIF(ISNULL(@nvcLanguage, N'') = N'', N'FR', @nvcLanguage);
 
	IF OBJECT_ID (N'dbo.' + @nvcTableName, N'U') IS NOT NULL
	BEGIN
		INSERT INTO MaTableHistorique
			VALUES (...);
		EXEC (N'SELECT	T.nvcLabel
				  FROM	' + @nvcTableName + N' S
							INNER JOIN	tblTranslation T ON T.CdeLabel = S.CdeLabel
							INNER JOIN	tblLanguage L ON T.CdeLanguage = L.LID
				 WHERE	L.chrCodeISO2 = ''' + @nvcLanguage + N'''
				 ORDER
					BY	T.nvcLabel')
	END;
END;
GO

[JeanYvette]

Bonjour,

Désolé du temps de réponse, et merci pour le code. Par contre je sais pas si je suis stupide ou quoi, mais je comprends pas vraiment le code que tu m'as donné

[ypelissier]

Arfff, c'est vrai qu'avec un peu de commentaires c'est toujours mieux

Pour mon cas, c'était assurer de récupérer les listes déroulantes d'une table dans la bonne langue.

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
-- Si la procédure existe alors la supprimer
IF OBJECT_ID (N'dbo.MaPS', N'P') IS NOT NULL
	DROP PROCEDURE MaPS
GO
 
-- Créer la procédure stockée
CREATE PROCEDURE MaPS
	@nvcTableName SYSNAME
,	@nvcLanguage typSmartCode = NULL
AS
BEGIN
	-- Controler que la table est bien renseigné, sinon mettre une chaine de caractères vide
	SET @nvcTableName = ISNULL(@nvcTableName, N'');
	-- Si la langue n'est pas renseignée alors prendre la langue par dé
	SET @nvcLanguage = IIF(ISNULL(@nvcLanguage, N'') = N'', N'FR', @nvcLanguage);
 
	-- Si la table existe alors récupérer la liste des labels de la table (cette ligne évite l'injection de code SQL par le nom de la table)
	IF OBJECT_ID (N'dbo.' + @nvcTableName, N'U') IS NOT NULL
		EXEC (N'SELECT		T.nvcLabel
			  FROM	' + @nvcTableName + N' S
					INNER JOIN	tblTranslation T ON T.CdeLabel = S.CdeLabel
					INNER JOIN	tblLanguage L ON T.CdeLanguage = L.LID
			 WHERE	L.chrCodeISO2 = ''' + @nvcLanguage + N'''
			 ORDER BY	T.nvcLabel')
END;
GO

Pour adapter à ton cas :

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
-- Si la procédure existe alors la supprimer
IF OBJECT_ID (N'dbo.MaPS', N'P') IS NOT NULL
	DROP PROCEDURE MaPS
GO
 
-- Créer la procédure stockée
CREATE PROCEDURE MaPS
	@nvcTableName SYSNAME
AS
BEGIN
	-- Si la table existe alors faire ce dont tu as besoin (cette ligne évite l'injection de code SQL par le nom de la table)
	IF OBJECT_ID (N'dbo.' + ISNULL(@nvcTableName, N''), N'U') IS NOT NULL
	BEGIN
		-- Historiser la demande de sélection
		INSERT INTO MaTableHistorique
			VALUES (...);
		-- Faire la sélection demandée de façon dynamique
		EXEC (N'SELECT	MesChamps
			  FROM	' + @nvcTableName + N'
			 WHERE	MaConditionCaractère = ''Toto''
			   AND	MaConditionNumérique = 5')
 /* Attention à bien mettre 2 fois la simple quote pour les chaines de caractères*/
	END;
END;
GO

[JeanYvette]

En effet, je test ça dès cet après midi je te dirais ce que ça aura donné
Merci

[ypelissier]

Alors ça a donné quoi ???

[ypelissier]

Youhou, il y a quelqu'un ??? Tu dis essayer et revenir me dire si c'est bon mais je ne vois personne revenir