Pourquoi je n'arrive pas à utiliser ArrayList<MyObj>.toArray() ?

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:

---

1 2 3

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

---

L'exception remontée :

Code:

---

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

---

Alors je peux modifier le coee pour avoir :

Code:

---

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

---

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

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 ?

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

Code:

---

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:

---

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

---

Salut,

Citation:

---

Envoyé par Djobird

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:

---

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:

---

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

---

a++

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

Merci bien pour ces infos ^^

Citation:

---

Envoyé par Djobird

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...:lol:

Merci adiGuba ;)

Citation:

---

Envoyé par George7

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

---

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++

Citation:

---

Envoyé par adiGuba

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 :

Citation:

---

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 ;)