Discussion :

[andyajram]

Bonjour , j'ai une petit problème , j'ai une requête avec une condition sur un Champ Date time , voici ma requête

Code :

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

1
2
select P.Projet,count(Remise) AS Nbre_Remises from Remise ,[Statsagents].[dbo].[Projet] AS P
where DateRemise= '04/09/2017'AND P.EtapeTraitement='AWB_CHQ_ALLER' group by P.Projet

quand j'utiliste la conversion

Code :

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

where CONVERT(VARCHAR, DateRemise, 120) like '%" + TextBox13.Text + "%'

le temps d'execution de la requête est très grand , mais quand j'utilise la requête comme ca dans Sql server

Code :

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

where DateRemise= '04/09/2017'

ca ne prend pas beaucoup de temps , comment puis-je indiquer au Textbox de prendre la valeur Date time comme je fais sur sql server ? parceque le textbox prend juste des string .


Merci d'avance

[andyajram]

c'est bon j'ai résolu le problème en mettant

Code :

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

 DateRemise = '" & TextBox13.Text & "'

Merci , même si personne ne m'a aidé

[François DORIN]

Merci , même si personne ne m'a aidé

Mais de rien

Plus sérieusement, il faut tout d'abord nous laisser le temps d'intervenir.

Ensuite, être dans le bon forum, ça aide. Ici, le forum accès aux données me semble plus approprié pour des requêtes SQL.

Enfin, il faut bien comprendre que les deux codes ne sont pas équivalent. Le premier est lent car il utilise l'opérateur LIKE avec le joker % en début de chaîne. Cela signifie que les index ne sont pas utilisable et qu'il va donc être nécessaire de parcourir toute la table. Avec l'opérateur d'égalité, les index sont utilisables.

De plus, dans la première requête toujours, tu convertis la colonne en texte pour faire le recherche. Dans la seconde, tu convertis la chaine en date pour faire la recherche. Bref, dans un cas, tu convertis une valeur, dans l'autre, tu convertis x valeur sans possibilité d'utiliser les index (x étant le nombre d'enregistrement de la table).

Pour terminer, il est très fortement déconseiller de construire à la mano des requêtes via des concaténations de chaînes, encore plus quand les données proviennent de l'utilisateur ! Il vaut mieux utiliser des requêtes paramétrées. C'est beaucoup plus sécurisés, et plus performant aussi.

[andyajram]

il est lent a cause du like oui mais quand j'ai essayer = dans management studio il n'est plus lent , mais pour indiquer le champ datetime a partir du textbox ca ne marché pas a cause du = parceque il prend la valeur string qui n'est pas égale au Date time donc voila

[andyajram]

je suis débutant , du coup je ne sais pas ce qui est sécurisé ou pas c'est pour ça je fais tous les essais jusqu’à ce que je trouve ce qui est mieux

[François DORIN]

il est lent a cause du like oui mais quand j'ai essayer = dans management studio il n'est plus lent , mais pour indiquer le champ datetime a partir du textbox ca ne marché pas a cause du = parceque il prend la valeur string qui n'est pas égale au Date time donc voila

Non. Lors de la comparaison string / datetime, SQL Server converti la chaîne de caractères en date (s'il n'y arrive pas, erreur), et effectue ensuite la comparaison sur les dates. Que la date vienne d'un textbox ou non ne change rien. Lors de l'écriture d'une requête SQL, la date prendra la forme d'une chaîne de caractères (que SQL Server convertira).

L'utilisation de requête paramétrée permet de s'abstraire de ce genre de considération. Tu lui passes un objet DateTime, il s'occupe tout seule de le convertir comme il faut. De plus, c'est sécurisé car prévient les injections SQL. Je t'invite donc à regarder de ce côté. Par exemple, un de nos membres avait écrit un article sur ce sujet

[andyajram]

d'accord je vais voir , merci pour tes conseils , ca me sera utile par la suite