Instructions IF multiples SQL Server 2000

Question

Je reçois une erreur lorsque j'essaie d'utiliser plusieurs instructions IF. Ceci est l'erreur ...

"Msg 156, Niveau 15, État 1, Procédure fnTNAccidentIndicator, Ligne 81 syntaxe incorrecte près du mot-clé 'END'."

C'est la structure de mon code ...

USE SS_TNRecords_Accident 
SET ANSI_NULLS ON 
GO 
SET QUOTED_IDENTIFIER ON 
GO 

CREATE FUNCTION dbo.fnTNAccidentIndicator 
(
    @inAccidentNumber nvarchar(100), 
    @inIndicatorMode int 
) 
RETURNS nvarchar 
AS 
BEGIN 
    DECLARE @AlcoholInd nvarchar(1) 
    DECLARE @DrugInd nvarchar(1) 
    DECLARE @SpeedInd nvarchar(1) 
    DECLARE @ReturnValue nvarchar(1) 

    SET @AlcoholInd = '1' 
    SET @DrugInd = '2' 
    SET @SpeedInd = '3' 
    SET @ReturnValue = 'N' 

    IF (@inIndicatorMode = @AlcoholInd) 
    BEGIN 
     SELECT AccidentNumber, AlcoholTestResult FROM tblAccidentUnit 
      WHERE AccidentNumber = @inAccidentNumber AND AlcoholTestResult NOT IN('00', '95', '96', '97', '98', '99') 
     UNION 
     SELECT AccidentNumber, AlcoholTestResult FROM tblAccidentOccupant 
      WHERE AccidentNumber = @inAccidentNumber AND AlcoholTestResult NOT IN('00', '95', '96', '97', '98', '99') 
     UNION 
     SELECT AccidentNumber, AlcoholTestResult FROM tblAccidentNonMotorist 
      WHERE AccidentNumber = @inAccidentNumber AND AlcoholTestResult NOT IN('00', '95', '96', '97', '98', '99') 
     IF (@@ROWCOUNT > 0) 
     BEGIN 
      SET @ReturnValue = 'Y' 
     END 
    END 

    IF (@inIndicatorMode = @DrugInd) 
    BEGIN 
     SELECT a.AccidentNumber,'AccidentUnit' AS TableFound, c.PrimaryKey AS TableKeyValue 
      FROM tblAccident a INNER JOIN tblAccidentUnit b 
       ON a.AccidentNumber = b.AccidentNumber INNER JOIN tblAccidentUnitDrug c 
        ON b.PrimaryKey = c.ForeignKey 
       AND a.AccidentNumber = '001' 
      WHERE c.DrugTestResult IN('02', '03', '04', '05', '06', '07', '08', '97', '98')  
      UNION  
      SELECT a.AccidentNumber, 'AccidentOccupant' AS TableFound, c.PrimaryKey AS TableKeyValue 
       FROM tblAccident a INNER JOIN tblAccidentOccupant b 
        ON a.AccidentNumber = b.AccidentNumber INNER JOIN tblAccidentOccupantDrug c 
         ON b.PrimaryKey = c.ForeignKey 
       AND a.AccidentNumber = '001' 
      WHERE c.DrugTestResult IN('02', '03', '04', '05', '06', '07', '08', '97', '98') 
      UNION 
      SELECT a.AccidentNumber, 'AccidentNonMotorist' AS TableFound, c.PrimaryKey AS TableKeyValue 
       FROM tblAccident a INNER JOIN tblAccidentNonMotorist b 
        ON a.AccidentNumber = b.AccidentNumber INNER JOIN tblAccidentNonMotoristDrug c 
         ON b.PrimaryKey = c.ForeignKey 
       AND a.AccidentNumber = '001' 
      WHERE c.DrugTestResult IN('02', '03', '04', '05', '06', '07', '08', '97', '98')  
     IF (@@ROWCOUNT > 0) 
     BEGIN 
      SET @ReturnValue = 'Y' 
     END     
    END 

    IF (@inIndicatorMode = @SpeedInd) 
    BEGIN 
     SELECT a.AccidentNumber,'AccidentUnit' AS TableFound, c.PrimaryKey AS TableKeyValue 
     FROM tblAccident a INNER JOIN tblAccidentUnit b 
      ON a.AccidentNumber = b.AccidentNumber INNER JOIN tblAccidentUnitDriverAction c 
       ON b.PrimaryKey = c.ForeignKey 
       AND a.AccidentNumber = '001' 
     WHERE c.DriverAction IN('28', '29') 
     IF (@@ROWCOUNT > 0) 
     BEGIN 
      SET @ReturnValue = 'Y' 
     END 
    END 

    Return @ReturnValue 

