Discussion :

[Mavoune]

Bonjour à tous,

Je viens de commencer un nouveau CDI et, par conséquent, commence à analyser le code SQL de l'appli sur laquelle je vais bosser.
Je tique sur la façon dont la vérification des statuts de commandes est réalisée.

Le statut de la commande est stockée dans une colonne "Statut" en int.
A coté de ça, j'ai des fonctions du genre.

Code :

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

1
2
3
4
5
6
 
CREATE FUNCTION [dbo].[COM_CHECK_PREPARED]() RETURNS INT AS
BEGIN
	RETURN 0x00000010
END
GO

Et pour vérifier si une commande est préparée le test est fait comme ceci

Code :

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

1
2
3
 
IF (@STATUT&dbo.COM_CHECK_PREPARED())=0
...

Le statut doit donc être >= 16 et <=27 pour que la fonction renvoi 0. (Si une commande est préparée son statut est à 16)

Y'a t'il un intérêt à cette façon de faire plutôt qu'une fonction qui prendrait le statut en entrée et renverrait Vrai si le statut est à 16(où même compris entre 16 et 27) avec une condition
IF @STATUT = 16 ou IF @STATUT BETWEEN 16 AND 27 directement incluse dans la fonction.

J'ai du mal à saisir la logique/intérêt de la chose.
Surtout que des fonctions de ce type sont foison et la gymnastique cérébrale pour convertir les 0x00000040, 0x00000003 et compagnie n'est pas aisée

Merci pour vos réponses!

[aieeeuuuuu]

bonjour,

Code :

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

1
2
3
 
IF (@STATUT&dbo.COM_CHECK_PREPARED())=0
...

Le statut doit donc être >= 16 et <=27 pour que la fonction renvoi 0.

Non, c'est l'inverse ! ça renverra 0 si le statut est inférieur à 16, sinon renverra 16 (0x00000010)
et la borne supérieure n'est pas 27 mais 31.
Enfin, ça renverra aussi 16 si @STATUT est entre 48 et 63, ou 80 et 95....
bref ça renverra 16 si la division entiére de @STATUT par 16 est impaire, et 0 si elle est paire.

Pour ce qui est de la question de fond, le seul interet que j'y vois est de... dégrader les performances.

En effet prenons la requete qui renvoie le nombre de commades livrées :

Code :

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

1
2
3
4
 
SELECT COUNT(*)
FROM Commande
WHERE Statut &dbo.COM_CHECK_PREPARED() <> 0

Elle ne pourra pas utiliser d'index car il faut appliquer un calcul sur la colonne Statut
Elle ne pourra pas être parallélisée du fait de l'utilisation d'une fonction scalaire.

donc sera bien moins performante que

Code :

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

1
2
3
4
 
SELECT COUNT(*)
FROM Commande
WHERE Statut BETWEEN 16 AND 31 -- (en supposant qu'il n'y a en fait pas de statut > 48, sinon les deux requêtes ne sont plus équivalentes)

[Mavoune]

J'ai bien fait de tiquer du coup! Ça me semblait un peu bancal

Merci pour les précisions.

[mail.spam]

Bonjour,

Je débute un peu en SQL et je voulais avoir une info.
Est-ce que la mise en place d'une vue "Commande" avec un champ calculé, COM_CHECK_PREPARED de type BIT , permettrais d'amélioré encore les performances?

[aieeeuuuuu]

Si la vue n'est pas indexée, cela ne changera rien aux performances

[mail.spam]

Merci pour ce retour.
Il me reste à chercher un peu de la doc pour faire des vues indexées.