Discussion :

[Mr. Magne]

Bonjour,

N'aimant pas laisser des warnings dans mon code j'aimerais trouvé une solution à mon problème utilisant des génériques.

Voilà ma fonction

Code :

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

1
2
3
4
5
6
7
protected int doCompare(Viewer viewer, Object e1, Object e2)
{
          Comparable c1 = foo.getComparable(e1);
          Comparable c2 = foo.getComparable(e2);
 
          return c1.compareTo(c2);
}

Code :

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

public Comparable<?> getComparable(Object obj) {...}

Sachant que la méthode compareTo de Comparable<T> a pour signature

Code :

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

int compareTo(T o)

et non

Code :

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

int compareTo(Comparable<T> o)

Je précise aussi que je sais que c1 et c2 seront du même type (par exemple deux Integer (qui implémentent Comparable<Integer>), ou deux String etc..)

J'ai essayé pas mal de choses, mais rien qui plaise à mon compilo (à part SuppressWarnings("unchecked"), m'enfin bon)

Merci d'avance aux gurus qui m'apporteront leur lumières

[adiGuba]

Salut,


Le warning unchecked est normal à partir du moment où tu utilises des classes Generics sans les typé.

Ici, si toi tu sais que tes objets e1 et e2 sont bien du même type et implémentent Comparable, le compilateur n'a aucun moyen de le vérifier et te signale un problème potentiel.

Donc tu as deux solutions :

• Soit tu utilises @SuppressWarnings, qui est la seule solution lorsqu'on utilise des classes Generics sans typage...
  
  
• Soit tu types ta méthode doCompare() afin qu'elle accepte des objets typés :
  

  Code :

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

  1
  2
  3
  	protected <C extends Comparable<C>> int doCompare(Viewer viewer, C c1, C c2) {
  		return c1.compareTo(c2);
  	}

  En plus la méthode est désormais sécurisée car toute mauvaise utilisation de la méthode sera sanctionnée par un warning voir une erreur...

a++

[Mr. Magne]

Merci de cette réponse rapide
J'avais commencé à explorer du côté de ta solution mais sans succès... Cette solution est plutôt bonne, mais ma fonction a besoin d'être appelée avec 2 Object ... donc ce n'est pas possible.

[CyberChouan]

ma fonction a besoin d'être appelée avec 2 Object

Dans ce cas, crée une fonction intermédiaire qui fait un cast explicite de tes "Object" avant de les envoyer à la méthode d'Adibuba afin qu'ils respectent sa signature.

Cependant, si tu passes des "Object" à une méthode qui doit faire des comparaisons dessus, les warnings ne me paraissent pas anormaux. Ce n'est pas une manière de procéder très propre, même si tu sais à l'avance que tes objets seront du bon type...

[Mr. Magne]

J'ai aussi essayé

Code :

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

public Comparable<Comparable<?>> getComparable(Object obj)

Code :

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

1
2
3
4
5
6
7
protected int doCompare(Viewer viewer, Object e1, Object e2)
{
          Comparable<Comparable<?>> c1 = foo.getComparable(e1);
          Comparable<?> c2 = foo.getComparable(e2);
 
          return c1.compareTo(c2);
}

Qui marche plutôt bien, mais du coup j'ai du mal à voir comment faire mes casts dans ma fonction getComparable, par exemple pour caster une String en Comparable<Comparable<?>>

Une idée ?

sinon en effet la solution du supresswarning est pas mal, mais c'est pas ce que j'appelle la solution 'propre' ... si elle existe ^^