Discussion :

[abir84]

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 :

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

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 :

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

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 :

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

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.

Merci

[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...

[abir84]

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

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

[abir84]

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!!
Merci pour la réponse.

[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() ?

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

çà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é
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éé.

[abir84]

[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?
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.

...

Si tu modifies l'objet topic, alors forcément il sera modifié
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é
J'ai essayé avec clone et ça marche

Code :

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

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

Merci

[adiGuba]

C,est quoi la différence?
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 :

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

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 :

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

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

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é
J'ai essayé avec clone et ça marche

Code :

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

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

Merci

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