存储过程在被另一个存储过程调用时未返回正确的值

时间:2017-07-03 08:39:46

标签: sql sql-server

我在另一个存储过程中执行存储过程时遇到问题。基本上,当我自己执行我的存储过程时,它工作正常,并插入并返回正确的值。但是,当我在另一个存储过程中调用相同的存储过程时,使用与我手动尝试时相同的输入,它不会插入任何内容,也不会返回正确的值。是否有一些奇怪的变量处理方式?还是出了别的问题?请参阅下面的代码。

第一个程序:

ALTER PROCEDURE [dbo].[createOrgLevel]
    @name varchar(255),
    @level int,
    @parentid bigint,
    @newid bigint OUTPUT
AS
    SET NOCOUNT ON;
    -- Check to see if it exists
    SELECT @newid = [id] from dbo.[Org 3]
        WHERE
        [Name] = @name and
        [Parent ID] = @parentid
        IF @newid IS NULL
        -- If it doesn't exist, insert
            BEGIN
                INSERT INTO [dbo].[Org 3]
                   ([Parent ID]
                   ,[Name]
                   ,[Level])
                VALUES
                   (@parentid
                   ,@name
                   ,@level)
                SET @newid = @@identity
            END
END

调用它的过程的缩写版本:

ALTER PROCEDURE [dbo].[createOrg]
    @level1 nvarchar(255),
    @level2 nvarchar(255),
    @level3 nvarchar(255),
    @level4 nvarchar(255),
    @level5 nvarchar(255),
    @level6 nvarchar(255),
    @level7 nvarchar(255),
    @level8 nvarchar(255),
    @orgid bigint OUTPUT
AS
BEGIN
    SET NOCOUNT ON;

    DECLARE
    @parid bigint,
    @curname nvarchar(255)
    DECLARE
    @levels table (num int, name nvarchar(255))
    INSERT @levels(num, name) values (1,@level1),(2,@level2),(3,dbo.@level3),
    (4,@level4),(5,@level5),(6,@level6),(7,@level7),
    (8,@level8)

    BEGIN
        BEGIN TRY
            SET @curname = @level1
            EXEC dbo.createOrgLevel @name = @curname, @level = 1, @parentid = 0, @newid = @orgid OUTPUT
            SET @parid = @orgid
        END TRY
        BEGIN CATCH
            SET @orgid = NULL
            RETURN -1
        END CATCH

        DECLARE @cnt INT = 2;

        WHILE @cnt < 9
        BEGIN
            BEGIN TRY
                SELECT @curname = name FROM @levels WHERE num = @cnt
    -----------------------------------
    -- This is where it is executed, and not inserting/returning the correct value. It keeps
    -- returning the value from the previous 'loop', and not inserting anything. It's not
    -- entering the Catch, though.

                EXEC dbo.createOrgLevel @name = @curname, @level = @cnt, @parentid = @parid, @newid = @orgid OUTPUT

    -----------------------------------
                SET @parid = @orgid
            END TRY
            BEGIN CATCH
                RETURN 1
            END CATCH
            SET @cnt = @cnt + 1
        END
    END
END

同样,如果我手动执行第一个过程,它会正确插入组织级别,并返回ID。但是,当在第二个程序中调用它时,它不会插入任何新的组织级别,只返回它找到的第一个组织级别的ID(我手动插入的所有组织的父级别)。 / p>

我尝试通过在第二个程序执行过程之前打印出所有参数来解决此问题,它们与我手动尝试时完全相同。我还检查了错误发生的位置,并且它正在进入catch语句,但它不是。任何人都可以帮助我吗?

1 个答案:

答案 0 :(得分:3)

是的,这是因为您在每次迭代中重构@newid变量,并且在每次连续运行时,它的上次运行值仍然存储在其中。

你的问题实际上在这里:

   SELECT @newid = [id] from dbo.[Org 3]
        WHERE
        [Name] = @name and
        [Parent ID] = @parentid

@newid已经有了一个值,由于查询没有返回任何内容,因此NULL不会覆盖值,但会保留上次值。

你应该做的是:

  • 在内部存储过程开始时将@newid设置为NULL 或
  • 使用变量的不同名称来检查组织是否存在然后返回输出的那个(设置SP末尾的返回值)或
  • 您可以将SELECT重写为SET,如果没有返回任何内容,将使用NULL覆盖值

     SET @newid = (SELECT id FROM dbo.[Org 3]..)