SQL存储过程中的事务不匹配

Question

我创建了一个存储过程，当出现错误时它应该回滚所有内容，我一直在寻找，但是我找不到不断弹出的错误。

[异常：源'.Net SqlClient数据提供者'在程序行'ST_IV_ItemPrice.SP_Insert'中的错误号266.16.2处发生意外的SQL错误。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

Answer 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