Discussion :

[babylone7]

Bonjour,

Je souhaite trier une liste en créant un comparator portant sur des doubles.

Avec des int je procède comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
Collections.sort(
   (List) maListe,
   new Comparator() {
	public int compare(Object o1, Object o2) {
	    Produit p1 = (Produit) o1;
	    Produit p2 = (Produit) o2;
	    int i1 = 
	        p1.getPrix() != null
	        ? p1.getPrix().intValue()
	        : 0;
	     int i2 =
	        p2.getPrix() != null
	        ? p2.getPrix().intValue()
	        : 0;
	     return i1 - i2;
	}
   });

Mais aujourd'hui mes prix sont des double ... Comment faire alors ?
Pour information, j'ai besoin de travailler sur 2 décimales, je peux donc éventuellement convertir mon prix.

D'avance merci.

Flo

[ploxien]

Salut,

Je comprend mal ta question. Pourquoi ne peux tu pas simplement remplacer tes int par des double?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
Collections.sort(
                   (List) maListe,
                   new Comparator() {
                    public double compare(Object o1, Object o2) {
                        Produit p1 = (Produit) o1;
                        Produit p2 = (Produit) o2;
                        double i1 = 
                            p1.getPrix() != null
                            ? p1.getPrix().doubleValue()
                            : 0;
                         double i2 =
                            p2.getPrix() != null
                            ? p2.getPrix().doubleValue()
                            : 0;
                         return i1 - i2;
                    }
                   });

A+

[babylone7]

Car il arrondi et du coup mon tri est moyen :

Dans les logs, si j'affiche mes valeurs, j'ai :

p1.getPrix()=4.936708860759493
p2.getPrix()=15.822784810126583
i2 - i1=15 - 4= 11

Du coup voici ma liste triée par prix, le problème est en gras :

15.822784810126583
13.670886075949367
9.620253164556962
7.341772151898734
6.329113924050633
6.962025316455696
5.443037974683544
4.936708860759493
4.6835443037974684
4.430379746835443
3.9240506329113924
3.5443037974683547
2.0253164556962027

[ploxien]

Mais dans ta classe Produit, le prix est bien en double? Car si c'est le cas je ne comprend pas pourquoi il arrondi...

Vérifie bien que tout est passé en double...

J'espère t'aider

[babylone7]

Merci bcp ploxien.

Oui le prix est bien en double dans la classe Produit.
Quand je mets tout le comparator en double comme tu le suggérais j'ai l'errreur suivante :

Le type de retour n'est pas compatible avec Comparator.compare(Object, Object)

Dans le doute j'ai mis

Code :

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

return i2;

mais c'est pareil ..

[adiGuba]

Salut,


On pourrait avoir le code de ta méthode compare() avec les double ?

a++

[babylone7]

Voici

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
Collections.sort(
     (List) maListe,
     new Comparator() {
          public double compare(Object o1, Object o2) {
               Produit p1 = (Produit) o1;
               Produit p2 = (Produit) o2;
               double i1 =
                    p1.getPrix() != null
	       ? p1.getPrix().intValue()
                    : 0;
               double i2 =
                    p2.getPrix() != null
	       ? p2.getPrix().intValue()
                    : 0;
 
               System.out.println("i2 - i1="+i2+" - "+i1+"= "+(i2 - i1));
 
               return i2;
          }
     });

[chtig]

Ben tel quel c'est un peu normal que ça déconne ou je délire ? Parce que quand tu arrondis, tu risques d'avoir des comparaisons de deux double qui renvoient 0 alors que les deux doubles ne sont pas identiques, ce qui va te poser des problèmes dans ton classement.

Pour quoi tu ne fais pas

Code :

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

1
2
3
4
5
6
 
if (i1==i2)
  return 0;
if (i1<i2)
  return -1;
return 1;

Edit : j'ai posté en même temps que toi. La méthode que tu as posté est étrange ... compare ne peut pas renvoyer un double... Et en plus pourquoi tu renvoies i2 ??

[adiGuba]

la méthode compare() ne doit pas retourner un double mais un int qui doit correspondre aux valeurs -1, 0 ou 1 selon que le premier paramètre est plus petit, égal ou plus grand que le second...

De plus tu utilises intValue() à la place de doubleValue() et donc c'est logique que tu perdes la précision...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
		Collections.sort(
				   (List) maListe,
 
			     new Comparator() {
			          public int compare(Object o1, Object o2) {
			               Produit p1 = (Produit) o1;
			               Produit p2 = (Produit) o2;
			               double i1 =
			                    p1.getPrix() != null
				       ? p1.getPrix().doubleValue()
			                    : 0.0;
			               double i2 =
			                    p2.getPrix() != null
				       ? p2.getPrix().doubleValue()
			                    : 0.0;
 
				       		if (i1==i2) return 0;
				       		if (i1<i2) return -1;
				       		return 1;
			          }
			     });

a++

[babylone7]

Tout à fait chtig, j'ai ecris n'importe quoi

Voici ma méthode qui marche enfin

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
Collections.sort(
    (List) maListe,
    new Comparator() {
        public int compare(Object o1, Object o2) {
	int retour=0;
	Produit p1 = (Produit) o1;
	Produit p2 = (Produit) o2;
 
	if(p1.getPrix().doubleValue() < p2.getPrix().doubleValue())
	    retour = 1;
	if(p1.getPrix().doubleValue() > p2.getPrix().doubleValue())
	    retour = -1;
 
	return retour;
    }
});

[babylone7]

Oui merci adiGuba, j'ai fait comme ça (voir au dessus)