Quelle est la meilleure utilisation de la gestion des erreurs SQL Server T-SQL? [fermé]

Nous utilisons actuellement Ce modèle pour toutes les requêtes que nous exécutons (vous pouvez omettre les éléments de Transaction, si vous n'en avez pas besoin dans une instruction DDL, par exemple):

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

Bien sûr, vous pouvez facilement insérer l'exception interceptée dans votre table de journal des erreurs.

Ça marche vraiment bien pour nous. Vous pourriez probablement même automatiser une partie de la conversion de vos anciens procs stockés vers un nouveau format en utilisant la génération de Code (par exemple CodeSmith) ou un code C# personnalisé.

Il n'y a pas de meilleures pratiques pour la gestion des erreurs. Tout se résume à vos besoins et cohérente.

Voici un exemple d'une table et d'une procédure stockée qui stocke les numéros de téléphone.

 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

Il semble que vous ayez déjà une très bonne maîtrise de cela. Je soupçonne que vous faites plus de 95% des programmeurs SQL là-bas.

, Vous devriez trouver quelques informations intéressantes ici:

• détection et signalement des erreurs dans les procédures stockées-Partie 1: SQL Server 2000
• détection et signalement des erreurs dans les procédures stockées-Partie 2: SQL Server 2005 try-CATCH Blocks *

Une suggestion [sans rapport]: commencez à utiliser ' ' au lieu de '!='.

[*SQL Junkies a disparu, donc le deuxième article n'est pas disponible. Je vais essayer de le republier quelque part et mettre à jour le lien.]