Existe-t-il une raison valable de coder un faux booléen sous le nom "Boolean.FALSE" en java?

Question

Les réviseurs de code se plaignent donc du code suivant:

 boolean myFlag = false;
 

Ils disent que cela devrait être:

 boolean myFlag = Boolean.FALSE;
 

Est-ce juste un fétiche de ne pas utiliser de mots-clés ou existe-t-il une raison valable de le faire?

Answer 1

Non, c'est complètement inutile. Il serait logique d'utiliser:

 // Note capital B
Boolean myFlag = Boolean.FALSE;
 

pour éviter l'appel à Boolean.valueOf (sélection automatique), mais dans votre code, il n'y a pas de boxe, et leur suggestion introduit une opération de déballage inutile.

Comme toujours, si quelqu'un suggère quelque chose et que vous ne comprenez pas pourquoi, votre premier port d'escale devrait le lui demander .

Answer 2

Il n'y a rien de mal à utiliser le mot clé false. En fait, dans votre code, il serait idiot d’utiliser Boolean.False puisqu’il faut ununboxing automatique implicite pour l’affecter à votre champ / variable primitif (Boolean.False est un booléen et non un booléen).

Answer 3

En tant que plainte, cela n’a pas beaucoup de sens puisque Boolean.FALSE n’est pas emballé dans false toute façon. Mais peut-être demander aux gens de vous dire de changer le code, pourquoi?

Answer 4

Je ne peux que penser à une raison, si vous voulez vraiment creuser pour une raison possible:

Dans un IDE comme Eclipse, vous pouvez droit-cliquez sur FALSE (en Boolean.FALSE) et sélectionnez Ouvrir Hiérarchie d'Appel ou de Références > (option). Vous ne pouvez pas faire cela avec le littéral false. Mais je ne sais pas comment utile de trouver toutes les références d' Boolean.FALSE sera dans vos cycles de développement.

Curieusement, quand j'ai cherché des références de FALSE, il a été constaté éparpillées dans la Java de base (JDK/JRE) le code source! Si nous voulons l'utiliser ou pas, une chose est sûre, le gars qui a codé et maintenir Java utilisent fortement.

Comme pour les autres réponses, il est inutile dans le cas contraire et vous pouvez simplement utiliser le littéral false, surtout si la performance est importante - en évitant l'unboxing de frais généraux. La révision du Code doit être la priorité des optimisé lisible et bien commenté code unoptimized un code lisible.