Discussion :

[Ogtraba]

Bonjour à tous,

Je souhaite trier un ListOrderMap en fonction de ses values.

j'ai écrit le lambda suivant :

Code :

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

1
2
3
4
 
list =  list.entrySet().stream().sorted(Entry.comparingByValue())
	     .collect(Collectors.toMap(list::get,list::getValue,
                                               (e1, e2) -> e1, ListOrderedMap::new));

et je me retrouve avec l'erreur suivante

Code :

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

1
2
 
java.lang.ClassCastException: org.apache.commons.collections.map.ListOrderedMap$ListOrderedMapEntry cannot be cast to java.lang.Integer

Avant d'utiliser un listOrderedMap, j'avais une LinkedHashMap et je faisais ceci qui fonctionné très bien :

Code :

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

1
2
3
4
5
6
 
alist = alist.entrySet().stream()
               .sorted(Entry.comparingByValue())
               .collect(Collectors.toMap(Entry::getKey, Entry::getValue, 
                                                  (e1, e2) -> e1,
                                                  LinkedHashMap<Long, JOB>::new));

J'ai constater que le problème venait de l'exécution du .collect(). Je ne sais pas vraiment ce qui ne fonctionne pas. Je sais que dans le cas de la ListOrderedMap la fonction entrySet retourne un Set et non un Set<Map.Entry<K,V>> comme pour la LinkedHashMap. Du coup j'ai utilisé list::get et list::getValue au lieu de Entry::getKey et Entry::getValue. Je suppose que le soucis vient de là. Mais je ne sais pas comment faire autrement.
Ma ListOrderedMap contient un Long comme clé et un enum comme value.

votre expertise est la bienvenu.

Ogtraba

[tchize_]

Je sais que dans le cas de la ListOrderedMap la fonction entrySet retourne un Set et non un Set<Map.Entry<K,V>> comme pour la LinkedHashMap. Du coup j'ai utilisé list::get et list::getValue au lieu de Entry::getKey et Entry::getValue. Je suppose que le soucis vient de là.

Ce n'est pas parce que ce n'est pas typé dans la signature (commons collection date d'avant les generic) que ce n'est pas le cas aussi. Ce qui s'applique à une Map s'applique aussi de manière identique aux autres Map puisqu'on utilise des méthodes faisant partie de l'interface. Donc je ne vois pas pourquoi tu essaie de tapper des lambdas diffénrentes dans le cas de ListOrderedMap, si ça marche avec une LinkedHashMap, ca devrais marcher aussi avec une ListOrderedMap

Code :

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

1
2
3
4
5
6
 
alist = alist.entrySet().stream()
               .sorted(Entry.comparingByValue())
               .collect(Collectors.toMap(Entry::getKey, Entry::getValue, 
                                                  (e1, e2) -> e1,
                                                 ListOrderedMap::new));

Pour info, ton message d'erreur viens du list::get. Comme cette méthode accepte un paramètre, elle est acceptée comme lambda. A l'exécution, on, l'appel avec l'élément courant, donc on fait un list.get(Map.Entry), ce qui évidement ne plait pas à la méthode de base qui était list.get(Integer)

[Ogtraba]

Le problème c'est que si j'utilise comme tu le suggères Entry::getKey et Entry::getValue, ça ne compile pas. j'ai l'erreur suivante.

Code :

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

1
2
 
The type Map.Entry does not define getKey(Object) that is applicable here

C'est pour cela que j'avais changé pour list::get

[adiGuba]

Salut,

C'est pour cela que j'avais changé pour list::get

Sauf que cela ne veut rien dire du tout : tu est censé fournir une méthode te retournant la clef (Long) de l'objet Entry courant... donc Entry::getKey est bien la bonne méthode.
Toi tu fourni une méthode qui attend un Long et qui retourne un JOB, ce qui est incorrect : les types ne correspondent pas !!!


Du coup je me demande : ta ListOrderedMap est bien paramétrée ?
Car sinon tu aurais dû avoir des erreurs de compilation...

Ne cherches même pas à utiliser les Streams et les lambdas sans paramétrage Generics, car tu passes à coté de tout les contrôles du compilateur !


a++

[Ogtraba]

Sauf erreur de ma part une ListOrderedMap ne se "paramétrise" pas. Du coup si je comprends bien, pas de lambda possible avec se type de map. Il faut que je fasse un tri à l'ancienne en implémentant moi même un comparable.

Pour info après test list::get retourne la clé donc un Long et list::getValue mon enum JOB.

[adiGuba]

Sauf erreur de ma part une ListOrderedMap ne se "paramétrise" pas. Du coup si je comprends bien, pas de lambda possible avec se type de map. Il faut que je fasse un tri à l'ancienne en implémentant moi même un comparable.

On parle bien de ListOrderedMap des Jakarta Commons Collections ? Dans ce cas les Generics ont été introduit dans la version 4.0...


Ce n'est pas que c'est impossible d'utiliser les Streams et les lambdas, c'est juste qu'il n'y a aucune vérification des types et que tu encours donc toutes sortes de bugs difficiles à résoudre (comme dans ton cas).

Pour info après test list::get retourne la clé donc un Long et list::getValue mon enum JOB.

Le type de retour est bon... mais pas les paramètres !
De plus cela ne fait pas ce qu'on veut.

Tu devrais avoir (Entry)->Long pour la clef et (Entry)->JOB, or avec list::get et list::getValue tu proposes (int)->Object et (int)->Object.
Tu n'obtiens pas d'erreur à la compilation car tout est Object... mais cela ne passe pas à l'exécution...



a++

[Ogtraba]

Merci pour les informations. Je vais donc partir sur un tri "à l'ancienne ".

On parle bien de ListOrderedMap des Jakarta Commons Collections ? Dans ce cas les Generics ont été introduit dans la version 4.0...

Je ne sais pas si nous parlons des mêmes ListOrderedMap j'importe ceci.

Code :

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

org.apache.commons.collections.map.ListOrderedMap