Discussion :

[Sengnapha]

Bonjour,

Je débute sur ce forum. Je rencontre un souci tout bête, je souhaite exécuter une requête avec la condition WHERE qui renvoi à une saisi de l'utilisateur. Voici mon code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
declare @debut_nom_table varchar(10) 
set @debut_nom_table  = @BASE

declare @NOM varchar(100) 
set @NOM  =  @NAME 

declare @requete_test varchar(MAX)

set @requete_test = 'select * FROM ' + @debut_nom_table + '_PERSONNE WHERE NOM_PERS=' + @NOM

exec (@requete_test)

L'utilisateur doit juste saisir le nom de la société (@debut_nom_table) et le nom de la personne (@NOM) pour avoir les info.
-Pour @debut_nom_table, on peut saisir sans apostrophe et ça fonctionne.
-Par contre pour @NOM, on doit saisir avec des apostrophes sinon ça fonctionne pas (car c'est un champ type texte).


Comment peut-on faire afin d'éviter que l'utilisateur n'a pas à saisir les apostrophes? genre il a juste à taper le nom DUPOND et non 'DUPOND'


Merci d'avance pour votre aide précieuse

[Rei Ichido]

Bonjour,

1. Il faudrait commencer par donner le SGBD. Ici je pense qu'il s'agit de SQL Server, vu la syntaxe.

2. Laisser les utilisateurs piloter ainsi la requête est dangereux, j'espère que vous faites des tests sur les variables (sinon, faites une petite recherche sur l'injection SQL).

3. Avoir une table par société est particulièrement moche, vous devriez avoir une seule table, avec une colonne société.

4. Utilisez sp_executesql :

Code :

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

 EXEC sp_executesql N'SELECT @test', N'@test nvarchar', @test=1

[escartefigue]

J'abonde totalement dans le sens des remarques qui précèdent, et j'ajoute que faire un select '*' est dangereux car le résultat est soumis aux aléas de modification de la table et le plus souvent inutile car vous transportez des colonnes dont vous n'avez probablement que faire.

[Sengnapha]

Bonjour,

Oui il s'agit du SQL Server.

Dans mon entreprise, chaque BU on leur propre serveur où est installé l'outil comptable...
Cet outil nomme ses tables commençant pas le nom de la BU+Nom de la table.

Ces requêtes sont des tests (d'où "*") qui me permettra de construire un rapport avec Visual studio que je les déploierai sur différents serveurs.

Merci pour votre aide. Je vais tester le code et je vous dirai si cela marche

[Sengnapha]

J'ai testé ce code:

Code :

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

1
2
3
4
EXECUTE sp_executesql 
          N'select * FROM ' + @debut_nom_table + 'PERSONNE WHERE NOM_PERS=@NOM' ,
          N'@NOM nvarchar',
          @NOM = 1;

Mais ça ne marche pas. Erreur au niveau de ' + @debut_nom_table + ' dont je ne sais pas résoudre

[aieeeuuuuu]

Bonjour,

Vous ne pouvez pas effectuer la concaténation au moment de l'appel de la procédure sp_executesql.

Générez votre SQL complet dans une variable puis appelez sp_executesql.

[Sengnapha]

Bonjour,

J'ai réussi
Merci beaucoup!

Voici le code que j'ai fait:

Code sql :

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

1
2
3
4
5
6
7
DECLARE @sqlCommand nvarchar(1000)
DECLARE @columnList varchar(75)
DECLARE @NOM varchar(75)
SET @columnList = @BU
SET @NOM = @NOMPERS
SET @sqlCommand = 'SELECT * FROM ' + @columnList + 'ERSONNE WHERE NOM_PERS = @NOM'
EXECUTE sp_executesql @sqlCommand, N'@city nvarchar(75)', @NOM = @NOM

Je vais pouvoir adapter cet exemple aux rapports souhaités.

Pas évident lorsqu'on n'a pas du tout de formation en IT/codage (je suis financier de base).

[SQLpro]

Il aurait été mille fois plus intelligent et dix mille fois plus performant de jouer sur les schémas sql que d'utiliser les 10 premiers caractère du nom des tables pour vos bu...
Le mieux aurait sans doute été d'ajouter une colonne pour vos code bu...

[Sengnapha]

Il aurait été mille fois plus intelligent et dix mille fois plus performant de jouer sur les schémas sql que d'utiliser les 10 premiers caractère du nom des tables pour vos bu...
Le mieux aurait sans doute été d'ajouter une colonne pour vos code bu...

En faite le @BU fait référence à une table contenant les codes BU d'une entité.

Mais bon l'outil INFOR n'a pas été configurer pour être performant. Il est assez mal fichu pour les multi BU:
- 1 Domaine = 1 Entité (Pays)
- 1 Entité = multi BU
- et chaque BU = 1 DB avec le nom en préfixe.

Ce serai trop simple d'avoir 1 DB avec la colonne BU