Discussion :

[Hujii]

Bonjour,

J'ai un petit souci pour pouvoir accéder à une colonne initialisée dynamiquement avec une @variable

Voici ma requête :

Code :

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

1
2
3
4
5
6
7
8
9
 
set @requete=' Select *
From dbo.Rights
Where [User_ID] = Upper('+@UserID+')
And Lib_ID = '+@Lib+'
And TypeDoc_ID = '+@TypeDoc+'
And '+ @TypeDroit +' = true'
 
exec sp_executesql @requete

Quand je l'exécute j'ai cette erreur (En affectant la valeur Read à @TypeDroit en c#) :

Code :

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

1
2
3
 
Msg*156, Niveau*15, État*1, Ligne*7
Syntaxe incorrecte vers le mot clé 'Read'.

Il prend en compte les ' ' que j'ai affecté dans ma requête (Sinon ça me dit que ma variable n'existe pas ...)


J'ai recherché ici, sur d'autres sites, mais je n'ai pas trouvé la solution à mon problème (il y a juste la solution pour pouvoir créer une colonne dynamiquement, et non y accéder)


Si quelqu'un pouvait m'aider ...
Merci d'avance

[elsuket]

Bonjour,

Quel est le type de la colonne Read ?
Si elle est de type VARCHAR ou CHAR, pourquoi ne pas avoir choisi BIT ?
Si elle est de type BIT, vous devez remplacer 'true' par 1.

N'utilisez pas * dans vos SELECT, c'est contre-performant.
Spécifiez toutes les colonnes, même s'il s'agit de toutes les colonnes de la table cible.

@++

[Hujii]

Bonjour,

Quel est le type de la colonne Read ?
Si elle est de type VARCHAR ou CHAR, pourquoi ne pas avoir choisi BIT ?
Si elle est de type BIT, vous devez remplacer 'true' par 1.

N'utilisez pas * dans vos SELECT, c'est contre-performant.
Spécifiez toutes les colonnes, même s'il s'agit de toutes les colonnes de la table cible.

@++

Merci pour ta réponse tout d'abord.

Type de ma colonne Read : bit
J'ai remplacé 'true' par 1, mais ça ne change rien car la colonne est en bit, donc 1 = true, 0 = false.
J'ai spécifié toutes les colonnes, je ne savais pas que c'était contre performant de mettre *

J'ai toujours la même erreur cependant ...

[elsuket]

a colonne est en bit, donc 1 = true, 0 = false.

Pas sûr que ça marche, mais si tu le dis ...
Pourtant il me semble qu'on doit écrire CAST('true' AS BIT) par exemple ...

Fais un PRINT @requete avant l'appel à sp_executeSQL pour voir la chaîne de requête que tu as construit

@++

[Hujii]

Non pas besoin, j'ai déjà testé plusieurs fois, il ne fait pas la différence entre true / 1 et false / 0 quand la colonne est de type bit !

Code :

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

1
2
3
4
5
6
7
8
9
10
 
Select ID, User_ID, Lib_ID, TypeDoc_ID, Private_Level [Create], [Read], [Modify], [Suppress]
From dbo.Rights
Where [User_ID] = Upper(MON_ID)
And Lib_ID = MA_LIBRAIRIE
And TypeDoc_ID = MON_TYPE
And Read = 1
 
Msg*156, Niveau*15, État*1, Ligne*7
Syntaxe incorrecte vers le mot clé 'Read'.

Voilà ...
Et quand je met des [ ] autour de ma variable @TypeDroit, ça me met :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
Select ID, User_ID, Lib_ID, TypeDoc_ID, Private_Level, [Create], [Read], [Modify], [Suppress]
From dbo.Rights
Where [User_ID] = Upper(MON_ID)
And Lib_ID = MA_LIBRAIRIE
And TypeDoc_ID = MON_TYPE
And [Read] = 1
 
Msg*207, Niveau*16, État*1, Ligne*5
Nom de colonne non valide*: 'MA_LIBRAIRIE'.
Msg*207, Niveau*16, État*1, Ligne*6
Nom de colonne non valide*: 'MON_TYPE'.

[elsuket]

Non pas besoin, j'ai déjà testé plusieurs fois, il ne fait pas la différence entre true / 1 et false / 0 quand la colonne est de type bit !

Ok, je tâcherai de m'en rappeler, il doit le caster automatiquement. Chanceux

La seconde requête est correcte parce que READ comme vous le voyez par la coloration syntaxique est un mot clé T-SQL.
Vous devez néanmois encadrer MA_LIBRAIRIE et MON_TYPE par des quote.
Comme vous avez fait du SQL dynamique, vous devrez en mettre deux ou trois de part et d'autre de chacun de ces deux valeurs

@++

[Hujii]

C'est ok, je viens de résoudre le problème, c'est bien à cause du 'Read' donc j'ai rajouté des [ ] comme j'avais fait, mais j'avais encore un souci ...

Il faut mettre 3 ' à droite et à gauche de chaque variable représentant une donnée (et non une colonne) ... (Normalement 1 suffit, mais bon là, c'est à cause du SQL dynamique si j'ai bien compris lol)

Code :

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

1
2
3
4
5
6
7
8
9
10
 
set @requete='
Select *
From dbo.Rights
Where [User_ID] = Upper('''+@UserID+''')
And Lib_ID = '''+@Lib+'''
And TypeDoc_ID = '''+@TypeDoc+'''
And ['+@TypeDroit+'] = 1'
 
exec sp_executesql @requete

Voilà la requête qui fonctionne

Merci beaucoup CTEMan

[elsuket]

Marquez le sujet comme résolu

@++