Code complet en try / attraper bloc

• a) C'est une mauvaise pratique, de placer le code complet à l'intérieur d'un bloc d'essai. p>

  • A1) à côté des exceptions attrapantes, un bloc d'essai est une documentation dans laquelle une exception pourrait se produire. Alors placez-le près de la cause, vous avez à l'esprit. Li>
  • A2) Dans de mauvaises circonstances, vous avez un fichier à lire et ajouter plus tard un pour écrire, mais votre exception (FilenotfoundException) n'a été écrite qu'avec le premier à l'esprit. Une portée maigre autour des lieux problématiques vous aidera à identifier d'autres problèmes. li> ul> li>

  • b) Ne saisissez pas l'exception de base pour l'exhaustivité ou pour éviter plusieurs blocs de capture. Si vous souhaitez écrire dans un fichier, de nombreuses choses peuvent vous tromper: autorisation manquante, nom de fichier illégal, sans espace laissé sur le périphérique, .... Si vous présentez l'utilisateur, un message générique («Impossible d'écrire un fichier» + nom), il ne sait pas quoi faire. Soyez aussi précis que possible, et vous pouvez l'informer »seulement 20 Mo laissé sur le périphérique" + Devicename + "Nous avons besoin de 8 Mo supplémentaires (28 Mo au total); libérez-vous de l'espace et de répéter ou de choisir un autre appareil!") . Si vous attrapez une "exception", des chances sont élevées, que vous envisagez d'une exception, mais une autre se produit et n'est pas traitée correctement, car le bloc de capture n'était pas écrit avec cette possibilité. La meilleure chance de trouver cette exception est de le laisser apparaître, ou de le loger, si les journaux sont contrôlés régulièrement. p> li>

    Cela peut être une différence entre développer une application, qui est simplement utilisée par les utilisateurs finaux ou en développant une API, utilisée par d'autres développeurs. Dans une API, vous souhaitez souvent envelopper une exception dans une exception propre, afin de faciliter la gestion des utilisateurs de votre API, et si vous avez un moyen uniforme de gérer des exceptions. Si votre code peut lancer de nombreuses exceptions et conduirait au code de client laid, où votre client devra spécifier une série d'exceptions encore et encore, vous enveloppez souvent les exceptions et retirez-les: P>

    try  {
    ...
    }
    catch {FileNotFoundException fnfe}
    {
        throw new MyApiException (fnfe);
    }
    catch {PermissionDeniedException pde}
    {
        throw new MyApiException (pde);
    }
    catch {IOException  ioe}
    {
        throw new MyApiException (ioe);
    }
    

Enveloppez le code au point où vous pouvez vraiment gérer l'exception et où vous pouvez gérer l'erreur. Si vous ne pouvez pas gérer l'erreur dans la fonction, ne faites pas d'envelopper le code dans le bloc Essayer / Catch.

Je ne sais pas pour Java, mais en C ++, vous devriez attraper en const Référence: < Pré> xxx

C ++ n'est pas Java ou C # ou ... où vous avez besoin attraper (ou enfin ) clauses à nettoyer après vous-même. En C ++, Raii Est-ce que ça. Par conséquent, je n'écris rarement jamais essayer / catch en C ++, au point où je le considère comme une odeur de code.

Ainsi, plutôt que de contempler le style de code que vous devez utiliser conjointement avec essayer / attraper , vous devez vous demander si vous avez besoin de ce essayez / attrape du tout.