T-SQL: Một cách thích hợp để CLOSE/deallocate con trỏ trong bản cập nhật cò

Question

Hãy nói rằng tôi đã có một kích hoạt như thế này:

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 

Tôi muốn chắc chắn rằng Cursor1 luôn đóng cửa và deallocated. Ngay cả myProc1 hoặc myProc2 đều thất bại.

Tôi có nên sử dụng khối try/catch không?

Answer 1

Có, sử dụng TRY/CATCH nhưng chắc chắn rằng bạn deallocate vv sau. Thật không may, không có cuối cùng trong SQL Server.

Tuy nhiên, tôi khuyên bạn nên gói này trong một thử/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 

Cho dù một con trỏ vào một kích hoạt là một ý tưởng tốt là một vấn đề khác ...

Answer 2

Điều bạn nên làm là không bao giờ sử dụng con trỏ trong trình kích hoạt. Thay vào đó, hãy viết mã dựa trên tập hợp đúng. Nếu ai đó đã nhập dữ liệu vào bảng 100.000 hồ sơ mới của bạn, bạn sẽ khóa bảng trong nhiều giờ và mang lại cơ sở dữ liệu của bạn để tạm dừng. Nó là một thực tế rất nghèo để sử dụng một con trỏ trong một kích hoạt.

Answer 3

Bạn có thể sử dụng CURSOR_STATUS() chức năng.

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

tham khảo: http://msdn.microsoft.com/en-us/library/ms177609.aspx