Discussion :

[chercheur111]

Bonjour,
Mon code est le suivant:

Code java :

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

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:

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.

[Mathieu.J]

Bonjour,

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

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 :

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

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

Ou utiliser addAll de vector qui doit modifier la taille.

Code :

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

1
2
3
 
Vector v2 = new Vector();  
v2.addAll(v1);

[chercheur111]

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

Code java :

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

1
2
3
 
Vector v2 = new Vector();  
v2.addAll(v1);

[Hizin]

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.

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

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

Code java :

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

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:

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.

[thelvin]

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.

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 :

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

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.