Copier Vecteur dans un Vecteur

Bonjour,
Mon code est le suivant:

Code:

---

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

import java.util.Collections; import java.util.Vector;   public class CopyElementsOfVectorToVectorExample {   public static void main(String[] args) {   //create first Vector object Vector v1 = new Vector();   //Add elements to Vector v1.add("1"); v1.add("2"); v1.add("3");   //create another Vector object Vector v2 = new Vector();   Collections.copy(v2,v1);   System.out.println("After copy, Second Vector Contains : " + v2); }}

---

Mais j'ai eu l'erreur suivante:

Citation:

---

Exception in thread "main" java.lang.IndexOutOfBoundsException: Source does not fit in dest
at java.util.Collections.copy(Collections.java:548)
at CopyElementsOfVectorToVectorExample.main(CopyElementsOfVectorToVectorExample.java:20)

---

Avez-vous une idée ?
Merci.

Bonjour,

Ton erreur est précisée dans la javadoc:

Citation:

---

The destination list must be at least as long as the source list.
...
IndexOutOfBoundsException - if the destination list is too small to contain the entire source List.

---

Initialiser la capacité du vecteur au départ.

Code:

---

Vector v2 = new Vector(v1.size());

---

Ou utiliser addAll de vector qui doit modifier la taille.

Code:

---

1 2 3

Vector v2 = new Vector(); v2.addAll(v1);

---

Merci :)
J'ai utilisé la deuxième solution :

Code:

---

1 2 3

Vector v2 = new Vector(); v2.addAll(v1);

---

Attention, tu ne fais absolument pas la même chose entre ces 2 codes.

Dans le premier cas, tu copies les éléments dans un autre tableau, faisant de fait ce second tableau la copie du premier. Un changement d'un élément n'impacte qu'un tableau.

Dans le second cas, tu ajoutes tout les éléments du premier tableau au second. Un changement sur un élément impacte donc les 2 tableaux.

Oui, tout à fait.
Après la copie, j'ai modifié le premier vecteur et j'ai remarqué que les deux vecteurs ont été modifié ...

Mon objectif est de garder la copie intact même si je modifie le vecteur source..
J'ai modifié le code comme suit :

Code:

---

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

import java.util.Collections; import java.util.Vector;   public class CopyElementsOfVectorToVectorExample {   public static void main(String[] args) {   //create first Vector object Vector v1 = new Vector();   //Add elements to Vector v1.add("1"); v1.add("2"); v1.add("3");   //create another Vector object Vector v2 = new Vector(v1.size());     Collections.copy(v2,v1);       System.out.println("After copy, Second Vector Contains : " + v2); }}

---

L'erreur persiste:

Citation:

---

Exception in thread "main" java.lang.IndexOutOfBoundsException: Source does not fit in dest
at java.util.Collections.copy(Collections.java:548)
at CopyElementsOfVectorToVectorExample.main(CopyElementsOfVectorToVectorExample.java:21)

---

Merci pour l'aide.

Citation:

---

Envoyé par Hizin

Attention, tu ne fais absolument pas la même chose entre ces 2 codes.

---

Euh, non. La différence est minime.
Collections.copy() assure qu'elle n'essaiera pas de modifier la taille de la liste destination, ce qui lui permet d'être toujours en temps linéaire.
addAll() modifie toujours la taille de la liste destination, puisqu'elle ne fait qu'ajouter des éléments. Il y a de nombreux cas où elle ne sera pas linéaire (bien que la différence ne soit pas énorme avec les List standard.)

Fondamentalement, Collections.copy() est bien pour remplacer le contenu d'une List. Si on a besoin de le faire souvent, c'est bien plus efficace que d'effacer d'abord et insérer ensuite.

En dehors de ça c'est pareil. Dans les deux cas si tu changes un élément de l'une, ça change l'élément de l'autre.

Citation:

---

Envoyé par chercheur111

Oui, tout à fait.
Après la copie, j'ai modifié le premier vecteur et j'ai remarqué que les deux vecteurs ont été modifié ...

---

Ça fera pareil avec Collections.copy().

Si tu ne veux pas ça, tu dois copier tous les éléments à la main.

Par exemple, si ce sont des Calendar :

Code:

---

1 2 3 4

for(Calendar cal : list1) { Calendar copy = cal.clone(); list2.add(copy); }

---

Note : de nos jours les Vectors sont obsolètes, on préfèrera utiliser ArrayList qui évite le coût de la synchronisation.
Tu devrais vérifier si tu as vraiment besoin de Vector avec sa synchronisation.

Oh ... mes excuses pour l'erreur alors. Je pensais le comporte de Collections.copy() plus proche d'une clone.

Code:

---

1 2 3 4

for(Calendar cal : list1) { Calendar copy = cal.clone(); list2.add(copy); }

---

--> Dans mon cas, je dois utiliser les vecteurs ...

Citation:

---

Si tu ne veux pas ça, tu dois copier tous les éléments à la main.

---

j'ai essayé ce code:

Code:

---

1 2	for(int i=0;i<v1.size();i++) v2.addElement(v1.get(i));

---

Le même problème persiste , si je modifie le vecteur v1 , le deuxième se modifie aussi ...

Code:

---

1 2	Vector v2 = new Vector(); v2.setSize(v1.size());

---

Corrige "Source does not fit in dest"

Citation:

---

Envoyé par chercheur111

Code:

---

1 2 3 4

for(Calendar cal : list1) { Calendar copy = cal.clone(); list2.add(copy); }

---

--> Dans mon cas, je dois utiliser les vecteurs ...

---

? Mais encore ? Tu as des Vecteurs de Vecteurs ?

Citation:

---

Envoyé par chercheur111

j'ai essayé ce code:

Code:

---

1 2	for(int i=0;i<v1.size();i++) v2.addElement(v1.get(i));

---

Le même problème persiste , si je modifie le vecteur v1 , le deuxième se modifie aussi ...

---

Oui, on te l'a dit, il faut créer un nouvel objet à chaque fois, sinon c'est ce qui se passe.