Discussion :

[Dr.SoFtNaF]

Bonjour Tout le monde,

J'ai rencontré un sérieux problème dans un projet.
Je vous explique la situation avant de procéder à un exemple concret:
Nous possédons une table Congés et on souhaite afficher la liste des personnes qui ont demandé le congé entre 2 dates.
Après la création de la procédure stockées, une erreur s'est produite sur la conversion de la date en nvarchar.

Si vous pouvez m'aider à résoudre ce petit problème dans cet exemple:
table Personne qui comporte comme Colonnes: idPersonne int, nomPersonne nvarchar(50) et dateNaissancePersonne DateTime,
La procédure qui permet d'extraire les personnes qui ont la date de naissance > à la date passée en argument:

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
 
--On Vérifie si la procédure existe déjà, si oui, on la supprime
IF EXISTS (SELECT name
       FROM sysobjects
       WHERE name = 'Procedure_sur_Personne'
       AND type = 'P')
    DROP PROCEDURE [dbo].[Procedure_sur_Personne]
GO
 
 
--On passe un argument DateTime
 
CREATE PROCEDURE [dbo].[Procedure_sur_Personne] (@DateNaissancePersonne DateTime = null)
AS
declare @sql nvarchar(300)
BEGIN
    --strip any time portion from the date passed in
    SELECT @DateNaissancePersonne = Convert(datetime,convert(char(10),@DateNaissancePersonne,101)) 
set @sql = 'SELECT * FROM dbo.Personne';
if @DateNaissancePersonne is not null
	set @sql =@sql + ' WHERE (dateNaissancePersonne > '+ @DateNaissancePersonne +')';
print @sql;
exec @sql;
 
END

Si la date a été passé en argument, on va ajouter la clause WHERE qui extrait les personnes par date, sinon on affiche tous les personnes.

Quand j'exécute la procédure stockée:

Code :

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

1
2
3
4
 
DECLARE @DN DateTime
SET @DN = '01/01/1970'
EXECUTE [dbo].[Procedure_sur_Personne] @DN

Je reçoi ce message d'erreur:

Msg*241, Niveau*16, État*1, Procédure*Procedure_sur_Personne, Ligne*13
Échec de la conversion d'une valeur datetime à partir d'une chaîne de caractères.

Si vous pouvez me résoudre ce problème, je serai très reconnaissant.

Merci BCP,

Cordialement Votre,

Naoufal

[kagemaru]

bonjour,

dans ton cas, je ne vois pas à quoi sert ta conversion en varchar. Je me contenterais d'un :

Code :

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

1
2
3
4
5
6
7
CREATE PROCEDURE [dbo].[Procedure_sur_Personne] (@DateNaissancePersonne DateTime = NULL)
AS
 
IF @DateNaissancePersonne IS NOT NULL
    SELECT * FROM dbo.Personne WHERE dateNaissancePersonne > @DateNaissancePersonne
else
     SELECT * FROM dbo.Personne

[Dr.SoFtNaF]

Merciiiiiii kagemaru, ça marche nikel, le petit problème a été résolu

Merci encore,

Naoufal

[kodo]

Salut
juste une remarque, le problème est résolu dans le cas ou il y a un seul critère. mais dans le cas ou il y a plusieurs il y aura un problème.
je pense qu'on doit chercher une solution, écriture d'une requête paramétrable ou personnalisé comme il a procédé Dr.SoFtNaF.
si tu es d'accord avec moi Dr.SoFtNaF, tu peux enlever le tag 'Résolu' ?

[Dr.SoFtNaF]

Mercii kodo pour l'initiative
En faite, après on a rencontré un gros problème comme kodo a signalé, on arrive à avoir dans notre where, plusieurs paramètres, alors la solution proposé ne résoudre pas le problème, si on s'amuse à mettre if elseif pour récrire toute la requête, à la fin on va arriver vers une grande procédure, on été tous entrain de chercher une solution est la posté ici, Tant que Mr. Kodo a pri l'initation, je mets cette article en débat pour avoir une solution finale.

Merciii encore Kodo

Cdlmt Votre,

Naoufal

[hmira]

Quelques observations :
1 – Ton code n’est pas protégé contre l’injection SQL. Pour te protéger de l’injection SQL, utilises autant que possible les paramètres

Code :

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

EXECUTE sp_executesql @SqlStmt, @ParamDefinition, @Param1=... , @Param2=...

Remarque : L’utilisation des paramètres te permet de résoudre les 2 problèmes que sont l’injection SQL et la multiplicité des paramètres telle que décrite dans ton précédent message.

2 – Pour les dates exprimées sous forme de chaine de caractère, et afin d’éviter les erreurs (échec de conversion ec..), il est fortement recommandé d’utiliser le format ISO 8601 :
yyyy-mm-ddThh:mm:ss[.mmm]

En effet ce format :
- correspond à une norme internationale,
- ne dépend du format de la date (ne dépend pas de SET DATEFORMAT)
- est multilingue (ne dépend pas de SET LANGUAGE)
- n'a pas besoin d'une fonction CAST (ou CONVERT) pour convertir la chaine en DATETIME.

Code :

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

1
2
3
DECLARE @DN DateTime
SET @DN = '1970-01-01T00:00:00'
EXECUTE [dbo].[Procedure_sur_Personne] @DN

A+