compareTo avec les primitives -> Integer / int

Question

Est-il préférable d'écrire

int primitive1 = 3, primitive2 = 4;
Integer a = new Integer(primitive1);
Integer b = new Integer(primitive2);
int compare = a.compareTo(b);

ou

int primitive1 = 3, primitive2 = 4;
int compare = (primitive1 > primitive2) ? 1 : 0;
if(compare == 0){
    compare = (primitive1 == primitive2) ? 0 : -1;
}

Je pense que la seconde est meilleure, elle devrait être plus rapide et plus optimisée en termes de mémoire. Mais ne sont-ils pas égaux ?

Answer 1

Pour les performances, il est généralement préférable de rendre le code aussi simple et clair que possible, ce qui donne souvent de bons résultats (car le JIT optimise au mieux ce code). Dans votre cas, les exemples les plus simples sont également susceptibles d'être les plus rapides.

---

Je ferais soit

int cmp = a > b ? +1 : a < b ? -1 : 0;

ou une version plus longue

int cmp;
if (a > b)
   cmp = +1;
else if (a < b)
   cmp = -1;
else
   cmp = 0;

o

int cmp = Integer.compare(a, b); // in Java 7
int cmp = Double.compare(a, b); // before Java 7

Il est préférable de ne pas créer un objet si ce n'est pas nécessaire.

En termes de performances, le premier est le meilleur.

Si vous êtes sûr que vous n'obtiendrez pas de débordement vous pouvez utiliser

int cmp = a - b; // if you know there wont be an overflow.

tu ne pourras pas aller plus vite que ça.

Answer 2

Utilisez Integer.compare(int, int) . Et n'essayez pas de micro-optimiser votre code, sauf si vous pouvez prouver que vous avez un problème de performance.

Answer 3

Puis-je en proposer un troisième

((Integer) a).compareTo(b)

Answer 4

Transformer la primitive int en objet Integer vous coûtera un peu de mémoire, mais la différence ne sera significative que dans des cas très rares (demande de mémoire) (tableau avec 1000+ éléments). Je ne recommande pas d'utiliser le constructeur new Integer(int a) de cette façon. Ceci suffira :

Integer a = 3; 

Pour la comparaison, il existe Math.signum(double d).

compare= (int) Math.signum(a-b);

Answer 5

Ce sont déjà des ints. Pourquoi ne pas simplement utiliser la soustraction ?

compare = a - b;

Notez que Integer.compareTo() ne renvoie pas nécessairement que -1, 0 ou 1.