Discussion :

[Leniouns]

Bonjour,

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

Code :

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

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é

Add unimplemented methods

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

Code :

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

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.

[Tcharl]

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 :

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

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 :

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

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!

[thelvin]

Ou alors simplement :

Code :

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

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.

[Leniouns]

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