ArrayList et objets nuls

Bonjour ,

Je travaille depuis longtemps avec les listes en Java et c'est la premiére fois que je rencontre ce probléme. En fait j'ai créé des objets instanciés d'une classe bien déterminée et je les ajoute à ma liste.

Dans une autre méthode , j'appelle cette liste et j'essaie de récupérer ses objets mais seulement un objet est disponible ,c 'est comme si la liste ajoute toujours le nouveau objet à l'indice 0 de telle façon qu'il écrase les objets précédents.

C'est ce que j'essai de faire :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

private List<MaClasse> createList() {   List<MaClasse> list = new ArrayList<MaClasse>();   MaClasse objet1 = new MaClasse(); objet1.setXXX(val1);   list.add(objet1);   MaClasse objet2 = new MaClasse(); objet2.setXXX(val2);   list.add(objet2);   MaClasse objet3 = new MaClasse(); objet3.setXXX(val3);   list.add(objet3);     return list; }

---

Dans une autre méthode , je fais :

Code:

---

1 2 3 4 5 6

List<MaClasse> list = createList();   for(MaClasse obj: list) { System.out.println(list.indexOf(obj)); System.out.println(obj.getXXX()); }

---

J'obtiens deux objets nuls et un seul objet non nul dont l'index est 0 alors que c'est le dernier ajouté à la liste (dans ce cas obj3)

Merci pour vos réponses :) !!

Probablement la méthode equals() de ta classe MaClasse qui fait n'importe quoi.

List.indexOf() se sert de la méthode equals() de l'objet recherché, en l'appelant sur chaque élément de la List, jusqu'à en trouver un sur lequel ça fait true. C'est un peu l'intérêt de equals() en Java.
Donc si cette méthode fait n'importe quoi, List.indexOf() fera n'importe quoi aussi.

Un peu de lecture dans la FAQ concernant equals() : Pourquoi et comment redéfinir la méthode equals() ?

Au fait , j'ai essayé de faire du débuggage , maintenant je suis sur qu'à chaque fois que j'exécute la méthode add, le nouveau élément se met en position 0, c'est pour ça que finalement je récupére un seul objet et les autres sont écrasés à chaque fois que j'ajoute un nouveau élément.

Peux tu m'expliquer pourquoi le nouveau élément ne se met pas dans une nouvelle position ?

Non, je ne peux pas, c'est totalement invraisemblable.

- Sûr, tu es. Comment en es-tu sûr ? Ne dis pas "tests de débogage." Dis "Lorsque je regarde <truc> au moment <machin>, il y a <bidule> qui est à valeur <chouette>, alors qu'il devrait se passer <description>. J'en déduis qu'après add(), <description de la déduction>." Quelqu'un qui se retrouve face à un problème en faisant de l'informatique, ça parle comme ça.

- Peux-tu nous montrer le code complet, avec les imports de List et ArrayList ?

Bonjours a tous,

Je ne pense pas que l'objet se met réellement à l'indice 0. Tu précises que le programme t'affiches deux objets null et un non-null dont l'indice est 0. Ce qui signifie que ta boucle fait 3 tour et que donc tu as 3 objets dans ta liste.

Bonne journée.

-- Momoth --

Est-il possible que ta "MaClasse" soit une sorte de Singleton qui fait pointer la première instance sur la nouvelle?
Cela expliquerait la position 0 pour ton résultat.

Pour les NULL, voir comment se débrouille "MaClasse"

Je ne vois qu'un truc comme ça :O

Citation:

---

Envoyé par oni13

Est-il possible que ta "MaClasse" soit une sorte de Singleton qui fait pointer la première instance sur la nouvelle?

---

Non, ce n'est pas possible puisqu'ils sont créés par l'appel de "new" (pour un singleton, ce serait un appel à une méthode qui crée le singleton).

Sinon, concernant le probleme en lui meme, compte tenu que getXXX renvoie le dernier ajouté à la liste, je pense que c'est parce que MaClasse definit ses attributs comme étant static (tant qu'à faire, tous) et donc que c'est le dernier setXXX qui gagne (puisqu'il modifie la valeur pour toutes les instances). Et comme tous les attributs sont statiques, les objets sont tous égaux et c'est donc normal de voir toujours la ligne 0 (j'avoue que je ne vois pas le rapport avec les valeurs null mais bon, on va dire que null, c'est 0 :) )

Pour etre sur, il faudrait donner le code de MaClasse...