Erreur "Add unimplemented methods"

Bonjour,

J'ai un problème avec une partie de mon code que voici :

Code:

---

1 2 3 4 5 6 7 8 9

import java.util.Comparator;   Collections.sort(elementsTries, new Comparator() { public int compare(Map.Entry<String, String> e1, Map.Entry<String, String> e2) { // TODO Auto-generated method stub return ((String)e1.getValue()).compareTo((String)e2.getValue()); } });

---

Il est indiqué

Citation:

---

Add unimplemented methods

---

et lorsque j'effectue ce qui est demandé, j'obtiens :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

Collections.sort(elementsTries, new Comparator() { public int compare(Map.Entry<String, String> e1, Map.Entry<String, String> e2) { // TODO Auto-generated method stub return ((String)e1.getValue()).compareTo((String)e2.getValue()); }   @Override public int compare(Object arg0, Object arg1) { // TODO Auto-generated method stub return 0; } });

---

Suis-je obligé de faire ça ?
Et lorsque mon code se lance, quelle méthode est utilisée parmi les deux compare ?

Merci d'avance pour votre aide.

Hello,

La méthode 'compare' de 'Comparator' compare des 'Object', pas des Map de String...

Tu es donc obligé de rajouter la méthode avec la bonne signature (on la remarque car elle porte l'annotation 'Override'.
C'est celle-ci qui sera appelée, il faut donc que tu appelles l'autre et castant tes objets (leur donnant le bon type):

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

Collections.sort(elementsTries, new Comparator() { public int compare(Map.Entry<String, String> e1, Map.Entry<String, String> e2) { return ((String)e1.getValue()).compareTo((String)e2.getValue()); }   @Override public int compare(Object arg0, Object arg1) { return compare((Map.Entry<String, String>) arg0, (Map.Entry<String, String>) arg1)   } });

---

Par souci de lisibilité, tu peux aussi renommer la méthode de comparaison des Maps (en l'appelant par exemple 'compareMapOfStrings').

Tu peux aussi enchainer les Casts:

Code:

---

1 2 3 4 5

@Override public int compare(Object arg0, Object arg1) { return ((String)((Map.Entry<String, String>)e1).getValue()).compareTo((String)((Map.Entry<String, String>)e2).getValue()); }

---

Mais ça a tendance à rendre le code moins lisible.

Prochaine!

Ou alors simplement :

Code:

---

1 2 3 4 5 6 7

new Comparator<Map.Entry<String, String>>() { public int compare(Map.Entry<String, String> e1, Map.Entry<String, String> e2) { // TODO Auto-generated method stub return e1.getValue().compareTo(e2.getValue()); } }

---

Si tu ne mets pas le <Map.Entry<String, String>> cette syntaxe ne peut pas deviner que c'est ça que ton objet est censé comparer. En l'absence, il applique la règle sans génériques, donc avec des Object.

En Java 1.8, les lambda permettront une syntaxe bien plus concise qui déduira toute seule ce que tu cherches à comparer.

Ok merci à vous deux. J'ai résolu mon problème.