Discussion :

[Papy214]

Bonjour

J'ai créé une fonction pour indiquer l'existence d'une valeur dans une table:

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
 
CREATE FUNCTION [dbo].[CodebarreExiste] (@Codebarre VARCHAR(13))
RETURNS INT
AS
  BEGIN
      DECLARE @Result INT
 
      SELECT @Result = Count(*)
      FROM   catalogue
      WHERE  Codebarre = @Codebarre
 
      IF @Result <> 0
        SET @Result = 1
      ELSE
        SET @Result = 0
 
      RETURN @Result
  END

Je suis bien sur qu'il y a une manière plus courte d'écrire ça.

De plus, est-ce qu'il ne serait pas plus efficace de faire un simple select sans le count et utiliser @@ROWCOUNT ?

Qu'est-ce qui est le mieux ? Ou disons, le plus "normalisé" ??

Papy !

[rudib]

Code :

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

1
2
3
4
5
6
7
8
9
CREATE FUNCTION [dbo].[CodebarreExiste] (@Codebarre CHAR(13))
RETURNS BIT
AS BEGIN
  RETURN (
      SELECT CAST(Count(*) as BIT)
      FROM   catalogue
      WHERE  Codebarre = @Codebarre
  )
 END

ou bien mieux :

Code :

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

1
2
3
4
5
6
7
8
CREATE FUNCTION [dbo].[CodebarreExiste] (@Codebarre CHAR(13))
RETURNS BIT
AS BEGIN
  IF EXISTS (SELECT 1 FROM catalogue WHERE  Codebarre = @Codebarre)
      RETURN 1
 
   RETURN 0
 END

Attention aux performances : n'utilise pas cette fonction dans la partie SELECT d'une requête qui retourne beaucoup de lignes.

[Papy214]

Impeccable ! Je prends le "IF EXISTS".

Merci.