Discussion :

[Boubou2020]

bonjour a tous
au début je suis débutant en terme transact ,mais j'ai commencer a apprendre les chose
Aujourd’hui j'ai eu une demande qui consiste a mettre en place un script sur un table nommé "Historique_Stock" qui permet de garder juste l’historique du Jour Actuelle et supprimer le reste ?
Pour cela j'ai penser a exécuter cette requête

Code :

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

1
2
delete FROM Historique_Stock 
WHERE DateCRE < DATEADD(day,-1,getdate());

ma collègue ma demander d’exécuter

Code :

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

1
2
delete FROM Historique_Stock 
WHERE DateCRE < convert(VARCHAR,getdate(), 103)

qu'elle est la requête la plus correcte qui permet de répond a ma besoin juste pour info le colonne DateCRE il est de type datetime

[julien94320]

la requete de ta collegue est bonne car elle va supprime toute les dates inférieur a la date du jour.

Si tu as un doute joue le script suivant et vérifie les données qui vont etre supprimé.

Code :

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

1
2
3
4
 
SELECT * 
FROM Historique_Stock 
WHERE DateCRE < convert(VARCHAR,getdate(), 103)

PERSO j'aurais meme joué la requete suivante si DATECRE est du type DateTime

Code :

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

1
2
3
4
 
SELECT * 
FROM Historique_Stock 
WHERE cast(DateCRE as date ) < cast(getdate() as date)

Mais j'ai une question pourquoi as tu chercher a calculer la date de la veille si tu cherches a supprimer toute les données inférieur a la date du jour ?

Code :

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

DATEADD(day,-1,getdate())

Cette fonction te retourne la veille du jour actuel

[aieeeuuuuu]

Bonjour,

Avec votre requête, si elle est exécutée un jour J à 18h, elle ne supprimera que les données antérieures au jour J-1 à 18h, ce qui ne correspond pas exactement à la demande.

Votre collègue a donc raison.

Pour la forme, j'éviterais toutefois un transtypage en varchar, et je ferais ainsi :

Code :

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

1
2
3
 
delete FROM Historique_Stock 
WHERE DateCRE < CAST(CAST(getdate() AS DATE) AS DATETIME)