Éviter printStackTrace(); utiliser un enregistreur d'appel, au lieu

Dans ma demande, je suis en cours d'exécution de mon code par le biais de la PMD.Il m'indique ce message:

  • Éviter printStackTrace(); utiliser un enregistreur d'appel à la place.

Ça veut dire quoi?

InformationsquelleAutor user1305398 | 2012-05-07
  1. 107

    Cela signifie que vous devez utiliser la journalisation cadre comme logback ou log4j et au lieu de l'impression des exceptions directement:

    e.printStackTrace();

    vous devez vous connecter en utilisant cette frameworks' API:

    log.error("Ops!", e);

    La journalisation des cadres de vous donner beaucoup de souplesse, par exemple, vous pouvez choisir si vous souhaitez vous connecter à la console ou le fichier - ou peut-être passer certains messages si vous en trouvez n'est plus pertinent dans certains environnement.

    InformationsquelleAutor Tomasz Nurkiewicz
  2. 34

    Si vous appelez printStackTrace() sur une exception de la trace écrite à System.err et il est difficile de route ailleurs (ou filtre). Au lieu de faire cela, vous êtes conseillé d'utiliser une structure de journalisation (ou un wrapper autour de plusieurs journalisation des cadres, comme Apache Commons Logging) et le journal de l'exception à l'aide de ce cadre (par exemple,logger.error("some exception message", e)).

    Faire, qui vous permet de:

    • écrire le journal de déclaration à différents endroits à la fois, par exemple, la console et un fichier
    • filtrer le journal des déclarations par gravité (error, warning, info, debug, etc.) et l'origine (normalement paquet de classe ou de fonction)
    • avoir une certaine influence sur le format de journal sans avoir à modifier le code
    • etc.
    • Il suffit de dire merci ici pour les précieux commentaires qu'ailleurs 😉
    InformationsquelleAutor Thomas
  3. 16

    Une production de qualité, le programme doit utiliser l'un des nombreux journalisation des alternatives (par exemple, log4j, logback, java.util.l'exploitation forestière) pour signaler des erreurs et d'autres diagnostics. Cela a un certain nombre d'avantages:

    • Les messages du journal configurable emplacement.
    • L'utilisateur final ne voit pas les messages, sauf si vous configurez l'enregistrement afin qu'il/elle n'.
    • Vous pouvez utiliser différents enregistreurs et les niveaux d'enregistrement, etc pour contrôler la façon dont beaucoup peu ou beaucoup de journalisation est enregistré.
    • Vous pouvez utiliser différents appender formats de contrôler ce que l'enregistrement ressemble.
    • Vous pouvez facilement brancher la sortie d'enregistrement dans un plus grand contrôle /de journalisation.
    • Tous les ci-dessus peut être fait sans modification de votre code; c'est à dire en modifiant le déploiement de l'application d'enregistrement de fichier de configuration.

    En revanche, si vous utilisez seulement printStackTrace, le deployer /utilisateur final a si peu de contrôle, et la journalisation des messages sont susceptibles d'être perdus ou présentée à l'utilisateur final dans des circonstances inappropriées. (Et rien terrifie un timide utilisateur plus d'un aléatoire trace de la pile.)

    InformationsquelleAutor Stephen C
  4. 4

    En Simple,e.printStackTrace() n'est pas une bonne pratique,car elle se contente d'afficher la trace de la pile d'erreur standard. De ce fait, vous ne pouvez pas vraiment contrôler la destination de cette sortie.

    InformationsquelleAutor Sandip S.
  5. 1

    Presque toutes les fonctionnalités de journalisation fournit une méthode dans laquelle nous pouvons passer l'objet throwable, avec un message. Comme:

    public trace(Marker marker, String msg, Throwable t);

    Ils imprimer la stacktrace de l'objet throwable.

    InformationsquelleAutor abhi shukla
  6. -1

    La raison principale est que Proguard serait de supprimer le Journal des appels à partir de la production. Parce que par l'enregistrement ou l'impression StackTrace, il est possible de voir (de l'information à l'intérieur de la trace de pile ou Journal) à l'intérieur du téléphone Android, comme par exemple, Logcat application de Lecteur. De sorte que c'est une mauvaise pratique pour la sécurité. Aussi, nous n'avons pas accès au cours de la production, il serait préférable de se retiré de la production. Comme ProGuard supprimer tous les appels du Journal de ne pas stackTrace, il est donc préférable d'utiliser un Journal dans les blocs catch et de les laisser retiré de la Production par Proguard.

    InformationsquelleAutor Amirhossein Hashemi