problème de lecture dans un vector

Version imprimable

Bonjour,
J'utilise un vector pour stocker des hashtable, le stockage se fait sans problèmes. Mais quand je veux parcourir le vector, il me retourne un élément qui semble ne pas exister dans le vector!

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
//ecriture
if (new_entry) {
                    topics.addElement(topic);
                    System.out.println("elements lu dans la 1ere boucle " + topics.elementAt(f));
f++;
                    topic.clear();
...
                }
//lecture
           for (int m = 0; m < topics.size(); m++) {
                    System.out.println("========= "+ " " + topics.elementAt(m));
}

Le résultat est :

Code:

1
2
3
4
elements lu dans la 1ere boucle {topic2_name=topic 3, topic1_name=topic 3}
elements lu dans la 1ere boucle {topic2_name=topic 8, topic1_name=topic 8}
elements lu dans la 1ere boucle {topic2_name=topic 9, topic1_name=topic 9}
elements lu dans la 1ere boucle {topic2_name=topic 14, topic1_name=topic 14}...

et ...

Code:

1
2
3
4
5
 
=========  {topic2_name=topic 1, topic1_name=topic 329}
=========  {topic2_name=topic 1, topic1_name=topic 329}
=========  {topic2_name=topic 1, topic1_name=topic 329}
...

J'ai essayé avec elementAt(i) en changeant i à la main; et avec lastElement() et firstElement mais ça renvoie toujours la même entrée. Pourtant le size() renvoie la bonne valeur. :calim2:

Merci

19/08/2009, 07h19
adiGuba

Salut,

Ton Vector ne semble contenir qu'un seul et unique élément... plusieurs fois !

a++

[edit] Et préfères List/ArrayList à Vector... sans oublier qu'il est préférables d'utiliser l'Iterator ou la boucle for-étendus plutôt qu'une boucle indexé pour affiché tous les éléments...
19/08/2009, 09h47
abir84

Citation:

Envoyé par adiGuba

Salut,

Ton Vector ne semble contenir qu'un seul et unique élément... plusieurs fois !

Bonjour

donc comment on peut expliquer que l'affichage est bon dans la 1ère boucle?

Merci
19/08/2009, 10h07
adiGuba

Parce que tu dois faire l'affichage juste après avoir changé la valeur de l'objet...

Il n'y a pas de mystère : si tu veux plusieurs objets ? Alors il faut créer plusieurs objets.

Je rappelle que contrairement au C++, il n'y a pas de copie implicite en Java...

a++
19/08/2009, 10h16
abir84

Citation:

Envoyé par adiGuba

Parce que tu dois faire l'affichage juste après avoir changé la valeur de l'objet...

Il n'y a pas de mystère : si tu veux plusieurs objets ? Alors il faut créer plusieurs objets.

Je rappelle que contrairement au C++, il n'y a pas de copie implicite en Java...

a++

çàd qu'à chaque fois que le contenu de topic change, le contenu change dans toute les entrées du vecteur? J'ai cru que quand je fais topics.addElement(topic), il rajoute juste le contenu de topic sans chercher si ce contenu changera par la suite!! :oops:
Merci pour la réponse.
19/08/2009, 10h18
Astartee

Il manque des éléments dans ton bout de code pour que nous puissions voir ce que tu fais :
- d'où sort l'objet "topic" ?
- que fait la fonction topic.clear() ?

Citation:

Envoyé par abir84

donc comment on peut expliquer que l'affichage est bon dans la 1ère boucle?

Déjà on peut remarquer que dans la première boucle, juste après avoir affiché un élément, tu le modifies (topic.clear()) donc ça commence mal si tu t'attends à obtenir le même résultat à l'affichage.
Ensuite, comme dit plus haut, nous ne risquons pas de deviner ce que tu fais en dehors des quelques lignes de code montrées :roll:... Si ça se trouve tu utilises toujours le même objet topic, comme l'a suggéré adiGuba au vu de l'affichage dans la deuxième boucle.

Citation:

Envoyé par abir84

çàd qu'à chaque fois que le contenu de topic change, le contenu change dans toute les entrées du vecteur? J'ai cru que quand je fais topics.addElement(topic), il rajoute juste le contenu de topic sans chercher si ce contenu changera par la suite!!

Si tu modifies l'objet topic, alors forcément il sera modifié :aie:
Quand à "copier le contenu" : qu'est-ce que ça veut dire pour toi ? Tu as un objet, tu le mets dans un vecteur... eh bien c'est cet objet, cette instance, qui est stocké dans le vecteur. Si tu ne fais pas appel à un constructeur ou à une méthode "copy" ou "clone" (qui elle-même appellera un constructeur) aucun nouvel objet ne sera créé.
19/08/2009, 19h50
abir84
Citation:

Envoyé par adiGuba

[edit] Et préfères List/ArrayList à Vector... sans oublier qu'il est préférables d'utiliser l'Iterator ou la boucle for-étendus plutôt qu'une boucle indexé pour affiché tous les éléments...

C,est quoi la différence? :oops:
En fait j'ai utilisé Vector juste parce que j'ai besoin de stocker des éléments dont je connais pas le nombre, donc j'ai besoin d'une structure qui ressemble aux tableaux mais sans connaitre la taille a priori.

Citation:

Envoyé par Astartee

...

Si tu modifies l'objet topic, alors forcément il sera modifié :aie:
Quand à "copier le contenu" : qu'est-ce que ça veut dire pour toi ? Tu as un objet, tu le mets dans un vecteur... eh bien c'est cet objet, cette instance, qui est stocké dans le vecteur. Si tu ne fais pas appel à un constructeur ou à une méthode "copy" ou "clone" (qui elle-même appellera un constructeur) aucun nouvel objet ne sera créé.

Merci, j'avais une confusion. J'ai cru que lorsque je fais un addElement(objet), un nouveau objet est construit automatiquement. JE savais pas que ça pointait vers l'objet ajouté :oops:
J'ai essayé avec clone et ça marche
Code:

1 2 topics.addElement(topic.clone()); topic.clear();
Merci :ccool:
19/08/2009, 20h44
adiGuba
Citation:

Envoyé par abir84

C,est quoi la différence? :oops:
En fait j'ai utilisé Vector juste parce que j'ai besoin de stocker des éléments dont je connais pas le nombre, donc j'ai besoin d'une structure qui ressemble aux tableaux mais sans connaitre la taille a priori.

Vector date du tout début de Java, et possède deux gros défaut :
- elle est synchronisé (et donc potentiellement moins performance si tu l'utilise en mono-thread).
- elle a une API pas très clair, avec énormément de méthode
En gros tu la remplaces par ceci :

Code:

List<TaClasseTopic> topics = new ArrayList<TaClasseTopic>();

puis tu utilises add() au lieu de addElement()...

Quand à l'iterator et la boucle étendu de Java 5, cela te permet de parcourir tous les éléments comme ceci :
Code:

1 2 3 for (TaClasseTopic topic : topics) { System.out.println("========= "+ " " + topic); }
Citation:
Envoyé par abir84

Merci, j'avais une confusion. J'ai cru que lorsque je fais un addElement(objet), un nouveau objet est construit automatiquement. JE savais pas que ça pointait vers l'objet ajouté :oops:
J'ai essayé avec clone et ça marche

Code:

1 2 topics.addElement(topic.clone()); topic.clear();

Merci :ccool:
Plutôt que de faire un clone() pas très joli. Crée simplement l'objet juste avant de l'ajouter plutôt qu'au tout début de ton code comme tu dois le faire actuellement...

a++