Discussion :

[alband85]

Bonjour,

J'utilise une sérialisation AMF pour échanger des objets entre un module Flex et une application Java.
Lors de la désérialisation, j'obtiens un objet de type Object que je caste en MonObjet (connu mais pouvant être n'importe quoi héritant de Object). Lors de ce cast, j'obtiens un warning tout ce qu'il y a de plus compréhensible ("Type safety: Unchecked cast from Object to T")... mais que je ne vois pas du tout comment corriger.

Ça fonctionne, mais s'il y a un warning, c'est qu'il y a probablement quelque chose de plus élégant à faire.

Y a-t-il une solution de "blindage" générique ?

Le code de la méthode incriminée :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
try {
    final byte[] input = dec.decode(amf);
    final InputStream bIn = new ByteArrayInputStream(input);
    final Amf3Input amf3Input = new Amf3Input(getSerializationContext());
    amf3Input.setInputStream(bIn);
    T obj = (T) amf3Input.readObject(); // Et là, ça couine. 
    return obj;
} catch (final IOException e) {
    // Hop hop.
} catch (final ClassNotFoundException e) {
    // Hop hop aussi.
}

Merci !

Alban

[adiGuba]

Salut,


Le problème vient du fait que les casts s'associent mal avec les Generics.

Les Generics te garantissent un code sans erreur de type, grâce au compilateur qui a la charge de vérifier la cohérence du typeage de ton code.

Or les casts peuvent engendrer des erreurs à l'exécution. De plus comme le code des Generics est "perdu", il y a certains casts qui n'en sont pas réellement et qui sont en fait exécuté plus tard.

Bref si tu mélanges des casts et du code Generics, le compilateur ne peut plus assurer la cohérence de l'ensemble, et il t'indique cela via ce warning.



Si tu es parfaitement sûr de toi, tu peux utiliser l'annotation @SuppressWarnings pour indiquer au compilateur que tu comprends le problème et que tu prends cela en charge :

Code :

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

1
2
@SuppressWarnings("unchecked")
T obj = (T) amf3Input.readObject();

a++

[alband85]

Merci pour les précisions.

Pour tout dire, je suis sûr de l'utilisation que je vais faire de ce code, je suis sûr du code lui-même (il est testé et éprouvé)... mais j'aimerais autant éviter à mes gentils collègues de mettre le pied dans un piège à loup si je peux l'éviter.

Ceci dit, je suis prêt à ne pas utiliser les génériques s'il y a un autre moyen (simple ou presque) de le faire. Sinon, je taperai dans le suppressWarnings... ça ne sera pas la première fois (même si ça me fait toujours mal au coeur ).

[adiGuba]

Il faudrait voir le code de la méthode entière... mais en général ce n'est pas bien méchant.

Ici le problème est déplacé de quelque ligne :

Code :

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

1
2
    T obj = (T) amf3Input.readObject();
    return obj;

En cas de type incorrect, il n'y aura pas de ClassCastException sur le cast, mais uniquement après le retour de la méthode...

a++

[alband85]

En fait, toute la méthode est là. C'est une méthode utilitaire qui se tape la conversion.

Ma requête HTTP contient l'objet en AMF (encodé en base 64), c'est le paramètre d'entrée de la méthode (String amf) et je renvoie l'objet désérialisé de type T :

Code Java :

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

MonObjet obj = AmfUtil.<MonObjet> fromAmf(amf); // Avec la-dedans le code ci-dessus

[adiGuba]

Je voulais avoir toute la méthode, avec sa signature exacte.


Je suppose que tu as quelques chose comme cela :

Code :

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

public <T> T method()

Le problème c'est qu'en cas d'usage incorrect, tu peux te trimballer un T avec un type incorrect, et qui risque de te péter à la figure bien plus loin dans le code, et de manière totalement inattendu.

Pour sécuriser cela il faudrait rajouter le type Class<T> en paramètre, afin de faire un cast sécurisé :

Code :

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

public <T> T method(Class<T> baseClazz)

Puis :

Code :

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

1
2
    T obj = baseClazz.cast( amf3Input.readObject() );
    return obj;

Ce qui a le mérite de te péter à la gueule tout de suite en cas de soucis


a++