事务不匹配的SQL存储过程

时间:2016-12-07 07:26:24

标签: sql sql-server stored-procedures

我已经创建了一个存储过程,当一个错误命中它应该回滚一切,我一直在寻找,但我找不到弹出的错误。

  

[例外:在第130行的过程'ST_IV_ItemPrice.SP_Insert'中,源'.Net SqlClient数据提供程序'发生了意外的SQL错误,错误号为266.16.2.EXECUTE后的事务计数表示BEGIN和COMMIT语句的数量不匹配。先前的计数= 1,当前计数= 0。]

我希望我能在这里得到一些帮助。

CREATE PROCEDURE [dbo].[ST_IV_ItemPrice.SP_Insert]
    /*parameters*/
AS
BEGIN
    IF OBJECT_ID(''tempdb..#tempPriceList'') IS NOT NULL
        /*Then it exists*/
        DROP TABLE #tempPriceList
    CREATE TABLE #tempPriceList(PriceListid int, NewCurrencyUnitPrice     decimal(19, 5))

    DECLARE @Error int,
        @NextListid int,
        @NewCurrencyUnitPrice decimal(19, 5),
        @NoRounding int = 0,
        @UserSpecified int = 4



    BEGIN TRANSACTION 
    BEGIN TRY
        /*Insert item to table*/

        SET @Id = SCOPE_IDENTITY()
        SET @Error = @@ERROR


        IF @Error = 0 AND @AutoGenerate = 1
        BEGIN
            INSERT INTO #tempPriceList(PriceListid, NewCurrencyUnitPrice)
            VALUES(@Id, @Price) 

            WHILE(EXISTS(SELECT * FROM #tempPriceList))
            BEGIN


            SELECT TOP 1 @NextListid = [id], @NewCurrencyUnitPrice =      [NewCurrencyUnitPrice]
            FROM #tempPriceList

            /*INSERT SELECT STATEMENT*/

            INSERT INTO #tempPriceList ([PriceListid],[NewCurrencyUnitPrice])
            Select [ListId] , [NewCurrencyUnitPrice]


            IF @Error = 0 AND @SetExpiredDate = 1 AND @FromDate IS NOT NULL
            BEGIN
            /*Update item that is same as inserted and set to inactive*/


            END

            DELETE FROM #tempPriceList WHERE PriceListid = @NextListid

        END 
    END 
    END TRY
    BEGIN CATCH
        ROLLBACK TRANSACTION    
    END CATCH

    IF @@TRANCOUNT>0
        COMMIT TRANSACTION  

    RETURN @Error
END

1 个答案:

答案 0 :(得分:1)

CREATE PROCEDURE [dbo].[ST_IV_ItemPrice.SP_Insert]
    /*parameters*/
AS
BEGIN
    /* don't drop what does not belong to you */
    CREATE TABLE #tempPriceList(PriceListid int, NewCurrencyUnitPrice     decimal(19, 5))

    BEGIN TRY
        /* begin/commit within a single code block */
        BEGIN TRANSACTION 

        /* some code */

        /* we are here if everything is ok */
        COMMIT TRANSACTION  
    END 
    END TRY
    BEGIN CATCH
        /* check xact_state instead of @@trancount, because @@trancount may be >0 whilst you are unable to commit/rollback anything */
        IF XACT_STATE() IN (-1, 1)
            ROLLBACK TRANSACTION

        /* do not suppress exceptions! */
        RAISERROR(...)
    END CATCH

    /* it will be dropped after you leave SP scope, but you may clean all created here by yourself; this is not required */
    IF OBJECT_ID('tempdb..#tempPriceList') IS NOT NULL
        EXEC('DROP TABLE #tempPriceList')

    RETURN @Error
END