Discussion :

[takinelinfo]

Bonjour,

Je viens de me rendre compte d'un gros problème.
J'ai l'impression que MemberwiseClone marche qu'avec les types primitifs.
Un exemple.

Code c# :

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
27
28
29
30
 
public class ObjetA
{
	public long Id { get; set; }
	public string Name { get; set; }
	public List<ObjetB> ObjetBList { get; set; }
 
	public ObjetA Clone()
	{
		return (ObjetA)MemberwiseClone();
	}
}
 
public class ObjetB
{
 
}
 
var obj1 = new ObjetA(){Name = "Test1", .....};
//obj1.ObjetBList.Count => 5
 
 
var obj2 = obj1.Clone();
obj2.Name = "Test2";
obj2.ObjetBList.Clear();
 
//obj1 => Test1
//obj2 => Test2
//obj1.ObjetBList.Count => 0
//obj2.ObjetBList.Count => 0

En fait quand je modifie l'obj1.ObjetBList cela se répercute obj2.ObjetBList et vis versa.
Hors ce n'est pas le comportement que je souhaite.
Suis je obligé de parcourir la liste et d'appliquer clone à tous?
Par avance merci.

[Aldryl]

Tu peux instancier ta liste pour ton objetB avec comme paramètre la liste de ton objetA. Dans ce cas il va faire une copie.

Du côté de MemberwiseClone, il ne permet pas de faire une copie des "sous-objets"

Code :

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

1
2
3
4
5
6
7
8
 
public ObjetA Clone()
{
   ObjetA objectA = new ObjetA();
   objectA = (ObjetA)this.MemberwiseClone();
   objectA.ObjetBList = new List<ObjetB>(this.ObjetBList);
   return objectA;
}

[tomlev]

J'ai l'impression que MemberwiseClone marche qu'avec les types primitifs.

Non, c'est pas ça, c'est juste que ça fait une copie "superficielle" (shallow copy). Ca copie les valeurs des champs, donc pour les types valeur (struct) ça marche comme on peut s'y attendre, mais pour les types référence (class), la valeur du champ est une référence, donc ça copie la référence ; ça ne clone pas l'objet pointé par la référence.

Si tu veux une copie "profonde" de l'objet, il faut l'implémenter toi-même. Pour copier la liste tu peux faire comme indiqué par Aldryl, ou utiliser la méthode d'extension ToList (ça revient au même mais c'est plus concis). Par contre, n'oublie pas que les objets contenus dans la liste ne seront pas clonés : les 2 listes contiendront des références vers les mêmes objets. Là encore, tu peux les cloner manuellement si besoin