SQL Server 2000 多个 IF 语句

Question

当我尝试使用多个 IF 语句时出现错误。这是错误...

“消息 156，级别 15，状态 1，过程 fnTNAccidentIndicator，第 81 行 关键字附近的语法不正确 ‘结束’。”

这是我的代码的结构......

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

我在这里看到了很多问题。

1. 您有一个 nvarchar 类型的参数，但未声明大小。发生这种情况时，SQL Server 默认为 1 个字符，这可能不是您想要的。我鼓励您指定大小。

2. 您的函数返回一个 nvarchar，但您没有声明大小。

3. 您已声明 nvarchar 类型的局部变量，但未声明大小。

4. 您在不使用 set 关键字的情况下分配变量。 SET @ReturnValue = 'Y'

编辑

1. 您将 @inIndicatorMode 作为整数传递，然后将其与 nvarchar 变量进行比较。这将导致 SQL Server 在比较之前进行类型转换。类型转换通常非常快，但最好尽可能避免它们。

2. 您似乎只是为了比较的目的而对值进行硬编码。每个声明和变量赋值都需要少量的时间来执行。我认为这种方法的唯一优点是自记录代码。我建议您将比较更改为硬编码值，然后在代码中使用注释，而不是声明变量、分配变量，然后比较这些变量。

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

1. 此过程似乎只有 2 个有效返回：“N”或“Y”。我鼓励您稍微更改返回数据类型。这允许您编写其他代码，将此函数的输出视为布尔值而不是字符串。

2. 在您的帖子中，您展示了 3 个代码块。在代码中，您有“select 语句”的注释。我假设每个 select 语句都是不同的。之后您似乎还检查了 @@RowCount，如果行计数大于 0，则将函​​数的输出设置为“Y”。相反，我鼓励您使用 Exists 函数。使用exists 时会稍微提高性能，因为一旦SQL 找到满足查询的单行，它就会返回true。例如：

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

Answer 2

根据您的编辑。不能有空的 begin..end 块。它需要包含一些东西，即使只是一个打印语句或冗余赋值。

Answer 3

只要看看你的函数，我认为问题不在于多个 if 语句。看起来问题是您在每个 if 语句中执行 @ReturnValue = 'Y' （忘记 SET）。它应该是：

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

另外，看起来你在每个 if 中都做了同样的事情，你可以将 if 语句组合成一个带有 OR 的 if 。

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