T-SQL: Правильный способ закрыть/DEALLOCATE курсор в триггере обновление

Question

Допустим, у меня есть триггер, как это:

CREATE TRIGGER trigger1 
    ON [dbo].[table1] 
    AFTER UPDATE 
AS 
BEGIN    
    --declare some vars 
    DECLARE @Col1 SMALLINT 
    DECLARE @Col1 TINYINT 

    --declare cursor   
    DECLARE Cursor1 CURSOR FOR 
    SELECT Col1, Col2 FROM INSERTED    

    --do the job 
    OPEN Cursor1 
    FETCH NEXT FROM Cursor1 INTO @Col1, @Col2 

    WHILE @@FETCH_STATUS = 0 
    BEGIN 
     IF ...something... 
     BEGIN   
      EXEC myProc1 @param1 = @Col1, @Param2 = @Col2 
     END    
     ELSE 
     IF ...something else... 
     BEGIN   
      EXEC myProc2 @param1 = @Col1, @Param2 = @Col2 
     END  

     FETCH NEXT FROM Cursor1 INTO @Col1, @Col2    
    END 

    --clean it up  
    CLOSE Cursor1 
    DEALLOCATE Cursor1     
END 

Я хочу быть уверен, что CURSOR1 всегда замкнут и освобождаться. Даже myProc1 или myProc2 терпят неудачу.

Должен ли я использовать блок try/catch?

Answer 1

Да, используйте TRY/CATCH, но убедитесь, что вы освобождаете и т.д. после. К сожалению, в SQL Server нет окончательного решения.

Однако я предлагаю оборачивать это в другой Try/уловом

CREATE TRIGGER trigger1 ON [dbo].[table1] AFTER UPDATE 
AS 
BEGIN       
    --declare some vars 
    DECLARE @Col1 SMALLINT, @Col1 TINYINT 

    BEGIN TRY 
     --declare cursor    
     DECLARE Cursor1 CURSOR FOR 
     SELECT Col1, Col2 FROM INSERTED      

     --do the job 
     OPEN Cursor1 
     FETCH NEXT FROM Cursor1 INTO @Col1, @Col2 

     WHILE @@FETCH_STATUS = 0 
     BEGIN 
      IF ...something... 
        EXEC myProc1 @param1 = @Col1, @Param2 = @Col2 
      ELSE 
      IF ...something else... 
        EXEC myProc2 @param1 = @Col1, @Param2 = @Col2 

      FETCH NEXT FROM Cursor1 INTO @Col1, @Col2        
     END 
    END TRY 
    BEGIN CATCH 
     --do what you have to 
    END CATCH 

    BEGIN TRY 
     --clean it up    
     CLOSE Cursor1 
     DEALLOCATE Cursor1         
    END TRY 
    BEGIN CATCH 
     --do nothing 
    END CATCH 
END 

ли курсор в триггер является хорошей идеей, это совсем другое дело ...

Answer 2

Что вы должны делать, никогда не используйте курсор в триггере. Вместо этого напишите правильный код на основе набора. Если бы кто-то импортировал данные в вашу таблицу из 100 000 новых записей, вы бы запирали таблицу в течение нескольких часов и приводили вашу базу к кричащей остановке. Очень плохо использовать курсор в триггере.

Answer 3

Вы можете использовать CURSOR_STATUS() функция.

if CURSOR_STATUS('global','cursor_name') >= 0 
begin 
close cursor_name 
    deallocate cursor_name 
end 

ссылка: http://msdn.microsoft.com/en-us/library/ms177609.aspx