Jeter l'Exception VS Retour d'Erreur dans un Try,Catch,finally

J'question de la séparation des préoccupations. À moins que cette fonction est la partie de l'INTERFACE utilisateur, il ne doit pas se préoccuper des messages d'erreur. Il devrait être de lever des exceptions à la place. L'appelant de cette méthode, si elle fait partie de l'INTERFACE utilisateur, pourriez produire un message d'erreur à l'affichage. Si l'appelant avait un service web, il voudrait produire une erreur SOAP, qui ne peuvent pas utiliser le même message (si elle est utilisée tout message à tous).

Je voudrais aussi vous suggère fortement de vous connecter ex.ToString() et non ex.Message.

OriginalL'auteur John Saunders

Extraire les duplications de code dans une fonction, si vous vous sentez de vous répéter, est un problème.

error_code_t fail (string message) {
    logError(message);
    return someErrorMessage;
}

//...

try {    
    if (success) {
        return someSuccessMessage;
    }
    else {
        return fail("User input not correct format");
    }
}
catch (Exception ex) {
    return fail(ex.Message);
}

Pour être honnête, je ne serais pas s'inquiéter à propos de la duplication d'un couple de lignes dans la même fonction.

OriginalL'auteur dave4420

Je serai d'accord avec votre logique dans votre exemple, mais ce que exception pensez-vous de la manipulation dans votre bloc de gestion des exceptions vs votre programmatique test? Je soupçonne votre bloc de gestion des exceptions est vraiment un "juste au cas où quelque chose s'est passé". Donc il s'agit vraiment de la gestion de l'exception à la règle.

Si vous n'avez pas besoin de gérer une exception et ses pas à travers une nette architectural limite de ne pas s'en occuper. Si elle est sur le bord d'un composant limite que vous pourriez vouloir pour l'envelopper, de placer l'original à l'intérieur de l'exception.

Si la fonctionnalité a été appelé à partir de code, vous voulez tester le résultat avec une certaine représentation de l'état comme une réponse (HRESULT est un excellent exemple de cela. 0 == SUCCÈS, != 0 == échec) ou l'utilisation d'exceptions.

Tests pour les erreurs de programmation ou de défaillances de composants, vous devez utiliser les exceptions, si vous êtes à la validation des entrées de l'utilisateur sur une INTERFACE utilisateur, vous pourriez vouloir simplement utiliser la logique et les codes d'état de retour pour aider à communiquer l'erreur à l'utilisateur.

Penser enfin au sujet de la localisation. Si vous ondulation, un anglais de message d'erreur dans le système et de le présenter à votre francophone de l'utilisateur qui ne serait pas utile et que vous ne voulez pas commencer l'analyse de chaînes à votre INTERFACE utilisateur pour générer la version française, de sorte que les exceptions sont le chemin à parcourir, tant que la charge utile de l'exception dispose de suffisamment d'informations pour générer un utile message d'erreur à l'utilisateur de prendre des mesures correctives.

Utiliser le code d'état où vous avez un couplage étroit entre les composants et le composant appelant ne sait pas quoi faire dans les différentes conditions de l'état.

BTW, Vous pourriez voulez enregistrer la trace de la pile ainsi que le message à l'aide ToString() comme il vous donnera des informations plus utiles pour résoudre le problème.

HTH

OriginalL'auteur Philip