Discussion :

[Djobird]

Bonjour,

je reprend un module sur le quel j'avais bossé il y'a quelques temps, et qui sauf délire de me part, marchait sans problème.

Aujourd'hui, je me retrouve à l'execution avec un tat de classCastException qui arrive lorsque j'effectue quelque chose de ce genre :

Code :

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

1
2
3
 
ArrayList<MonObj> list = remplireMaList(); 
MonObj[] tab = (MonObj[] ) list.toArray();

L'exception remontée :

Code :

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

1
2
 
java.lang.ClassCastException: [Ljava.lang.Object; cannot be cast to [pack1.MonObj;

Alors je peux modifier le coee pour avoir :

Code :

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

MonObj[] tab = (MonObj[] ) list.toArray(new MonObj[0]);

Et la ca marche. Mais c'est juste un peu pourrie comme code

Bon MonObj est une classe créé via CORBA, mais ca étend quand même d'Object. Sachant que je bosse sous Netbeans avec du java 1.6, est ce que vous avez une idée d'où peut venir ce problème ?

[Invité]

Salut,
le cast n'est pas nécessaire dans le deuxième cas, voir )]la javadoc.
Le premier code compile chez moi, si bien sûr dans ton ArrayList, tu as uniquement des objets de type MonObj... Le problème doit être autre, sur quelle ligne exactement as tu l'erreur ? Ceci dit je préfèrerais la deuxième, sans cast, car à mon sens plus sûre niveau typage et vérification par le compilateur.

A plus

[ipingu]

Code :

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

1
2
 
MonObj[] tab = (MonObj[] ) list.toArray();

Une exception est genérée parce que tu essaies de caster l'objet renvoyé par list.toArray() en MonObj[], alors que list.toArray() renvoie Object[].

Or, MonObj[] n'est pas une sous-classe de Object[].

En précisant l'objet dans lequel tu castes en paramètre de toArray(), tu évites ce problème en précisant plus fortement le type de tableau renvoyé.

Par contre, il n'est pas nécessaire de caster en MonObj[] si je ne me trompe pas. A vérifier.

Code :

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

1
2
 
MonObj[] tab = list.toArray(new MonObj[0]);

[adiGuba]

Salut,

je reprend un module sur le quel j'avais bossé il y'a quelques temps, et qui sauf délire de me part, marchait sans problème.

Aujourd'hui, je me retrouve à l'execution avec un tat de classCastException qui arrive lorsque j'effectue quelque chose de ce genre :

Code :

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

1
2
3
 
ArrayList<MonObj> list = remplireMaList(); 
MonObj[] tab = (MonObj[] ) list.toArray();

Désolé pour toi, mais c'est surement un délire de ta part !
Ce genre de code compile sans erreur mais produira toujours un ClassCastException car toArray() renvoi obligatoirement un Object[] qui n'est donc pas castable en MonObj[]...

D'ailleurs la présence du cast dans un code utilisant des Generics signale un problème potentiel !


Comme tu l'as dit, la solution est de passer par la classe toArray(array) qui permet d'utiliser un tableau du type souhaité :

Code :

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

MonObj[] tab = list.toArray(new MonObj[list.size()]);

a++

[Djobird]

Rah mais quel con, j'ai complètement inversé les logique de castage.

Merci bien pour ces infos ^^

[Invité]

Rah mais quel con, j'ai complètement inversé les logique de castage.

Merci bien pour ces infos ^^

J'ai parlé trop vite : chez moi ça compile mais ça crash au runtime (et c'est normal ). Il faut que jer evoie comment j'ai configuré mon eclipse...

Merci adiGuba

[adiGuba]

Il faut que jer evoie comment j'ai configuré mon eclipse...

Cela n'a rien à voir avec eclipse ou le compilateur ! Ce comportement est tout à fait normal.


Lorsque tu fais un cast, tu indiques au compilateur que tu sais ce que tu fais, et que tu prend en charge les possibles erreurs d'exécutions.

Le compilateur ne peut t'avertir que des cast complètement erroné... mais dans ce cas le cast reste possible selon l'implémentation de la méthode toArray()...


Il faut bien prendre conscience que même si un cast compile sans erreur, cela peut engendrer une ClassCastException à l'exécution...

Les Generics sont justement là pour éviter au maximum les casts, et donc éviter les erreurs à l'exécution en les détectant dès la compilation.


a++

[Invité]

Cela n'a rien à voir avec eclipse ou le compilateur ! Ce comportement est tout à fait normal.


Lorsque tu fais un cast, tu indiques au compilateur que tu sais ce que tu fais, et que tu prend en charge les possibles erreurs d'exécutions.

Le compilateur ne peut t'avertir que des cast complètement erroné... mais dans ce cas le cast reste possible selon l'implémentation de la méthode toArray()...


Il faut bien prendre conscience que même si un cast compile sans erreur, cela peut engendrer une ClassCastException à l'exécution...

Les Generics sont justement là pour éviter au maximum les casts, et donc éviter les erreurs à l'exécution en les détectant dès la compilation.


a++

ouais tout ça je sais, mais je réfléchissais au fait que par exemple Object[] -> String[] n'est pas complètement impossible, suivant comme tu as dit l'implémentation de toArray. J'ai eu un doute en lisant :

Une exception est genérée parce que tu essaies de caster l'objet renvoyé par list.toArray() en MonObj[], alors que list.toArray() renvoie Object[].

Or, MonObj[] n'est pas une sous-classe de Object[].

que j'avais mal interprété et je me suis donc interrogé sur ma configuration