Discussion :

[L'aigle de Carthage]

Bonjour à toutes et à tous,

je voulais savoir une explication sur ce code:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 TreeSet<Integer> s = new TreeSet<Integer>();
        TreeSet<Integer> subs = new TreeSet<Integer>();
 
        for(int i=606;i<613;i++)
            if(i%2==0) s.add(i);
 
        subs = (TreeSet)s.subSet(608, true,611,true);
        subs.add(629);
        System.out.println(s+" "+subs);

L'exécution de code donne cette exception:

Exception in thread "main" java.lang.IllegalArgumentException: key out of range
at java.util.TreeMap$NavigableSubMap.put(TreeMap.java:1386)
at java.util.TreeSet.add(TreeSet.java:238)
at revcertif.Main.main(Main.java:28)
Java Result: 1

La question est pourquoi je ne peut pas ajouter la valeur 629 dans mon TreeSet alors que je peut ajouter par, exemple 611 !

Merci pour la réponse dés l'avance!

[adiGuba]

Salut,


Dans la doc de la méthode subSet() :

The returned set will throw an IllegalArgumentException on an attempt to insert an element outside its range.

Au passage ton cast en TreeSet est incorrect. Tu devrais utiliser le type NavigableSet !

a++

[danimo]

Bonjour,

Code :

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

        for(int i=606;i<613;i++)

La réponse ne viendrait pas d'ici ?

[L'aigle de Carthage]

@adiGuba: Merci pour l'éclaircissement et à propos le cast, pourquoi est-il incorrect puisque le code fonctionne correctement?

@danimo: C'est quoi le problème ici? Il s'agit d'une boucle pour insérer des valeurs pairs dans s

[adiGuba]

@adiGuba: Merci pour l'éclaircissement et à propos le cast, pourquoi est-il incorrect puisque le code fonctionne correctement?

Ca dépendra de l'implementation du Set.

Tu as eu de la chance car la méthode subSet() de TreeSet renvoi un objet héritant de TreeSet... mais ce n'est pas garanti par les spec !

Cela pourrait être n'importe quelle classe implémentant NavigableSet...

a++

[thelvin]

Autrement dit, le cast marche peut-être aujourd'hui avec la JRE employée sur l'ordinateur employé. Mais c'est un coup de chance, un simple hasard improbable.
Rien ne dit qu'il marchera demain, avec une prochaine JRE, une autre version, future ou antérieure. Personne a jamais dit que c'est censé pouvoir se faire.

Par ailleurs, lorsqu'il n'y a pas de raison de faire de cast, il est toujours mieux de ne pas le faire : code plus court, plus clair, plus maintenable, et une faiblesse en moins.

[guti11]

Tout simplement parce que l’élément 29 que tu souhaites ajouté dans subs il dépasse l'intervalle [ 608 - 611]de subSet(608, true,611,true);

Code :

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

1
2
3
 
  subs = (TreeSet)s.subSet(608, true,611,true);
        subs.add(629);