T-SQL: Une bonne façon de fermer/curseur DEALLOCATE dans la gâchette

Question

mise à jour Disons que j'ai un déclencheur comme ceci:

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 

Je veux être sûr que Cursor1 est toujours fermé et désalloué. Même myProc1 ou myProc2 échoue. Dois-je utiliser try/catch block?

Answer 1

Oui, utilisez TRY/CATCH mais assurez-vous de libérer, etc. Malheureusement, il n'y a pas enfin dans SQL Server.

Cependant, je suggère enroulant cela dans un autre try/catch

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 

Que ce soit un curseur dans un déclencheur est une bonne idée est une autre affaire ...

Answer 2

Ce que vous devez faire est de ne jamais utiliser un curseur dans un déclencheur. Écrivez le bon code basé sur l'ensemble à la place. Si quelqu'un faisait une importation de données dans votre table de 100 000 nouveaux enregistrements, vous verrouiller la table pendant des heures et mettre votre base de données à un arrêt cinglant. C'est une très mauvaise pratique d'utiliser un curseur dans un déclencheur.

Answer 3

Vous pouvez utiliser le CURSOR_STATUS() fonction.

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

référence: http://msdn.microsoft.com/en-us/library/ms177609.aspx