Discussion :

[GLDavid]

Bonjour

J'ai un warning dont je souhaite me débarasser dans mon code :

Code :

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

1
2
 
Collection<PPath>list = this.selectionEventHandler.getSelection();

La méthode getSelection me retourne une collection que je veux paramétrer en objet PPath. Mais c'est insuffisant et j'ai un warning :

Code :

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

1
2
 
Type safety: The expression of type Collection needs unchecked conversion to conform to Collection<PPath>

Sans doute un manque de connaissance du langage pour java 1.5.
Quelqu'un peut-il éclairer ma lanterne ?

Merci d'avance.

@++

[mavina]

Salut,

getSelection te renvoie un objet de type Collection<PPath> ou Collection ?
Car si c'est juste Collection c'est logique qu'il te renvoie un warning

bien à toi

mavina

[GLDavid]

Hello Mavina

En fait getSelection me renvoit une Collection. Ni plus ni moins.
Quelle est alors la meilleure solution ?

@++

[adiGuba]

Salut,


Tu as ce warning car la méthode getSelection() doit renvoyer une Collection non typé. Du coup le compilateur ne peut pas t'assurer que ce code est sécurisé puisque la collection retourné peut contenir autre chose que des objet PPath...


Tu as deux solutions :

• Laisser ce warning tel quel...
• L'ignorer avec @SuppressWarnings, à condition d'être sûr que getSelection() renvoit une Collection avec des objets de type PPath seulement (si ce n'est pas le cas tu pourrais obtenir des ClassCastException un peu "bizarre").

Je te conseille toutefois de limiter au maximum l'usage de @SuppressWarnings, et d'éviter de l'utiliser pour de grande méthode mais plutôt de faire une méthode spécifique. Par exemple pour "caster" une Collection en Collection<T> :

Code :

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

1
2
3
4
    @SuppressWarnings("unchecked")
    public static <T> Collection<T> genericsCast(Collection<?> c) {
        return (Collection<T>) c;
    }

Ce qui te donnerait :

Code :

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

Collection<PPath>list = genericsCast( this.selectionEventHandler.getSelection() );

A utiliser à tes risques et périls bien sûr

a++

[GLDavid]

Hello adiGuba

A mon avis, la librairie que j'utilise (Piccolo 1.2 pour les connaisseurs) ne doit pas être encore passée en java 5.0. Je vais donc utiliser l'annotation @SuppressWarning et faire une requête auprès des auteurs de la libraire pour savoir s'ils comptent faire passer leur libraire en Java 5.0.

Merci de vos conseils.

@++