Discussion :

[Thomad]

Bonjour à tous,

Je sollicite votre aide car impossible de trouver la solution, voici mon problème.

J'ai besoin de mettre un DATEDIFF dans une clause WHERE

Code :

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

1
2
3
 
WHERE
DATEDIFF(day,MaReponse,GETDATE())<100

et je me retrouve avec l'erreur : Échec de la conversion de la date et/ou de l'heure à partir d'une chaîne de caractères.

Dans MaReponse la date est du genre : 20150105 , varchar(8)

Merci d'avance pour votre aide.

@+

[Invité]

Mauvaise modélisation... Mettre des dates dans des varchar, erreur de débutant !
Et forcément DATEDIFF compare des pommes avec des pommes et non des poires et de pommes...
Si c'est possible, modifier votre type de données.

Une solution de contournement serait :

Code :

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

1
2
declare @MaReponse varchar(8) = '20150105'
select DATEDIFF(day, cast(@MaReponse as datetime), GETDATE())

[SQLpro]

C'est normal... Un VARCHAR c'est du littéral et pas des date.
Commencez par revoir votre modèle en mettant du type DATE à la place de cet immonde varchar....

On ne vous à pas appris à l'école qu'on ne mélangeait pas les choux et les carottes dans les calculs ?

A +

[Thomad]

Je reprends un existant et je ne peux pas modifier le modèle. Petite erreur, c'est colonne reçoit divers types de réponses, sa taille est en réalité un varchar(50) et je ne peux pas la transformer en DATE.

Dois-je comprendre que je suis dans la m..de ?

Code :

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

1
2
 
CAST(...)

ne fonctionne pas !

[Invité]

Je reprends un existant et je ne peux pas modifier le modèle. Petite erreur, c'est colonne reçoit divers types de réponses, sa taille est en réalité un varchar(50) et je ne peux pas la transformer en DATE.

Un beau modèle de merde...
Bon, comme j'ai aussi fait les joies de ce genre d'héritage empoisonné, tu peux bricoler avec la fonction ISDATE ( https://msdn.microsoft.com/en-us/library/ms187347.aspx )
J'aurais tendance à faire une colonne calculée qui convertirait les données pourries en format DATE quand c'est possible.
Mais il faut garder à l'esprit que ça n'empêchera pas d'avoir des données convertibles en DATE alors que ça n'en ai pas vraiment une, comme un chiffre anodin genre 2015 ou 1999...

Code :

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

1
2
3
4
5
declare @MaReponse varchar(50) = '20150105'
select DATEDIFF(day, case when ISDATE(@MaReponse) = 1 then cast ( @MaReponse as DATE ) else NULL END, GETDATE())
 
SET @MaReponse = '2015'
select DATEDIFF(day, case when ISDATE(@MaReponse) = 1 then cast ( @MaReponse as DATE ) else NULL END, GETDATE())

[iberserk]

Toutes vos lignes sont-elles convertibles en DATE?
A propos quelle est votre version de SQL SERVER?
Si >=2008 faites un index filtré sur les lignes convertible en date, le cast devrait alors fonctionner car ne prendra pas en compte les lignes non convertibles...