T-SQL Reporting Duplicate Key Exception à l'utilisateur de manière simple

Question

J'ai une simple table de pays utilisant une colonne d'identité pour la clé primaire. Il y a également des colonnes utilisées pour contenir les codes de pays ISO-3166 à 2 lettres et 3 lettres. Chacune de ces colonnes est définie comme un index unique.

Sur insertions/mises à jour Je souhaite simplement informer l'utilisateur si le code ISO entré est déjà utilisé. Je compte sur l'exception de base de données pour déclencher le processus. Je ne veux pas submerger les utilisateurs avec les détails techniques. Je veux juste lui dire que la valeur ISO entrée ne peut pas être utilisée.

Voici le T-SQL dans le Procès stocké que j'ai écrit. Il semble long et sujet à de futurs bugs et/ou à une maintenance continue au fur et à mesure que l'application se développe et que les choses changent. Y a-t-il un moyen meilleur ou plus simple? Je pourrais être un peu mouillé derrière les oreilles avec .net, mais je pensais que ce serait plus facile.

BEGIN TRY 
    UPDATE [Country] SET [CountryName] = @CountryName, [CountryISO] = @CountryISO, [CountryISO3] = @CountryISO3, [UpdateDate] = @updateDate WHERE (([CountryID] = @CountryID) AND ([RowVersion] = @Original_RowVersion)); 
END TRY 

BEGIN CATCH 
    DECLARE @ErrSeverity int, @ErrNumber int, @ErrLine int 
    DECLARE @ErrMsg nvarchar(4000) 
    SELECT @ErrSeverity = ERROR_SEVERITY(), @ErrNumber = ERROR_NUMBER(),@ErrState = ERROR_STATE(), 
     @ErrMsg = 
      CASE WHEN ERROR_NUMBER() = 2601 
      THEN 
       CASE 
       WHEN ISNULL(CHARINDEX('IX_COUNTRYISO3', ERROR_MESSAGE()), 0) > 0 
       THEN 'The 3 letter ISO-3166 value entered is already in use. Please enter a unique 3 letter ISO-3166 value.' 
       WHEN ISNULL(CHARINDEX('IX_COUNTRYISO', ERROR_MESSAGE()), 0) > 0 
       THEN 'The 2 letter ISO-3166 value entered is already in use. Please enter a unique 2 letter ISO-3166 value.' 
       ELSE 
       ERROR_MESSAGE() + '(SQL ErrNo: ' + CONVERT(varchar(50), ERROR_NUMBER()) + ')' 
       END 
      ELSE 
       ERROR_MESSAGE() + '(SQL ErrNo: ' + CONVERT(varchar(50), ERROR_NUMBER()) + ')' 
      END; 
    RAISERROR(@ErrMsg, @ErrSeverity, @ErrState) 
END CATCH 

Ma solution est-elle viable?
Quelle est la meilleure façon de partager ce code d'exception entre l'insertion et la mise à jour StoredProcs?

Je suppose que je demande essentiellement une révision de code.

Merci beaucoup Mike

Answer 1

Je suis d'accord, le code comme ça va devenir un casse-tête d'entretien. Il n'y a rien de mal avec le code en soi, c'est juste que le problème est pris trop tard.

Les contraintes/index uniques sont une «dernière ligne de défense», du moins dans mon monde. Si vous souhaitez offrir une bonne expérience utilisateur, vous ne pouvez pas attendre que les données soient déjà soumises à la base de données. Vous devez vérifier activement les doublons au niveau de l'interface utilisateur et avertir l'utilisateur qu'il est sur le point de soumettre une entrée en double (et désactiver la soumission si les clés en double ne sont pas autorisées).

Si une clé dupliquée finit par être soumise, cela décrit essentiellement un bogue dans la logique de l'application, donc il est normal que votre base de données crache un message d'erreur "base de données", vous n'avez pas besoin d'essayer pour le dire. L'application doit essayer de gérer ces erreurs aussi élégamment que possible, comme avec toute autre exception inattendue.

Si vous demandez simplement s'il existe un moyen de modulariser ce type de fonctionnalité dans T-SQL ... Je ne pense pas. Pas sans un désordre impie de SQL dynamique, de toute façon.

Mise à jour - juste vu un commentaire à l'autre réponse, si vous voulez obtenir le numéro d'erreur SQL retour en C# alors il est assez simple:

const int SqlDuplicateKeyError = 2601; 

try 
{ 
    db.Update(record); 
} 
catch (SqlException ex) 
{ 
    switch (ex.Number) 
    { 
     case SqlDuplicateKeyError: 
      // Custom error handling here 
     default: 
      throw; 
    } 
} 

Answer 2

Personnellement, j'imagine que le simple fait de donner à vos contraintes unqique/index des noms sensibles et verbeux serait une meilleure option que cela.

Si les utilisateurs interagissent directement avec la base de données, ils doivent au moins avoir une connaissance approfondie des technologies de l'information.

Si les utilisateurs interagissent via un frontal personnalisé, vous devez implémenter la gestion des erreurs dans le frontal.

Pour l'attraper à l'avant, voir ce link, un exemple de base (et l'erreur probable monté) exemple ...

Catch (SqlExpcetion ex) 
{ 
    if (e.Number == 2061 && ex.Message.Contains("IX_COUNTRYISO3") 
    { 
     MessageBox.Show("The 3 letter ISO-3166 value entered is already in use. Please enter a unique 3 letter ISO-3166 value."); 
    } 
    else 
    { 
     MessageBox.Show(String.Format("Error ({0}):{1}",ex.Number,ex.Message) 
    } 
} 
Catch (Exception ex) 
{ 
    MessageBox.Show(ex); 
}