Discussion :

[coolcool29]

Bonjour,

Je travaille actuellement sur un code (que je n'ai pas écrit) qui contient quelques comparaisons d'objets telles que :
if (date1 == date2)
...
Si je ne me trompe pas, on compare ici les références d'objet, alors qu'on voudrait comparer les valeurs jour (date1.compareTo(date2) == 0).

Dans l'idéal j'aimerai faire une revue du code en entier pour repérer ce genre de comparaison.
Quelqu'un connait-il un moyen de faire cette revue?
Peut-être y a t-il un plugin d'Eclipse qui détecte ce genre de comparaisons ?

[xavlours]

Bonjour,

En utilisant le menu Search avec les expressions régulières, en cherchant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if\(([\w]+) == ([\w]+)\)

Tu devrais pouvoir trouver toutes les occurrences dans le projet entier.

Pour remplacer, en utilisant le pattern :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(\$1.equals(\$2))

Ca fera les remplacements tout seul.

[CyberChouan]

Très bonne technique, mais évite d'aller trop vite en faisant un "replace All" un peu rapide. Tu risques de remplacer les comparaisons des types de base également (int, etc.) qui risquent de ne pas apprécier

[xavlours]

Bien vu, le "Replace all" est exactement la première chose que je ferais Mais avec l'autoboxing ça devrait compiler (même si c'est pas super joli).

P.S. J'ai corrigé la regexp de détection (j'avais oublié le "+" dans la deuxième opérande).