¿Cuál es la mejor práctica de manejo de errores de SQL Server T-SQL? [cerrado]

Actualmente usamos esta plantilla para cualquier consulta que ejecutemos (puede omitir el material de la Transacción, si no lo necesita, por ejemplo, en una declaración DDL):

BEGIN TRANSACTION
BEGIN TRY
    // do your SQL statements here

    COMMIT TRANSACTION
END TRY
BEGIN CATCH
    SELECT 
        ERROR_NUMBER() AS ErrorNumber,
        ERROR_SEVERITY() AS ErrorSeverity,
        ERROR_STATE() AS ErrorState,
        ERROR_PROCEDURE() AS ErrorProcedure,
        ERROR_LINE() AS ErrorLine,
        ERROR_MESSAGE() AS ErrorMessage

    ROLLBACK TRANSACTION
END CATCH

Por supuesto, puede insertar fácilmente la excepción capturada en su tabla de registro de errores.

Funciona muy bien para nosotros. Probablemente incluso podría automatizar parte de la conversión de sus antiguos procs almacenados a un nuevo formato utilizando la generación de código (por ejemplo, CodeSmith) o algún código C# personalizado.

No hay un conjunto de buenas prácticas para el manejo de errores. Todo se reduce a cuáles son sus necesidades y a ser consistentes.

Aquí hay una muestra de una tabla y un procedimiento almacenado que almacena números de teléfono.

 SET ANSI_NULLS ON
    GO
    SET QUOTED_IDENTIFIER ON
    GO
    SET ANSI_PADDING ON
    GO
    CREATE TABLE [dbo].[Phone](
        [ID] [int] IDENTITY(1,1) NOT NULL,
        [Phone_Type_ID] [int] NOT NULL,
        [Area_Code] [char](3) NOT NULL,
        [Exchange] [char](3) NOT NULL,
        [Number] [char](4) NOT NULL,
        [Extension] [varchar](6) NULL,
     CONSTRAINT [PK_Phone] PRIMARY KEY CLUSTERED 
    (
        [ID] ASC
    )WITH (PAD_INDEX  = OFF, STATISTICS_NORECOMPUTE  = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS  = ON, ALLOW_PAGE_LOCKS  = ON) ON [PRIMARY]
    ) ON [PRIMARY]

    GO
    SET ANSI_PADDING OFF
    GO
    /**/

    CREATE PROCEDURE [dbo].[usp_Phone_INS]
         @Customer_ID INT
        ,@Phone_Type_ID INT
        ,@Area_Code CHAR(3)
        ,@Exchange CHAR(3)
        ,@Number CHAR(4)
        ,@Extension VARCHAR(6)
    AS
    BEGIN
        SET NOCOUNT ON;

        DECLARE @Err INT, @Phone_ID INT

        BEGIN TRY
            INSERT INTO Phone
                (Phone_Type_ID, Area_Code, Exchange, Number, Extension)
            VALUES
                (@Phone_Type_ID, @Area_Code, @Exchange, @Number, @Extension)
            SET @Err = @@ERROR
            SET @Phone_ID = SCOPE_IDENTITY()
            /* 
                Custom error handling expected by the application.
                If Err = 0 then its good or no error, if its -1 or something else then something bad happened.
            */
            SELECT ISNULL(@Err,-1) AS Err, @Phone_ID
        END TRY
        BEGIN CATCH
            IF (XACT_STATE() <> 0)
                BEGIN
                    ROLLBACK TRANSACTION
                END

            /* 
                Add your own custom error handling here to return the passed in paramters. 
                I have removed my custom error halding code that deals with returning the passed in parameter values.
            */

            SELECT ERROR_NUMBER() AS Err, ISNULL(@Phone_ID,-1) AS ID
        END CATCH
    END

Parece que ya tienes un buen manejo de esto. Sospecho que usted está haciendo más del 95% de los programadores SQL por ahí.

Aquí encontrará información interesante:

• Detección y Notificación de Errores en Procedimientos Almacenados-Parte 1: SQL Server 2000
• Detección y Reporte de Errores en Procedimientos Almacenados-Parte 2: SQL Server 2005 TRY-CATCH Blocks *

Una sugerencia [no relacionada]: comience a usar ' ' en lugar de '!='.

[*SQL Junkies ha desaparecido, por lo que el segundo artículo no está disponible. Voy a tratar de conseguir que se vuelva a publicar en algún lugar y actualizar el enlace.]