END 
GO 

Answer 1

Il y a quelques problèmes que je vois ici.

1. Vous avez un paramètre de type nvarchar sans une taille déclarée. Lorsque cela se produit, SQL Server par défaut à 1 caractère, ce qui n'est probablement pas ce que vous voulez. Je vous encourage à spécifier la taille.

2. Votre fonction renvoie un nvarchar, mais vous n'avez pas déclaré la taille.

3. Vous avez déclaré des variables locales de type nvarchar sans déclarer la taille.

4. Vous attribuez des variables sans utiliser le mot-clé défini. SET @ReturnValue = 'Y'

EDIT

1. Vous passez @inIndicatorMode comme un entier, mais on le compare à une variable nvarchar. Cela entraînera SQL Server à effectuer une conversion de type avant la comparaison. Les conversions de type sont généralement assez rapides, mais il est préférable de les éviter lorsque vous le pouvez.

2. Vous semblez être des valeurs de codage absolues uniquement à des fins de comparaison. Chaque déclaration et assignation de variable prendra un peu de temps à exécuter. Le seul avantage que je vois à cette méthode est le code auto-documenté. Au lieu de déclarer des variables, d'assigner des variables, puis de comparer ces variables, je suggère que vous changiez la comparaison en une valeur codée en dur, puis que vous utilisiez des commentaires dans le code.

IF (@inIndicatorMode = 1) --Alcohol Indicator 
BEGIN 
    --select statment 
    IF (@@ROWCOUNT > 0) 
    BEGIN 
    SET @ReturnValue = 'Y' 
    END 
END 

1. Il semble que il n'y a que deux déclarations valides de cette procédure, 'N' ou 'Y'. Je vous encourage à changer le type de données de retour à un peu. Cela vous permet d'écrire un autre code qui traite la sortie de cette fonction comme un booléen au lieu d'une chaîne.

2. Dans votre message, vous montrez 3 blocs de code. Dans le code, vous avez un commentaire de "select statement". Je suppose que chaque instruction select est différente. Vous semblez également vérifier le @@ RowCount par la suite, et si le nombre de lignes est supérieur à 0, vous définissez la sortie de la fonction sur 'Y'. Au lieu de cela, je vous encourage à utiliser la fonction Exists à la place. Il existe un léger gain de performances lorsque vous utilisez existe car il renvoie true dès que SQL trouve une seule ligne qui satisfait la requête.Par exemple:

IF (@inIndicatorMode = 1) -- Alcohol Indicator 
BEGIN 
    If Exists(-- Your select statement Here) 
    BEGIN 
    SET @ReturnValue = 'Y' 
    END 
END 

Answer 2

Après votre édition. Vous ne pouvez pas avoir un bloc begin..end vide. Il doit contenir quelque chose même si c'est juste une déclaration d'impression ou une affectation redondante.

Answer 3

Il suffit de regarder votre fonction, je ne pense pas que le problème est avec le multiple si les déclarations. Il semble que le problème est que vous faites @ReturnValue = 'Y' dans chaque instruction if (oubliant SET). Il devrait être:

IF (@@ROWCOUNT > 0)  
BEGIN   
    SET @ReturnValue = 'Y'  
END 

, il semble aussi que vous faites la même chose à l'intérieur de chaque cas, vous pouvez combiner les instructions if dans un seul cas avec salles d'opération.

IF (@inIndicatorMode = @AlcoholInd OR @inIndicatorMode = @DrugInd OR @inIndicatorMode = @SpeedInd) 
BEGIN  --select statment   
    IF (@@ROWCOUNT > 0)  
    BEGIN   
    SET @ReturnValue = 'Y'  
    END 
END