Findbugs problème avec le “Boxing/unboxing pour analyser un primitif” avec Entier.valueOf(String)

J'ai ce bout de code:

public void someMethod(String id) {
   someOtherMethod(Integer.valueOf(id));
}

public void someOtherMethod(int id) {
   //do something with id
}

Et sur la deuxième ligne, Findbugs est de lancer cette exception:

Boxing/unboxing pour analyser une primitive

Pourquoi est Findbugs se plaindre à ce sujet quand je suis tout simplement en appelant Entier.valueOf() /comment puis-je résoudre ce problème?

InformationsquelleAutor mac | 2015-09-11
  1. 51

    Le problème est que Integer.valueOf retourne un Integer, pas un int, mais votre someOtherMethod s'attend à une int. Findbugs est fondamentalement pour vous avertir que vous êtes en train de faire un long-winded manière qui implique potentiellement la création d'un objet (le Integer) que vous n'avez pas besoin de qui vous êtes alors immédiatement va unbox en la passant à someOtherMethod(int), par exemple:

    String => int => Entier => int 
^^^^^^^^^^^^^^ 
\--- C'est à l'intérieur Entier.valueOf

    Au lieu de cela, vous pouvez et devriez probablement éviter que inutile d'aller-retour à travers Integer et il suffit de faire:

    String => int 
^^^^^^^^^^^^^ 
\--- Entier.parseInt

    Il n'y a aucun besoin pour les temporaires Integer et le potentiel de l'allocation de mémoire et de tels qui l'entourent.

    Si someOtherMethod attendaient un Integer, vous n'en aurez pas l'avertissement, parce que les Integer n'est pas purement temporaire.

    C'est juste un d'une classe de l'inutile-boxe-conversions de Findbugs et des outils comme il a obligeamment remarquer.

    InformationsquelleAutor T.J. Crowder
  2. 14

    Il m'a fallu un certain temps pour comprendre que l'un (en partie parce que Jenkins a juste dit "Boxing/unboxing pour analyser un primitif"), mais apparemment, le problème /solution est dans ce Entier.valueOf() fait en interne, à savoir:

    Integer.valueOf(parseInt(s, 10));

    Donc la solution est de simplement appeler parseInt() directement à la place:

    someOtherMethod(Integer.parseInt(id));

    La description détaillée de la question (DM_BOXED_PRIMITIVE_FOR_PARSING) peuvent être trouvés sur le findbugs page.

    • ", mais apparemment, le problème / solution est dans ce Entier.valueOf() n'en interne" ce n'est pas le problème, pas de. Il ne transportent les graines de la solution, mais ce n'est pas le problème.
    • Em ... votre solution est exactement ce que j'ai dit (premier) ... à savoir utiliser Entier.parseInt() directement au lieu d'un Entier.valueOf() qui provoque le boxing/unboxing de se produire. Vous venez d'utiliser plus de mots pour l'expliquer.
    • La différence, c'est que je ne l'expliquer. Le ci-dessus dit juste "utiliser parseInt", sans une explication de pourquoi Findbugs est en pointant un problème.
    • FindBugs est ce point que sur: "Un coffret primitive est créé à partir d'une Chaîne, juste pour en extraire le "unboxed" valeur primitive. Il est plus efficace de simplement appeler le statique parseXXX méthode". Si vous avez une meilleure idée de la façon de le formuler, tirez-les demandes de sont les bienvenus.
    • Ce n'est pas une question en expliquant boxing/unboxing, c'est une question sur la façon de résoudre ce problème en particulier. Je pense toujours que vous simplement pris ma réponse et renforcé avec un peu unrelevant de l'information (à la question de toute façon).
    • Hmmm ... vous avez raison, plus d'informations peuvent être trouvées si vous creusez un peu plus, mais tous mes Jenkins construire souligné, c'est: [INFO] BugInstance taille est de 1 [INFO] Erreur de taille est de 0 [INFO] le montant Total des bugs: 1 [INFO] Boxing/unboxing pour analyser une primitive
    • Ensuite, vous ne me connaissez pas très bien. Et je ne pense pas que vous avez vraiment lire la réponse.
    • Ouais, vous avez raison, je ne sais pas vous, à tous. 😉 Et oui, j'ai lu votre réponse. De toute façon ...
    • J'ai mis à jour ma réponse avec les commentaires de @TagirValeev (merci!). Je suppose que c'est plus que beaucoup d'info (à partir de deux réponses) pour savoir comment résoudre le problème lorsque vous tombez sur elle. Dans l'avenir, je dirais TagirValeev son approche: Fournir des commentaires afin d'apporter une réponse mieux ...

    InformationsquelleAutor mac