Discussion :

[ahmadou_20]

Bonjour,

J ai cree une classe Test pour laquelle j ai defini un comparatuer :

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
 
 
public class Test implements Comparable<Test > {
 
 
  private Float         number;
 
  private Integer       weight;
 
  private String        category;
 
  public Test() {
 
  }
 
 public Test(float number, int weight, float ratio, String category) {
 
    this.number= number;
    this.weight = weight;
    this.category = category;
 
  }
 
public String toString() {
 
    String result = "";
 
    result = result.concat(" -- Number-->> " + number)
                   .concat(" -- Weight -->> " + weight)
                   .concat(" -- Category -->> " + category);
 
    return result;
  }
public int compareTo(test compareTest) {
 
    if(this.number!= compareTest.number) {
      return this.number.compareTo(compareTest.number);
    }
    else if(this.weight!= compareTest.weight) {
      return this.weight.compareTo(compareTest.weight);
    }
    else if(this.category != compareTest.category) {
      return this.category.compareTo(compareTest.category);
    }
    else {
      return 0;
    }
  }
 
  public static Comparator<Test> comparator = new Comparator<Test>() {
 
                                                            public int compare(Test test1,
                                                                               Test test2) {
 
                                                              return test1.compareTo(test2);
                                                            }
 
                                                          };
 
}

En gros, j aimerais stocker dans un TreeSet differents objets Test tries par ordre ascendant suivant number, weight (si number est le meme pour deux objets) ou category (si number et weight sont les memes pour deux objets).

Le probleme c est que etant donne ce code,

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
 
 
TreeSet<Test> tree= new TreeSet<Test>(Test.comparator);
 
    Test t1 = new Test(2f, 12,"1");
    Test t2 = new Test(3f, 11, "2");
    Test t3= new Test(4f, 10, "3");
 
    tree.add(t1);
    tree.add(t2);
    tree.add(t3);
 
for(Test test : tree) {
      System.out.println(test.toString());
    }

je m attendrais a tree contenant [t1, t2, t3] alors que je n obtiens dans tree que t1.
Je comprends pas pourquoi bien que les trois objets sont tous differents et donc ne s excluent pas dans le TreeSet.

Auriez vous des idees ?
La logique du comparateur est elle correcte ?

Merci de votre aide !!!

[thelvin]

Auriez vous des idees ?

Montrer un vrai code qui compile et qui affiche des trucs, me semblerait une bonne première étape. Vraiment première. Genre avant de poser la question, il aurait déjà fallu faire ça.

En l'état :
- Il y a confusion entre Test, test, CancellationOption et le constructeur. On va pas deviner ce que cette confusion donne, appliquée à ton code qui compile vraiment et que tu n'as pas montré.
- Tu dis que tu n'obtiens que t1, mais je vois mal comment tu pourrais savoir ça. Il n'y a aucun affichage du contenu de tree, ni de quoi que ce soit d'ailleurs.

La logique du comparateur est elle correcte ?

Oui, bien qu'il soit inutile.
Tes objets implémentent déjà Comparable, et donc un TreeSet peut les trier sans devoir lui fournir un Comparator.

[ahmadou_20]

Oui desole il n ya pas de Cancellation option.

Il y que Test.

J ai corrige le code ci dessus.

mais le probleme subsiste.

desole pour le malentendu !!!!

Tes objets implémentent déjà Comparable, et donc un TreeSet peut les trier sans devoir lui fournir un Comparator.

Oui mais je veux aussi definir l ordre selon lequel je compare les champs de ma classe Test.

c est l interet de mon comparateur je pense.

[joel.drigo]

Salut,

mais le probleme subsiste.

Lequel ? Si c'est le fait que le TreeSet ne contient que l'élément t1, comme tu l'indiques dans ton premier message, je ne sais pas comment tu obtiendrais ce résultat : la classe Test présentée ne compile pas, la classe test n'existant pas (les noms de classe, entre autres, c'est sensible à la casse). Ensuite, quand bien même elle compilerait, le second extrait de code ne compilerait pas non plus : le constructeur a 4 arguments et tu n'en passe que 3. En corrigeant ces erreurs, le TreeSet contiendra bien les 3 éléments.

[thelvin]

Oui mais je veux aussi definir l ordre selon lequel je compare les champs de ma classe Test.

c est l interet de mon comparateur je pense.

Non. Ton Comparator ne définit rien du tout, il ne fait qu'appeler la méthode compareTo() de tes objets. Ça ne sert à rien, TreeSet le fera tout seul si tu ne lui donnes pas de Comparator.
Il n'a aucun intérêt.